动态二维数组崩溃

Question

我正在从一个文件中读取（每行包含 1 个单词）并将每一行放入一个数组中。它在即将关闭文件时崩溃（* glibc检测到* proj：损坏的双链表：0x0000000002139240 ***）。除了第一个元素之外的所有内容都被正确复制（第一个元素应该是“你好吗”，但实际上是“0”）。非常感谢您对此提供任何帮助。

int i = -1;
int numb;
int wsize; 
while (fgets(word,30,file)!=NULL)
{
    if (i==-1)
    {
         if(word[strlen(word)-1]=='\n')
         {
             word[strlen(word)-1] = 0;
         }
         numb = atoi(word);
         ptr = malloc(sizeof(char*)*numb);
    }
    else
    {
        if(word[strlen(word)-1]=='\n')
        {
             word[strlen(word)-1] = 0;
        }
        wsize = strlen(word);
        ptr[i] = malloc(sizeof(char*)*wsize);
        strncpy(ptr[i],word,strlen(word));
        size++;
     }
     i++;
}
int j=0;
while(j<16)     //prints to see if they were copied corectly
{               //ptr[0] was the only one that did not copy corectly
    printf("%s\n",ptr[j]);       
    j++;
}
fclose(file);
printf("test\n"); //was never printed so I assume it crashes at fclose()
return 1;

Answer 1

ptr[i] = malloc(sizeof(char*)*wsize); 行错误，原因有二：

1. 应该是sizeof(char)，而不是sizeof(char*)（或者直接省略这个，因为sizeof(char)在定义上等于1）

2. 如果要存储长度为wsize的字符串，需要分配wsize+1字节

编辑——更多问题：

1. size++; 行的用途是什么？你是说wsize++;吗？

2. while(j<16) 中的数字 16 来自哪里？我建议你改用while(j<i)。

3. 如果main() 返回一个非零值，这表示发生了错误。将此更改为 return 0;，除非您有充分的理由返回其他值。

还有一个：

6. 我刚刚注意到您使用的是strncpy()。这不会在字符串末尾添加终止 '\0' 字节，因为您已要求它使用不超过 wsize 字节的长度复制长度为 wsize 的字符串。将此行更改为strcpy(ptr[i],word);，它应该可以工作。

完成此操作后，您需要从代码中删除所有潜在的缓冲区溢出。 （有很多。）