0

我已经动态分配了一个结构，在概念上与矩阵非常相似，以保存一组字符串。我在尝试释放内存时遇到了问题。我的代码如下所示：

# include <stdio.h> 
# include <string.h>
# include <malloc.h>
# define SIZE 2

typedef struct fork{
char** dataPointersArray;
char*  dataArray;
}fork;

int main(int argc, char* argv[]){

fork forkDS;
int i;
char* dataArrayPtr;
unsigned char data[255] = "some data"; /* this is actually a function's output */
int PtrIndex;

/* allocate memory for the arrays */
    forkDS.dataPointersArray = (char**) calloc(SIZE ,sizeof(char*));

    if(forkDS.dataPointersArray == NULL){
        printf("couldn't allocate memory \n");

    }

    forkDS.dataArray = (char*) calloc(SIZE, 255);

    if( forkDS.dataArray == NULL){
        free(forkDS.dataPointersArray);
        printf("couldn't allocate memory \n");

    }
    dataArrayPtr = forkDS.dataArray;
    for(i = 0; i < SIZE; i++){
    /* update the dataPointers Array */
        forkDS.dataPointersArray[i] = dataArrayPtr;


        /* copy data into data array */
        memcpy(dataArrayPtr,data,20);

        dataArrayPtr[255] = '\0';

        /* update the pointer of the data array */
        dataArrayPtr = dataArrayPtr + 256;
    }


    for (PtrIndex = 0; PtrIndex < 2; PtrIndex++) {
        if (*(forkDS.dataPointersArray + PtrIndex) != NULL) {
            *(forkDS.dataPointersArray + PtrIndex) = NULL;
        }
    }

    /* DEBUG comment -  this 2 lines works */
    free(forkDS.dataArray); 
    forkDS.dataArray = NULL;

    /* DEBUG comment - the next line fails */
    free(forkDS.dataPointersArray);
    forkDS.dataPointersArray = NULL;


return 0;
}

所以该结构实际​​上包含 2 个数组，一个是指向字符串的指针，另一个包含一个接一个对齐的字符串，由一个终止符 \0 分隔。

代码运行良好，最后的 for 循环也运行良好。第一次调用 free 也有效。问题是最后一次调用 free 失败。尽管尝试搜索有关该问题的所有可能数据，但我发现的所有示例都考虑了保存字符串的第二个数组在 for 循环中逐步分配，然后也在 for 循环中释放的情况.

我想避免在循环中使用动态分配，因此我的代码看起来不同。

有谁知道问题是什么？

==================================================== =====================================

非常感谢所有回答我的人。最终，这个错误被解决了。问题是 dataPointersArray 在其他一些代码中填充了超过 SIZE 的元素，乍一看似乎很无辜，实际上导致免费调用失败。

再次感谢您的评论！沙查尔

4

2 回答 2

6

您正在分配 SIZE*255 字节，但使用 SIZE * 256 字节：

forkDS.dataArray = (char*) calloc(SIZE, 255); 
dataArrayPtr = forkDS.dataArray;   

//SIZE TIMES loop:
    dataArrayPtr = dataArrayPtr + 256;

因此，当您将指针归零时，您可能会覆盖由 free 正在寻找的 malloc 放置在数组末尾之后的控制数据。

于 2011-07-19T22:02:26.133 回答
0

您为一个数组分配了空间，该数组由 SIZE 行组成，每行 255 个字符。因此，每行的最高索引是 254 = 255 - 1。当你写 \0 字符时，你写在下一行的开头。在最后一次迭代之后，您将偏离 SIZE 个字节。

另一个细节：如果任何内存分配失败，程序只会打印它的错误消息，但它不会停止导致 SEGFAULT 稍后。

于 2011-07-20T20:35:56.247 回答