以不正确的方式使用free时崩溃

如何解决以不正确的方式使用free时崩溃

typedef char* DString;
DString dstring_initialize(const char* str);
int dstring_concatenate(DString* destination,DString source);
void dstring_delete(DString* stringToDelete);

int main(void)
{
    DString str1,str2;

    str1 = dstring_initialize("hello ");
    str2 = dstring_initialize("world");
    dstring_concatenate(&str1,str2);
    dstring_delete(&str1);
    dstring_delete(&str2);

}

DString dstring_initialize(const char* str)
{
    char* res = malloc(strlen(str) + 1);
    if (res != NULL)
    {
        strcpy(res,str);
    }
    return res;
}

int dstring_concatenate(DString* destination,DString source)
{
    DString newstr = realloc(*destination,((strlen(*destination)+1)+ strlen((&source)+1))* sizeof *source);
    if(newstr == NULL)
    {
        printf("Error");
    }
    strcat(newstr,source);
    *destination = newstr;

    return 1;
}

void dstring_delete(DString* stringToDelete)
{
    assert(stringToDelete != NULL);
    free(stringToDelete);
    assert(*stringToDelete == NULL);
}

我怀疑我的分配错误或释放分配的内存不正确，因为当我调试代码时，它卡在 free(stringToDelete); 部分然后崩溃。另一件事是我怀疑 realloc 部分不正确。任何帮助将不胜感激，如果问题很糟糕，请随时发表评论，以便我提高我的沟通技巧并进一步改进问题。

解决方法

所以问题是你传递给 dstring_delete 一个 DString* 类型的变量 然后你要求它释放它，但是如果我们查看 DString* 我们发现它是 char** 但是在像 dstring_initialize 这样的其他函数中，你分配了 char* 类型的 res 所以你要求编译器释放你从未分配过的东西，这会导致你的代码崩溃： 这是新代码，检查它是否有效：

#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include <assert.h>

typedef char* DString;
DString dstring_initialize(const char* str);
int dstring_concatenate(DString* destination,DString source);
void dstring_delete(DString stringToDelete);

int main(void)
{
    DString str1,str2;

    str1 = dstring_initialize("hello ");
    str2 = dstring_initialize("world");
    dstring_concatenate(&str1,str2);
    dstring_delete(str1);
    dstring_delete(str2);

}

DString dstring_initialize(const char* str)
{
    char* res = malloc(strlen(str) + 1);
    if (res != NULL)
    {
        strcpy(res,str);
    }
    return res;
}

int dstring_concatenate(DString* destination,DString source)
{
    DString newstr = realloc(*destination,((strlen(*destination)+1)+ strlen((&source)+1))* sizeof *source);
    if(newstr == NULL)
    {
        printf("Error");
    }
    strcat(newstr,source);
    *destination = newstr;

    return 1;
}

void dstring_delete(DString stringToDelete)
{
    assert(stringToDelete != NULL);
    free(stringToDelete);
    stringToDelete = NULL;
    assert(stringToDelete == NULL);
}

另外一件事free 不会给被释放的指针赋值 NULL，它变成了垃圾数据，所以你需要在断言它是否为 NULL 之前将它赋值为 NULL

,

对于 realloc，请仔细查看此处：

 strlen((&source)+1))

您正在将 char ** 传递给 strlen。你想要的是：

 strlen(source)+1

完整的一行是：

DString newstr = realloc(*destination,(strlen(*destination) + 1 + 
                                        strlen(source) + 1) * sizeof *source);

free 的问题在于您传递的是分配的内容。而是在 main 中传递局部变量的地址。您需要取消引用：

free(*stringToDelete);

此外，释放内存不会将指针设置为 NULL，因此您应该删除 assert 之后的 free。实际上，您也可以删除另一个断言，因为将 NULL 指针传递给 free 是明确定义的。