I have this array of structs and this function takes a pointer to the pointer of the array. The original size is 2, so whenever it reaches the size, I need to realloc and double the size. When this code runs, I get an invalid old size error from the realloc. What am I doing wrong?
int PopulateArray(struct Inventory **inv, int *size, FILE *inputFile) {
int count = 0;
printf("address: %u\n", inv);
printf("address: %u\n", **inv);
int itemNumber;
int quantity;
float price;
int month;
int year;
while (fscanf(inputFile, "%i %i %f %i/%i", &itemNumber,
&quantity, &price, &month, &year) != EOF) {
(*inv)->itemNumber = itemNumber;
(*inv)->quantity = quantity;
(*inv)->price = price;
(*inv)->expDate.month = month;
(*inv)->expDate.year = year;
printf("count: %i size: %i\n", count, *size);
if (count == *size - 1) {
inv = realloc(inv, (*size * 2 * sizeof(struct Inventory)));
*size *= 2;
}
inv++;
count++;
}
return count;
}
In your function, inv
is (presumably) the address of a pointer variable. Its the value of that variable you want to pass to realloc
.
*inv = realloc(*inv, (*size * 2 * sizeof(struct Inventory)));
For the same reason, incrementing inv
itself won't do what you expect.
Because you need to use realloc
, you should use count
to reference the array.
while (fscanf(inputFile, "%i %i %f %i/%i", &itemNumber,
&quantity, &price, &month, &year) != EOF) {
(*inv)[count].itemNumber = itemNumber;
(*inv)[count].quantity = quantity;
(*inv)[count].price = price;
(*inv)[count].expDate.month = month;
(*inv)[count].expDate.year = year;
printf("count: %i size: %i\n", count, *size);
if (count == *size - 1) {
*inv = realloc(*inv, (*size * 2 * sizeof(struct Inventory)));
if (*inv == NULL) {
perror("realloc failed");
exit(1);
}
*size *= 2;
}
count++;
}
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With