Ошибка при освобождении массива malloc'd структур

Question

У меня возникли проблемы с освобождением временного массива структур, используемых для удвоения размера другого массива. Кажется, у меня нет проблем с освобождением исходного массива.

void foo(StruName **structName,i nt *size) 
{ 
... 
StruName *temp_array = (StruName*) malloc(*size * 2 * sizeof(StruName)); 
for (i = 0; i < *size; i++) 
    temp_array[i] = (*original_array)[i]; 
free(*original_array); 
*original_array = temp_array; 
free(*temp_array); 

Я получаю следующее сообщение об ошибке с помощью г ++ -Wall

error: cannot convert ‘StruName’ to ‘void*’ for argument ‘1’ to ‘void free(void*)’ 

Любые идеи, что может быть причиной этого? Если я оставлю свободное (* temp_array); в целом программа компилируется и работает нормально. бесплатно (temp_array); вызывает выдаёт ошибку сегментации

Answer 1

В чем причина, почему original_array является указателем на указатель?

В любом случае, мне кажется, вы не должны освобождать массив temp, так как ваш фрагмент кода подразумевает, что вы заменяете его *original_array. Если это так, вы делаете , работая с освобожденной памятью; это может закончиться некоторое время, но в конечном итоге вы можете найти, что ваши значения массива будут перезаписаны, так как освобожденная память переназначается на что-то еще.

// here *original_array will be pointing to the new `malloc`ed memory 
*original_array = temp_array; 

// After this line, *original_array is left dangling to the recently freed memory 

free(*temp_array); 

Я бы удалил эту последнюю свободную линию.

* Редактировать * Я уже упоминал, что память будет освобождена дважды, это не так, поскольку бесплатно нужно было вызывать без разыменования * temp_array, чтобы это было правдой.

Answer 2

Следующая часть определения функции не является правильным:

void foo(Struct **structName, int count, int *size) 

Вы объявили о своей структуре, как structName? Если да, вам нужно указать и параметр, а не только его тип. И даже если ваша структура является structName, тогда вы должны написать struct structName** вместо Struct **structName. (Struct - это не то же самое, что и struct в C)

Вот еще одна ошибка:

malloc(*size * 2 * sizeof(original_array) 

вы должны заключить *size в фигурные скобки, как это не ясно, будет ли вы разыменования size или вы разыменования size*2*sizeof(original_array)

Answer 3

Это основной принцип использования памяти, который освобождает то, что вы выделили. Если вы освободите то, что вы не выделили, это может вызвать segmentfault.

код выделили память с:

StruName *temp_array = (StruName*) malloc(*size * 2 * sizeof(StruName)); 

Затем он должен освободить память с:

free(*temp_array); 

Убедитесь, что содержание обоих параметров являются правильными.