为什么在 C 中出现双重释放或损坏错误？我释放了我的 malloc

Question

我收到此错误：

`./sorter' 中的错误：双重释放或损坏 (!prev)：0x0000000000685010

然后是一堆数字，这是内存映射。 我的程序从标准输入读取电影及其属性的 CSV 文件并将其标记化。带有逗号的电影标题用引号括起来，所以我将每一行分成 3 个标记，并使用逗号作为分隔符再次标记前后标记。我在代码末尾释放了所有的 malloc，但仍然出现此错误。 csv 被扫描到最后，但我收到一条错误消息。如果我根本不释放 malloc，我不会收到错误消息，但我非常怀疑它是否正确。这是我的 main() ：

char* CSV = (char*)malloc(sizeof(char)*500);
char* fronttoken = (char*)malloc(sizeof(char)*500);
char* token = (char*)malloc(sizeof(char)*500);
char* backtoken = (char*)malloc(sizeof(char)*500);
char* title = (char*)malloc(sizeof(char)*100);
while(fgets(CSV, sizeof(CSV)*500,stdin))
{   
    fronttoken = strtok(CSV, "\"");     //store token until first quote, if no quote, store whole line
    title = strtok(NULL,"\"");          //store token after first quote until 2nd quote
    if(title != NULL)                   //if quotes in line, consume comma meant to delim title
    {
        token = strtok(NULL, ",");      //eat comma
    }
    backtoken = strtok(NULL,"\n");  //tokenize from second quote to \n character (remainder of line)

    printf("Front : %s\nTitle: %s\nBack: %s\n", fronttoken, title, backtoken);  //temp print statement to see front,back,title components

    token = strtok(fronttoken, ",");    //tokenizing front using comma delim
    while (token != NULL)
    {
        printf("%s\n", token);
        token = strtok(NULL, ",");
    }
    if (title != NULL)      //print if there is a title with comma
    {
        printf("%s\n",title);
    }   
    token = strtok(backtoken,",");  //tokenizing back using comma delim
    while (token != NULL)
    {
        printf("%s\n", token);
        token = strtok(NULL, ",");
    }
}

free(CSV);
free(token);    
free(fronttoken);
free(backtoken);
free(title);

return 0;

Answer 1

关注这里：

char* title = (char*)malloc(sizeof(char)*100);
title = strtok(NULL,"\"");

1. 您可以动态分配title 指向的内存。
2. 您将strtok 的返回值分配给title，丢失任何 引用malloc()动态分配的内存！这 意味着您肯定会发生内存泄漏，因为您将 永远无法取消分配您动态分配的内存 之前。

strtok() 的 ref 示例有一个非常丰富的示例：

/* strtok example */
#include <stdio.h>
#include <string.h>

int main ()
{
  char str[] ="- This, a sample string.";
  char * pch;
  printf ("Splitting string \"%s\" into tokens:\n",str);
  pch = strtok (str," ,.-");
  while (pch != NULL)
  {
    printf ("%s\n",pch);
    pch = strtok (NULL, " ,.-");
  }
  return 0;
}

因此，无需为 strtok() 返回的内容分配内存 - 正如我之前解释的那样，这实际上很糟糕。

回到你的代码：

free(title);

什么都不做，因为此时title 是NULL（因为strtok() 之后的while 循环。

与token 相同。

---

此外，fronttoken 和backtoken 也会导致内存泄漏，因为在调用malloc() 之后，它们被分配了strtok() 的返回值。但是它们的free() 也有问题（与title 和token 的其他取消分配相比），因为它们指向为CSV 分配的原始内存。

因此，当调用free(backtoken); 时，会发生双重释放或内存损坏。

---

此外，改变这个：

while(fgets(CSV, sizeof(CSV)*500,stdin))

到这里：

while(fgets(CSV, sizeof(*CSV)*500,stdin))

因为您想要CSV 指向的位置的大小（即您动态分配的内存大小）。