释放 malloc 的结构数组时出错

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);

我使用 g++ -Wall 得到以下错误

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

任何想法可能导致这种情况？如果我离开 free(*temp_array);总之，程序编译并运行良好。免费（临时数组）；导致段错误

Answer 1

以下部分函数定义不正确：

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

您是否已将您的结构声明为structName？如果是，你需要在定义中提到参数，而不仅仅是它的类型。即使structName是你的结构，那么你应该写struct structName**而不是Struct **structName。（Struct是与 C 中的 struct 不同）

这里还有一个错误：

malloc(*size * 2 * sizeof(original_array)

您应该将*size 括在大括号中，因为不清楚您是取消引用size 还是取消引用size*2*sizeof(original_array)

Answer 2

original_array是指向指针的指针是什么原因？

无论如何，在我看来，您不应该释放 temp 数组，因为您的代码 sn-p 似乎暗示您正在用它替换 *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 的情况下调用 free 才能实现这一点。

Answer 3

释放分配的内存是内存使用的基本原则。如果你释放你没有分配的东西，它可能会导致段错误。

代码已分配内存：

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

然后它应该释放内存：

    free(*temp_array);

确保两个参数的内容都正确。