Question

我运行createNewBoard，调用createNewMatrix，我退出程序，我有一个内存泄漏，我找不到。这是代码

BoardP createNewBoard(int width, int high)
{
    BoardP board = (BoardP) malloc(sizeof(Board));

    if (board == NULL)
    {
        reportError(MEM_OUT);
        return NULL;
    }
    board->height = high;
    board->width = width;
    board->matrix = createNewMatrix(width,high);
    printf("%c",board->matrix[1][1]);
    if (board->matrix == NULL)
    {
        reportError(MEM_OUT);
        freeBoard(board);
    return NULL;
    }
return board;
}

static char** createNewMatrix(int width, int height){
    char** newMatrix = (char**) calloc(height,sizeof(char*));
    int i;
    for (i=0; i<height; i++)
    {
        newMatrix[i] = (char*) calloc(width,sizeof(char)); //LINE 71
        if (newMatrix[i] == NULL)
        {
            int j;
            for (j=0; j<i; j++)
            {
                free(newMatrix[j]);
            }
            free(newMatrix);
            return NULL;
        }
    }
    return newMatrix;
 }

这让我发疯了。我所做的就是创建一个指向Board结构的指针（它包含整数和二维指针数组）并且我有一个内存泄漏。这是消息：

==10436== HEAP SUMMARY:
==10436==     in use at exit: 100 bytes in 10 blocks
==10436==   total heap usage: 12 allocs, 2 frees, 196 bytes allocated
==10436== 
==10436== 100 bytes in 10 blocks are definitely lost in loss record 1 of 1
==10436==    at 0x4C2380C: calloc (vg_replace_malloc.c:467)
==10436==    by 0x4008C6: createNewMatrix (Board.c:71)
==10436==    by 0x40081E: createNewBoard (Board.c:55)
==10436==    by 0x4007C6: createNewDefaultBoard (Board.c:37)
==10436==    by 0x400F0C: main (PlayBoard.c:11)
==10436== 
==10436== LEAK SUMMARY:
==10436==    definitely lost: 100 bytes in 10 blocks
==10436==    indirectly lost: 0 bytes in 0 blocks
==10436==      possibly lost: 0 bytes in 0 blocks
==10436==    still reachable: 0 bytes in 0 blocks
==10436==         suppressed: 0 bytes in 0 blocks

它指向第71行，它将calloc称为矩阵中的一行。当程序退出时，它调用freeBoard：

void freeBoard(BoardP board)
{
    if (board != NULL)
    {
        if(board->matrix != NULL)
        {
            free(board->matrix);
        }
        free(board);
    }
}

为什么我有内存泄漏的任何想法？谢谢！

Answer 1

在释放matrix之前，您必须释放各行。

for (i=0; i<height; i++)
{
    free(board->matrix[i]);
}

Answer 2

对于Matrix，您可以为所有元素分配内存，但不要在函数freeBoard()中释放它们。

Answer 3

我没有看到混淆，你显然不释放任何height大小为width*sizeof(char)的数组。根据输出，height和width都是10。

Answer 4

您的免费free(board->matrix)需要释放矩阵，以便在createNewMatrix的NULL检查中释放它。

void freeBoard(BoardP board)
{
    if (board != NULL)
    {
        if(board->matrix != NULL)
        {
            for (int i = 0; i < board->height; i++)
            {
                free(board->matrix[i]);
            }
            free(board->matrix);
        }
        free(board);
    }
}

创建一个接受高度的freeMatrix函数或创建一个保持其自身高度和宽度的矩阵结构可能更容易。

Answer 5

board->matrix是指向动态分配的指针的指针。首先，您使用calloc分配第一个维度，并使用为您使用calloc分配其他维度的每个索引。因此，您有一个指向内存块的指针数组，您必须释放每个块，由每个索引指示。

在您的代码中，您只需释放第一个维度，其中每个维度都指向不同的块。另一方面，第二维中的这些块不会被释放。

你应该这样做：

void freeBoard(BoardP board)
{
    int i;
    if (board != NULL)
    {
        if(board->matrix != NULL)
        {
            for (i=0; i<board->height; i++);
               free(board->matrix[i]);
        }
        free(board);
    }
}

我尝试过in this post

在c中找不到内存泄漏

5 个答案: