尽管免费，但 C 内存泄漏

Question

在使用 Valgrind 调试我的程序时，我发现了内存泄漏，尽管我认为是对 free 的有效调用。首先，分配内存并存储它的代码：

    row = malloc(sizeof(Row));
    row->columns = malloc(sizeof(char*) * headcnt);
    row->numcol  = 0;

    ...

    row->numcol    = colcnt;
    rows           = realloc(rows, (rowcnt+1) * sizeof(Row));
    rows[rowcnt++] = *row;

负责尝试释放内存的代码：

void cleanUp(){
    int i = 0;
    int j = 0;

    for (i = 0; i < rowcnt; i++){
        for (j = 0; j < rows[i].numcols; j++){
            free(rows[i].columns[j]);
        }
        free(&rows[i]);
    }
    free(rows); 
    exit(0);
}

Row 的声明：

typedef struct {
    char** columns;
    unsigned short int numcol;
} Row;

Row* rows = NULL;

更糟糕的是，这个程序有时会在free(&rows[i]) 处导致一个 glibc 错误，抱怨双重释放。我是 C 新手，希望有人能提供任何指点（咳咳）。

Answer 1

执行rows[rowcnt++] = *row; 会有效地复制您分配的内存。您的数组行应该是一个指针数组。同样就像 Oli Chalesworth 指出的那样，您对列的免费应该是对所有列的一次免费。

rows = malloc(count * sizeof(Row*)); // This is probably done somewhere

row->columns = malloc(sizeof(char*) * headcnt);
row->numcol  = 0;

...

row->numcol    = colcnt;
rows           = realloc(rows, (rowcnt+1) * sizeof(Row*));
rows[rowcnt++] = row;

现在如果你的清理工作

void cleanUp(){
    int i = 0;
    int j = 0;

    for (i = 0; i < rowcnt; i++){
        free(rows[i]->columns);
    }
    free(rows); 
    exit(0);
}

Answer 2

对malloc（或realloc）的每次调用都必须与对free 的对应调用相匹配。如果你这样动态分配一个数组：

int *p = malloc(sizeof(int) * NUM);

你可以这样释放它：

free(p);

不是这样的：

for (int i = 0; i < NUM; i++)
{
    free(p[i]);
}

您的操作似乎不正确。我怀疑你的清理代码应该是：

void cleanUp(){
    int i = 0;
    int j = 0;

    for (i = 0; i < rowcnt; i++){
        for (j = 0; j < rows[i].numcols; j++){
            free(rows[i].columns[j]); // Free whatever rows[i].columns[j] points to
        }
        free(rows[i].columns); // Matches row->columns = malloc(sizeof(char*) * headcnt);
    }
    free(rows);  // Matches rows = realloc(rows, (rowcnt+1) * sizeof(Row));
    exit(0);
}

另外，没有办法匹配row = malloc(sizeof(Row));。我怀疑您的分配代码应该是：

row->numcol    = colcnt;
rows           = realloc(rows, (rowcnt+1) * sizeof(Row));
rows[rowcnt].columns = malloc(sizeof(char*) * headcnt);
rows[rowcnt].numcol = 0;
rowcnt++;

Answer 3

也许我很密集，但这不是完全没有必要吗？无论如何，一旦程序退出，您的所有内存都会被释放。