尝试使用 valgrind 的输出调试 c 程序分段错误

Question

我的一个 CLI 程序在 Windows 上编译并运行良好。在 linux 上编译正常，但在运行时会导致分段错误。

我向 stackoverflow 寻求帮助，发现一些问题类似于我将要提出的建议 valgrind 的问题，而我恰好安装了（哇！）。

所以我通过 valgrind 运行我的程序，得到了令人沮丧的大量输出，但我将从第一条错误消息开始：

==11951== Command: ./vt
==11951== 
Loading...
Load default database? (y/n)y
Opened input file vtdb.~sv, reading contents...
==11951== Invalid write of size 1
==11951==    at 0x400FA9: readnumberfromfile (in /home/rob/Documents/programming/c/vocabtest/vt)
==11951==    by 0x400C21: getrecordsfromfile (in /home/rob/Documents/programming/c/vocabtest/vt)
==11951==    by 0x401FFD: main (in /home/rob/Documents/programming/c/vocabtest/vt)
==11951==  Address 0x53b05bb is 0 bytes after a block of size 11 alloc'd
==11951==    at 0x4C28FAC: malloc (vg_replace_malloc.c:236)
==11951==    by 0x400EAC: readnumberfromfile (in /home/rob/Documents/programming/c/vocabtest/vt)
==11951==    by 0x400C21: getrecordsfromfile (in /home/rob/Documents/programming/c/vocabtest/vt)
==11951==    by 0x401FFD: main (in /home/rob/Documents/programming/c/vocabtest/vt)
==11951== 
...finished.
1180 entries read from vtdb.~sv.

问题好像出在readnumberfromfile，我翻遍了，好像没发现什么问题！

谁能解释一下？

int readnumberfromfile (int maxvalue,char separator)
{
    int number, i=0;
    char ch;
    char * buff = (char *)malloc(11);//allocate enough space for an 10-digit number and a terminating null
    if (!buff) {printf("Memory allocation failed!\n");return 0;}//return 0 and print error if alloc failed
    if (!maxvalue) maxvalue=MAXINTVALUE;

    ch=getc(inputfile);
    while (!isdigit(ch))
    {
        if (ch == separator||ch=='\n'||ch==EOF) {fprintf(stderr,"Format error in file\n");return 0;}//if no number found(reached separator before digit), print error and return 0
        ch = getc(inputfile);//cycle forward until you reach a digit
    }
    while (i<11 && ch!=separator && ch!='\n')//stop when you reach '~', end of line, or when number too long
    {
        buff[i++]=ch;
        ch = getc(inputfile); //copy number from file to buff, one char at a time
    }
    buff[i] = '\0';//terminate string
    number = atoi(buff)<=maxvalue ? atoi(buff) : maxvalue;//convert string to number and make sure it's in range
    free(buff);
    return number;
}

如果有任何用处，则从 getrecordsfromfile 调用：

void getrecordsfromfile(char * inputfilename,char separator)
{
    int counter = 0;
    struct vocab * newvocab;
    struct listinfo * newvocablist;
    if (!(inputfile = fopen(inputfilename, "r")))
    {
        printf("Unable to read input file. File does not exist or is in use.\n");
    }    
    else
    {
        printf("Opened input file %s, reading contents...\n",inputfilename);
        while (!feof(inputfile))
        {
            newvocab = (struct vocab *)malloc(sizeof(struct vocab));
            if (!newvocab)
            {
                printf("Memory allocation failed!\n");
                return;
            }
            else
            {
                newvocab->question=readtextfromfile(MAXTEXTLENGTH,separator);
                newvocab->answer=readtextfromfile(MAXTEXTLENGTH,separator);
                newvocab->info=readtextfromfile(MAXTEXTLENGTH,separator);
                newvocab->hint=readtextfromfile(MAXTEXTLENGTH,separator);
                newvocab->right=readnumberfromfile(1,separator);
                newvocab->counter=readnumberfromfile(0,separator);
                newvocab->known=readnumberfromfile(3,separator);

                switch (newvocab->known)
                {
                    case 0: newvocablist = &n2l;break;
                    case 1: newvocablist = &norm;break;
                    case 2: newvocablist = &known;break;
                    case 3: newvocablist = &old;break;
                }

                addtolist(newvocab,newvocablist);
                if (newvocab->question==NULL||newvocab->answer==NULL)
                {
                    printf("Removing empty vocab record created from faulty input file...\n");
                    removefromlist(newvocab,newvocablist,1);
                }
                else counter++;
            }
        }
        fclose(inputfile);
        printf("...finished.\n%i entries read from %s.\n\n",counter,inputfilename);
    }
    return;
}

完整源码可以从https://github.com/megamasha/Vocab-Testergitt

一些注意事项：我正在努力帮助自己，我已经完成了我的研究，查看了类似的问题，并且我自己发现了有关 valgrind 的信息。

虽然我仍然是一个相对初学者，虽然我很欣赏解决方案（如何解决它），但更有用的是知识（下次如何自己解决或避免它）。我在这里（并且非常渴望）学习。

Answer 1

buff[i] = '\0';//terminate string

在这里 i == 11，因为您只分配了 11 个字符，而当 i=11 时，while 条件结束。
因此，您访问了未分配的内存。

没有定义这种情况的行为。

您可以通过在 malloc 上分配一个额外的字符来解决此问题。

Answer 2

int number, i=0;
...
while (i<11 ...

对于 i = 0、1、2、3、4、5、6、7、8、9 和 10，您最多可以读取 11 位数字。然后尝试将 \0 粘贴到第十二个插槽 buff[11] .

它被称为“off by one error”。

因此，修复取决于您要更改的内容。如果要接受 11 个字符，更改 buff 的 malloc。如果您只想接受 10，则更改 while 条件。

Answer 3

大小为 1 的无效写入

你可能正在写一个字符

地址 0x53b05bb 是在一个大小为 11 的块分配后的 0 个字节

你刚刚溢出了大小为 11 的东西

都在readnumberfromfile

这是可疑的相关（按尺寸）：

char * buff = (char *)malloc(11);

这将在循环之后使用i = 11 完成，这已经超过了分配的结尾：

buff[i] = '\0'

正如 wormsparty 所说，通过在二进制文件中获取调试符号，您可以让 valgrind 更有帮助。

Answer 4

稍后，如果您使用 -g 进行编译，valgrind 将准确显示段错误发生在哪一行。