C语言中指针赋值后出现Segmentation Fault问题求助

首先，咱们直接定位你遇到的Segmentation Fault的核心原因：你没有处理链表为空的边界情况，另外循环里的指针移动逻辑也完全错了，还有单节点的判断时机不对，这些都导致了非法内存访问。

问题拆解 #

• 空链表直接访问导致崩溃
  当链表为空（head == NULL）时，(*previous)->next等价于head->next，这时候访问空指针的成员，直接触发EXC_BAD_ACCESS。而你的代码开头完全没做这个检查。
• 单节点判断的时机错误
  你写的单节点判断if(head->next == NULL && (*head).id == id)，如果链表为空的话，head->next已经先崩溃了，这个判断根本执行不到。
• 循环内指针移动逻辑完全错误
  你的循环里*previous = head->next; *current = previous;这两行完全不符合链表遍历的逻辑，会让指针指向错误的内存地址，甚至形成循环引用或者访问野指针。
• 循环未处理“找不到目标节点”的情况
  如果链表中没有对应id的节点，你的循环会一直访问current->next直到current变成NULL，这时候再访问(**current).id就会崩溃。

修复后的代码 #

#include <stdlib.h>

// 假设你的List结构体定义是这样的
typedef struct List {
    int id;
    struct List *next;
} List;

void pop(int id, List **head) {
    // 首先处理空链表的情况
    if (*head == NULL) {
        return;
    }

    List **previous = head;
    List *current = (*previous)->next; // 改成一级指针更直观，减少复杂度

    // 处理单节点的情况
    if (current == NULL && (*head)->id == id) {
        free(*head);
        *head = NULL; // 释放后置空，避免野指针
        return;
    }

    // 遍历链表找目标节点，同时检查current是否为空
    while (current != NULL && current->id != id) {
        previous = &current; // 跟踪前一个节点的指针地址
        current = current->next; // 正常遍历下一个节点
    }

    // 如果找到了目标节点再执行删除
    if (current != NULL) {
        (*previous)->next = current->next;
        free(current);
    }
}

关键修复点说明 #

• 先检查空链表：开头就判断*head == NULL直接返回，从根源避免非法访问。
• 调整单节点判断时机：确认链表非空后再判断单节点，释放后还要把*head置空，防止后续操作触发野指针问题。
• 修正循环内的指针移动：用previous = &current跟踪前一个节点的指针地址，current = current->next完成正常遍历，逻辑更清晰易懂。
• 循环增加current非空判断：避免找不到目标节点时，继续访问空指针的成员导致崩溃。
• 简化指针类型：current用一级指针就足够，没必要用二级指针，降低指针操作的出错概率。

另外你之前用printf("address = %d\n", &((*previous)->next));能输出地址，是因为那时候previous指向的head可能还不是空指针（或者刚好访问到了某个合法内存地址），但后续的逻辑错误还是会触发崩溃。

内容的提问来源于stack exchange，提问作者Jiolet Vuice