free() на массив структур

Question

У меня возникли проблемы с моей первой программой, использующей malloc. Моя проблема в том, что программа вылетает, когда выполняется строка free(). Я понятия не имею, почему это происходит и хотелось бы знать, как это предотвратить.

#include <stdio.h> 
#include <stdlib.h> 

struct product{ 
    int cost; 

    char thing[20]; 
}; 


int main() 
{ 
    int amount; 
    scanf("%d", &amount); 
    getchar(); 
    struct product *products; 
    products = (struct product *) malloc(amount); 
    for (int i = 0; i < amount; i++) 
    { 
     printf("Thing of %d ", (i + 1)); 
     gets(products[i].thing); 
     printf("Cost of %d: ", (i + 1)); 
     scanf("%d", &products[i].cost); 
     getchar(); 
    } 
    free(products); 
    return 0; 
} 

Answer 1

Вы не выделяете достаточное количество памяти. Оно должно быть:

products = (struct product *) malloc(amount * sizeof(struct product)); 

(таНос литая оставили в от оригинального кода, я не вводя эти дебаты.)

Answer 2

На самом деле вы malloc(), amount памяти и использует amount * sizeof(struct product). Это может сработать нормально, но когда вы вызываете free(), он вызывает сбой, как вы писали в какую-то нераспределенную память, а free() пытается освободить память, которая на самом деле не выделена для вас.

products = malloc(amount * sizeof(struct product)); // No casting too 

Там нет необходимости бросать значение, возвращаемое malloc(), так как он получает отлит неявно из void* указателя на someOther*.

Answer 3

Кроме того, не выделяя достаточное количество памяти с помощью функции malloc(), функция gets() небезопасна, устарела и поэтому никогда не должна использоваться.

Если в любой используемой точке вводится больше символов, чем буфер доступен, вы получите неопределенное поведение. Замените его на fgets(), который позволяет указать размер буфера.