7

我正在编写一个简单的 C 程序，它打开一个文件，并将每一行读入一个数组，其中 LISTS 是我希望读取的文件中的最大行数。

但是，当 MASTER_LIST 小于 LISTS 行时，我第二次在 printf() 上遇到分段错误，但不是第一次（我已经评论显示在哪里）。

我对这种行为有点困惑，我想知道是什么原因导致它以及我能做些什么来规避它。理想情况下，当 fgets 找到文件末尾时，我会停止阅读。

#define MASTER_LIST "master_list.txt"
#define LINE 64
#define LISTS 32

char **lists = malloc(LISTS * sizeof(char *));
int i; 
for (i = 0; i < LISTS; i++) {
    lists[i] = malloc(LINE * sizeof(char));
}

/*Open the file for reading.*/
FILE *fp = fopen(MASTER_LIST, "r");
if (fp != NULL) {
    /*Each line of the file, up to LISTS is read into lists.*/
    for (i = 0; i < LISTS; i++){

        lists[i] = fgets(lists[i], LINE, fp);
        /*NO SEGFAULT HERE*/ printf("Line Read: %s\n", lists[i]);                   
    }
}

/*print out each line*/
for(i = 0; i < LISTS; i++){
    printf("Are we segfaulting yet? %d\n", i);
    /*HERE I GET A SEGFAULT*/ printf("%s\n", lists[i]);
    printf("How about now? %d\n", i);       
}
4

1 回答 1

11

fgets如果它无法读取任何字符，则返回NULL，如果您的文件包含少于LISTS行数，则会发生这种情况。尝试printfNULL 指针是未定义的行为。

这是提醒自己未定义行为确实是未定义的好时机。看起来应该第printf一次崩溃。但是，C 规范没有说明应该发生什么，所以你只是打印（例如，许多 Linux libcs​​ 都这样做）。printf(null)

那么为什么第二次 printf崩溃呢？这是因为你使用了模式

printf("%s\n", lists[i]);

许多编译器会将其优化为

puts(lists[i]);

在您的系统上，puts不检查 NULL 指针，因此会出现段错误。

故事的道德启示？永远不要依赖未定义的行为，甚至不要期望一致的结果。

于 2013-04-03T22:14:27.907 回答