使用 fscanf 时无法调试 seg 错误

Question

下面的 fscanf 行出现分段错误。我为每个变量添加了字符宽度以尝试修复它，所以我不明白为什么它会出现段错误。

我从一个 CSV 文件读入一个结构数组。

结构是：

typedef struct Person
    {
    int ID;
    int salary;
    int deleted;
    char salutation[4];
    char firstName[21];
    char surName[31];
    char job[16];
    } Person;

我声明了结构数组：

Person* persons;
persons = (Person*)malloc(SIZE * sizeof(Person));

然后使用这个 while 循环将 CSV 文件读入值：

(i 初始化为 0)

while(fscanf(f, "%d,%3[^,],%20[^,],%30[^,],%15[^,],%d,%d", &inPersons[i].ID, inPersons[i].salutation, inPersons[i].firstName, inPersons[i].surName, inPersons[i].job, &inPersons[i].salary, &inPersons[i].deleted)!=EOF)
            {
            newID = inPersons[i].ID;
            i++;
            }

段错误发生在 fscanf 语句期间。我没有使用 valgrind 的经验，但它给了我该行的这个错误：

==16810== Invalid write of size 4
==16810==    at 0x578215: _IO_vfscanf (in /lib/libc-2.12.so)
==16810==    by 0x585368: __isoc99_fscanf (in /lib/libc-2.12.so)
==16810==    by 0x8048951: loadDb (Database.c:23)
==16810==    by 0x8048711: menu (Menu.c:37)
==16810==    by 0x804861E: main (main.c:6)
==16810==  Address 0x27230128 is not stack'd, malloc'd or (recently) free'd

我也得到了同一行的这些 Valgrind 错误：

==18457== Use of uninitialised value of size 4
==18457==    at 0x405A215: _IO_vfscanf (in /lib/libc-2.12.so)
==18457==    by 0x4067368: __isoc99_fscanf (in /lib/libc-2.12.so)
==18457==    by 0x8048943: loadDb (Database.c:23)
==18457==    by 0x8048711: menu (Menu.c:37)
==18457==    by 0x804861E: main (main.c:6)


==18457== Process terminating with default action of signal 11 (SIGSEGV)
==18457==  Access not within mapped region at address 0x5C5E4128
==18457==    at 0x405A215: _IO_vfscanf (in /lib/libc-2.12.so)
==18457==    by 0x4067368: __isoc99_fscanf (in /lib/libc-2.12.so)
==18457==    by 0x8048943: loadDb (Database.c:23)
==18457==    by 0x8048711: menu (Menu.c:37)
==18457==    by 0x804861E: main (main.c:6)

Answer 1

如果你有一个字母或标点字符，而你应该有一个数字，你就会有一个无限循环，并且你会溢出你分配的人员数组的边界。你应该检查：

while (i < SIZE && (num = fscanf(f, "...", ...)) == 7)
{
    ...process valid input...
}
...consider what to do here, using `num` to distinguish between EOF and failed conversions...

valgrind 无法识别内存的事实令人费解。您已经采取了大部分相关步骤来确保不会出现缓冲区溢出（除了正确检查fscanf() 的状态之外）。

哦……你分配persons；你正在阅读inPersons...这是正确的？

我想我会编写一个函数来调用fscanf() 并检测和报告错误，然后从while 循环中调用它：

while (i < SIZE && get_person(&persons[i]) != EOF)
   ...

或：

while (i < SIZE && get_person(&inPersons[i]) != EOF)
   ...

这还允许您从fscanf() 切换到fgets() 和sscanf()，或getline() 和sscanf()。