Question

问题是我无法释放它，控制台输出指针在内部自由函数中是相同的，当调用realloc时，xcode在函数StringList_add的行中检测到。

typedef struct stringlist_s {
        int max_str;
        char **str;
}stringlist_t;

//functions
stringlist_t *StringList_new()
{
    stringlist_t *lst = (stringlist_t *)malloc(sizeof(stringlist_t));

    return lst;
}

void StringList_add(stringlist_t *str_list,char *str)
{
    if(!str)
        return;
    if(!str_list)
        return;

    str_list->str = (char **)realloc(str_list->str, sizeof(char *)  * (str_list->max_str+1));

    str_list->str[str_list->max_str] = (char *)malloc(strlen(str) + 1);

    memcpy(str_list->str[str_list->max_str], str, strlen(str) + 1);

    str_list->max_str++;
}

void StringList_release(stringlist_t *strList)
{
    if(!strList) {
        printf("Releasing empty pointer\n");
         return;
    }

    for(int i = 0 ; i < strList->max_str; ++i )
    {
        free(strList->str[i]);
        printf("pointer inside is %p\n",strList->str[i]);
    }

    printf("list before is  %p\n",strList);
    free(strList);
    printf("list  now is %p\n",strList);  //value is the same as previous printf

}

我只是用它来测试上面的代码：

stringList_t *a = StringList_new();
StringList_add(a,"abc");
StringList_add(a,"edf");
StringList_release(a);

Answer 1

问题是StringList_new()分配了一个新的stringList_t但从未初始化它的成员。致电realloc()：

str_list->str = (char **)realloc(str_list->str, sizeof(char *)  * 
    (str_list->max_str+1));

str_list->str或str_list->max_str均未初始化。来自realloc()的参考页面：

它必须先由malloc（），calloc（）或realloc（）分配，但尚未使用free（）释放，否则结果是未定义的。

与单位指针一起使用的情况。

更改为：

stringlist_t *StringList_new()
{
    stringlist_t *lst = malloc(sizeof(*lst));
    lst->max_str = 0;
    lst->str     = NULL;

    return lst;
}

不要转换malloc()或realloc()的返回值。将NULL指针传递给realloc()很好，在这种情况下它的行为类似于malloc()。使用realloc()将返回值存储在临时指针变量中以避免在realloc()失败时发生内存泄漏：

char** tmp = realloc(str_list->str, sizeof(*tmp)  * (str_list->max_str+1));
if (tmp)
{
    str_list->str = tmp;
}

注意我没有在calloc()中使用StringList_new()，因为根据C标准，所有位零都不需要表示空指针。

c中的简单字符串列表是泄漏内存（malloc和realloc）

1 个答案: