从文件中读取某个字符串时程序执行中断

Question

我的代码有一个小问题，希望您能帮助我。 下面的程序读取写入 txt 文件的名称，并将它们存储在链表中，然后在命令行上打印出来。

该列表由以下名称组成：

Gustav Mahler
Frederic Chopin
Ludwig van Beethoven
Johann-Wolfgang Von-Goethe

但是当我运行程序时，程序的执行被中断，无论是在打印列表之前还是之后。

如果我删除最后一行，它会完美运行，但是当我将它添加回列表或用随机组合替换它时，例如“jlajfi3jrpiök+kvöaj3jiijm.--aerjj”，它会再次停止。

有人可以向我解释为什么程序执行会中断吗？

提前谢谢你！ :)

这是程序：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

typedef struct list {
    char* name;
    struct list *next;
}NODE;

char * getString(char *source);

int main() {

FILE *fpointer = NULL;              
char filename[100];

puts("\nEnter the name of the file:\n");
gets(filename);

if((fpointer = fopen(filename, "r")) == NULL ) {
    printf("\nThe file you have chosen is not valid.\n");
    return 1;
}

char buffer[200];
NODE *head = NULL;
NODE *current = NULL;


while(fgets(buffer, 200, fpointer) != NULL) {

    NODE *node = (NODE *) malloc(sizeof(NODE));                         
    node -> next = NULL;

    node -> name = getString(buffer);


    if(head == NULL) {
        head = node;
    } else {
        current -> next = node;
    }

    current = node;

}

current = head;
while(current) {
    printf("%s", current -> name);
    current = current -> next;
}


return 0;
}


char * getString(char* source) {
    char* target = (char*) malloc(sizeof(char));
    strcpy(target, source);
    return target;
}

Answer 1

在getString 中，您没有为要复制的字符串分配足够的空间：

char* target = (char*) malloc(sizeof(char));

这只是为单个字符分配空间。您需要足够的字符串长度，再加上 1 个空终止字节：

char* target = malloc(sizeof(strlen(source) + 1);

您实际上可以通过调用strdup 来替换整个函数，它的作用相同。

另外，don't cast the return value of malloc 和 never use gets。