释放 malloc 不会擦除 char 数据

Question

我为我的大问题做了一个小场景。我尝试做的是将一个字符串传递给一个函数，该函数将使用它生成一个新字符串。但是我遇到了一些问题。

我已将字符串定义为

typedef char string[1024];

然后我有一个函数，它接受一个字符串并创建一个新字符串，该字符串被旧字符串和一个点字符填充

string* add_sth_to_string(char* msg){
    string* newStr=malloc(sizeof(string)); // malloc a newStr
    strcpy(*newStr, msg);            // copying msg to newStr

    char buff[1024];                 // making a buffer
    strcat(buff, ".");               // adding a dot to buffer
    strcat(buff, *newStr);           // adding msg   to buffer
    strcpy(*newStr, buff);           // copying buffer to newStr
    return newStr;
}

然后在 main 中，我每次尝试使用这个函数 3 次来获取一个新字符串：

for (i=0; i<3; i++){
    string* newStr;
    newStr=add_sth_to_string("test");
    printf("str: %s\n", *newStr);
    free(newStr);
    // can even printf here
}

这是我得到的奇怪输出：

str: .test
str: .test.test
str: .test.test.test

当我期望得到：

str: .test
str: .test
str: .test

谁能指出发生了什么？我觉得奇怪的另一件事是，我可以在 释放 newStr 之后打印它的值。

Answer 1

您正在使用buff 而不对其进行初始化。试试：

char buff[1024];
buff[0] = 0;

我觉得奇怪的是我可以打印 newStr 的值 就在我释放它之后。

释放内存后访问它是未定义的行为。通常，出于性能原因，free 不会将内存归零。

这是同一问题中的 2 个未定义行为案例 + 一个非常奇怪的 typedef。继续加油！

Answer 2

尝试改变

strcat(buff, "."); 

进入

strcpy(buff, "."); 

或者在声明 buff 时将其初始化为：

 char buff[1024] = "";

Answer 3

您应该在每次迭代时清除buf[1024] 的内容。

---

更新

因为buf[1024] 在堆栈上分配时不会自动归零。而你选择strcat 来连接两个字符串，这将找到一个\0-terminate。因此，如果buf 包含一些默认值，则会引入混淆输出。

使用buf[1024] = ""; 分配缓冲区将更正输出。

Answer 4

string* newStr=malloc(sizeof(string)); // malloc a newStr
strcpy(*newStr, msg);            // copying msg to newStr

这也会崩溃。 string 是一个指针，因此它的 sizeof 将返回 4 或 8，而不是您想要做的。

好吧，忘记我的话，你做了一个 typedef，但我在这里告诉你为什么 typedef 是一个坏的想法。乍一看，它混淆了它是一个数组而不是指针的事实，在一个 30 行的程序上它不是问题，但是当你必须维护一个 200 000 行的项目时（就像我一样），你会开始讨厌这些之类的东西。

另外一点，您应该避免使用固定大小的 1024 字节字符串。 1024 并不是那么大（即使是 80 年代的家用计算机也有比这更大的屏幕），而且对于大多数相当短的字符串来说，你会白白浪费大量内存。