Назначение нового значения для строки - C

Question

Попытка взять несколько имен файлов у пользователя в командной строке и сохранить их в связанном списке, но я получаю ошибку сегментации. Идея состоит в том, чтобы пользователь вводил каждое имя файла, а затем вводил «q», когда они были закончены.

Я ввожу первое имя файла "man_on_moon.txt" и никаких ошибок не возникает. После ввода второго «sat_moons_rings.txt» я получаю .. Ошибка сегментации: 11 Я считаю, что это происходит в моем назначении char * name, но я не уверен.

char *name = malloc(sizeof(char) *50); 
scanf("%s", name); 
list *curr, *head; 
curr = malloc(sizeof(list)); 
head = malloc(sizeof(head)); 
if(name != "q") 
{ 
    curr->item = name; 
    head->next = curr; 
    curr = curr->next; 
    scanf("%s", name); 
} 
while(name != "q") 
{ 
    curr->item = name; 
    curr = curr->next; 
    scanf("%s", name); 
} 

Answer 1

Вы никогда не заходило curr->next ни к чему, так что это указывает на то NULLcurr вы установите на curr->next. Поэтому, когда вы попадаете в цикл while, вы получаете доступ к curr->item, вы пытаетесь получить поля NULL, и вы получите segfault.

Причина этого заключается в том, что вы всего лишь mallocing space для 2 узлов. У вас есть пространство malloc для каждого узла, который вы делаете.

Answer 2

Вы должны ограничить строку Scanf поэтому он не может переполнить ваш буфер:

scanf("%49s", Name); 

Затем сравнить конец вашего цикла вы должны проверить с характером - не сравнение строк:

if (name[0] != 'q') 

..

Наконец, вам нужно скопировать ваше имя в свой список.