Question

我的代码已经用C语言编写，并且出现内存泄漏错误。我在许多地方都使用malloc和realloc，有没有一种方法可以检查哪些指针没有被释放？是否有任何报告可以指出我正确的方向？

此代码涉及一个指针数组，该数组包含指向结构的指针以及结构的链接列表。代码中还有其他一些动态分配。

这是声明：

user **arrMen = NULL;
int menSize = 5;
womanUser *womHead = NULL;

这就是我调用该函数的方式：

freeAll(arrMen, womHead, &menSize);

这是释放数组的方式：

void freeAll(user **arrMen, womanUser *womHead,int *menSize)
{
    womanUser *current = womHead;
    womanUser *previous = current;
    int i;
    //free all pointers in woman list
    while (current != NULL)
    {
        if (current->womData->name != NULL)
            free(current->womData->name);
        if (current->womData->lastName != NULL)
            free(current->womData->lastName);
        if (current->womData->age != NULL)
            free(current->womData->age);
        if (current->womData->userName != NULL)
            free(current->womData->userName);
        if (current->womData->pass != NULL)
            free(current->womData->pass);
        if (current->womData->desc != NULL)
            free(current->womData->desc);
        previous = current;
        current = current->next;
        free(previous);
    }
    //free all pointer in men array
    for (i = 0; i < *menSize; i++)
    {
        if (arrMen[i]->name != NULL)
            free(arrMen[i]->name);
        if (arrMen[i]->lastName != NULL)
            free(arrMen[i]->lastName);
        if (arrMen[i]->age != NULL)
            free(arrMen[i]->age);
        if (arrMen[i]->userName != NULL)
            free(arrMen[i]->userName);
        if (arrMen[i]->pass != NULL)
            free(arrMen[i]->pass);
        if (arrMen[i]->desc!= NULL)
            free(arrMen[i]->desc);
    }
    if (arrMen != NULL)
        free(arrMen);
}

程序正在按预期方式运行并运行，但是它正遭受内存泄漏的困扰，我只是找不到我做错的地方，或者我没有释放的指针在哪里。

Answer 1

诸如Valgrind（在Linux上）之类的工具是进行泄漏检测和调试的最佳选择，其次是简单而仔细的编程-每次进行分配时，都要同时对匹配的解除分配进行编码。我认为最新版本的VC ++中的调试器还将报告终止时的内存泄漏。

但是在这种特定情况下，@ jamesdlin似乎已经发现了它。您无需先释放其元素即可释放arrMen：

for (i = 0; i < *menSize; i++)
{
    free(arrMen[i]->name);
    free(arrMen[i]->lastName);
    free(arrMen[i]->age);
    free(arrMen[i]->userName);
    free(arrMen[i]->pass);
    free(arrMen[i]->desc);

    free( arrMen[i] ) ;  // <<< Free the structure
                         //     after its members
}

free(arrMen);

请注意，释放空指针已定义且安全，因此每次检查NULL都没有用。您可能要做的是将每个指针设置为 NULL，这样，如果以后任何代码尝试取消引用它，则至少会导致运行时错误。例如，您可能有：

for (i = 0; i < *menSize; i++)
{
    free(arrMen[i]->name);
    free(arrMen[i]->lastName);
    free(arrMen[i]->age);
    free(arrMen[i]->userName);
    free(arrMen[i]->pass);
    free(arrMen[i]->desc);

    memset( arrMen[i], sizeof(*arrMen[i]), 0 ) ;
    free( arrMen[i] ) ;  // <<< Free the structure
                         //     after its members
    arrMen[i] = NULL ;
}

free(arrMen);
arrMen = NULL ;

检查哪些指针未在C中释放的方法

1 个答案: