2014-10-25 3 views
3

Я пытаюсь запустить проект C для моего университетского задания, и я ударил seg-ошибку в строке «while (current-> next! = NULL) {« в следующем сегменте кода :Tricky seg fault in C

FILE* f = fileOpen("test.txt"); 
if (f != NULL){ 
    functionList = fileReadToMemory(f, &graphParams);//functionList is a pointer to the first value of the linked list it creates 
    current = functionList; 

    while (current->next != NULL) { 
     printf("%d %d %d %s", current->red, current->green, current->blue, current->expression);//Prints value of linked list 
     current = current -> next; 
    } 
} 

ошибка, что GDB дает мне выглядит следующим образом:

Program received signal EXC_BAD_ACCESS, Could not access memory. 
Reason: KERN_INVALID_ADDRESS at address: 0x000000000000003a 
0x0000000100000b30 in main() at main.c:23 
23   while (current->next != NULL) { 

что я делаю неправильно?

Заранее благодарен!

ответ

11

Вам нужно сделать

while (current != NULL) 

вместо

current->next != NULL 

, так как последний элемент в списке будет вызывать Segfault.

+0

.. который, как побочный эффект, также будет функционировать правильно, если возвращенный 'functionList' оказался' NULL'. – usr2564301

+0

Да, в этом случае глава связанного списка имеет значение NULL, поэтому while (current! = NULL) можно интерпретировать как означающий «имеет ли связанный список больше элементов?» –

+0

Да, в этом случае глава связанного списка имеет значение NULL. Таким образом, значение NULL по-прежнему является действительным связанным списком, просто он является пустым связанным списком. Таким образом, while (current! = NULL) можно интерпретировать как означающий «имеет ли связанный список больше элементов?», И он работает с связанными списками любого размера, даже с 0. –