返回一个指向指针数组的指针

Question

我在释放指针数组（值）中的指针时遇到麻烦。

typedef struct MyStruct{  char** values;  }MyStruct;

在C中，我创建了动态数组。

JSDictionary **array = (JSDictionary **)malloc(sizeof(JSDictionary *) * numRows);

resultSet应该是JSDictionary指针的数组。 我创建如下的结构：

JSDictionary * newJSDictionaryWithSize(unsigned int size)

{
JSDictionary *new = malloc(sizeof(JSDictionary));
    printf("new %p\n", self);
new->_size = size;
new->count = 0;

new->values = (char **) malloc(sizeof(char *) * size);
for(unsigned int i = 0; i < size; i++){
    new->values[i] = (char *)malloc(sizeof(char *));
}

return new;
}

一切都会创建并正常运行。 免费是给我一个问题。

void deallocJSDictionary(struct JSDictionary *self)

{
printf("dealloc %p\n", self);
for(unsigned int i = 0; i < self->_size; i++){
    printf("free %p\n", &self->values[i]);
    free(self->values[i]);
}
free(self->values);
free(self);

}

我得到一个被释放的指针未分配错误。 传入的指针显示的内存地址与我创建并添加到数组的指针相同。 值指针（在调试器中）在dealloc函数中显示的内存地址与创建时相同。 问题是尝试释放values数组中的第一点。 有什么想法为什么值指针数组中的第一点不同？

Answer 1

到目前为止，我看到两个问题。 第一：

new->values[i] = (char *)malloc(sizeof(char *));

该行分配一个指针 -可能不是您想要的。 应该可能是这样的：

new->values[i] = malloc( MAX_SIZE_OF_VALUE );

即一个缓冲区来保存该值。 您还可以分配一个大卡盘以保留所有值，并仅偏移到该内存中。

第二：

printf("free %p\n", &self->values[i]);
free(self->values[i]);

这将打印并尝试free(3)两个不同的指针。 第一个是第二个的地址。

至于实际的释放错误-在管理字典的代码中是否有这样的分配？

dict->values[i] = some_value;

如果是这样，您将使用指向可能是堆栈或静态内存的指针来覆盖指向（并泄漏）已分配内存的指针。 您必须将值复制到“值缓冲区”中。 就像是：

strlcpy( dict->values[i], some_value, MAX_SIZE_OF_VALUE );

希望这可以帮助。

Answer 2

问题可能出在您的内存分配代码中。 下一行

new->values = (char **) malloc(sizeof(char *) * size);

分配一个char指针数组，其中包含size元素。 因此，这是不必要的：

for(unsigned int i = 0; i < size; i++){
    new->values[i] = (char *)malloc(sizeof(char *));
}

您在这里所做的是在new->values数组的每个元素中存储一个转换为char *的char ** 。 也就是说，您正在分配足够的内存来存储char *值，然后在new->values数组中存储指向该内存的指针。 我猜你做错了，因为

new->values = (char **) malloc(sizeof(char *) * size);

已经为new->values数组的每个元素分配了足够的内存。

我猜你想要的是：

for(unsigned int i = 0; i < size; i++){
    new->values[i] = (char *)malloc(sizeof(char) * MAX_STRING_LENTH);
}

为new->values数组的每个元素创建一个MAX_STRING_LENGTH元素的char缓冲区。 这使您可以在每个元素中存储一个字符串，并且free代码应该可以正常工作。

或者，如果将new->values的元素设置在其他位置，则您可能不希望在解构new->values中free它们，除非它们是动态分配的，并且您知道此时不存在对其的其他引用。

Answer 3

您的功能本质上没有错。 尽管出现了一些语法错误，但它们可以很好地编译，并且产生预期的输出。 因此，正如其他人指出的那样，您可能正在将未分配的数据（即堆栈数据）分配给已分配的指针，或者您正在传递的JSDictionary结构最初没有通过newJSDictionaryWithSize正确分配。

作为一项快速测试，请仅对一个JSDictionary执行alloc / dealloc，以验证您的函数正常工作。

JSDictionary *ptr = newJSDictionaryWithSize(10);
deallocJSDictionary(ptr);