C语言实现Shell多管道时无终端输出问题排查

你的代码存在两个直接触发管道死锁的核心逻辑错误，这是终端无任何输出的直接原因：

• 错误1：父进程每启动一个子进程就立刻调用waitpid()阻塞等待当前子进程退出，才会启动管道中的下一个子进程。管道是进程间实时传输数据的IPC机制，多命令管道要求所有子进程同时处于运行状态才能让数据在管道间流动，串行启动+等待的逻辑完全违背管道的工作模型。
• 错误2：父进程在fork子进程的过程中，始终持有所有管道的读写文件描述符，没有及时关闭自己不需要的管道端。根据管道的工作规则：只要任意一个进程持有管道的写端文件描述符，从该管道读端读取数据的进程就永远不会收到EOF（文件结束标识），会永久阻塞在read调用上。

你的代码实际执行路径会完全卡死：

1. 父进程创建2个管道共4个文件描述符，先fork第一个子进程执行ls -l，随即调用waitpid()阻塞，等待ls进程退出。
2. 第一个子进程把标准输出重定向到第一个管道的写端，关闭所有管道fd后执行ls，把输出写入第一个管道就退出了。但此时父进程还持有第一个管道的写端、第二个管道的读写端共3个管道fd没有关闭。
3. 父进程等到ls退出后，才fork第二个子进程执行grep i，随即再次调用waitpid()阻塞等待grep退出。
4. 第二个子进程把标准输入重定向到第一个管道的读端，标准输出重定向到第二个管道的写端，关闭所有管道fd后执行grep。grep会持续从第一个管道读数据，但因为父进程还开着第一个管道的写端，grep永远等不到输入结束的EOF信号，会一直卡死。
5. 父进程在waitpid上等待永远不会退出的grep进程，根本没机会fork第三个执行wc -l的子进程，整个程序直接死锁，自然不会有任何输出。

调整父进程的执行逻辑，遵循标准多管道实现规则即可：

1. 不要在fork循环中调用waitpid()，先完成所有子进程的fork操作，所有子进程启动完成后再统一回收所有子进程的退出状态。
2. 父进程在fork的过程中，要及时关闭自己用不到的管道描述符：每当确认某段管道的读/写端已经被子进程继承后，父进程立刻关闭自己持有的对应fd，避免残留的fd导致管道无法正常触发EOF。
3. 补全execve调用失败后的子进程退出逻辑，避免子进程执行失败后继续跑父进程的后续代码。
4. 确认命令计数逻辑正确：n条命令对应n-1个管道，不要出现差1的计数错误。

修正后核心逻辑参考片段：

// 前面创建管道的逻辑不需要修改
int r = 0;
pid_t pid;
int m;
while (gdata->cmds[r])
{
    pid = fork();
    if (pid < 0)
    {
        perror("Fork: ");
        exit(EXIT_FAILURE);
    }
    if (pid == 0)
    {
        // 子进程原有重定向逻辑基本正确，注意exec失败必须退出
        if (r > 0)
        {
            if (dup2(pipes[(r - 1) * 2], STDIN_FILENO) < 0)
            {
                perror("dup2 stdin");
                exit(EXIT_FAILURE);
            }
        }
        if (r < gdata->n_cmds - 1) // 注意替换为正确的命令总数字段，n_pipes = n_cmds - 1
        {
            if (dup2(pipes[r * 2 + 1], STDOUT_FILENO) < 0)
            {
                perror("dup2 stdout");
                exit(EXIT_FAILURE);
            }
        } 
        // 子进程关闭所有管道fd
        for (int k = 0; k < gdata->n_pipes * 2; k++)
            close(pipes[k]);
        handle_path(gdata->cmds[r], gdata->envp);
        // 走到这里说明execve执行失败，必须退出子进程
        perror("exec command");
        exit(EXIT_FAILURE);
    }
    // 父进程逻辑：fork完立刻关闭自己不需要的管道端
    if (r > 0)
        close(pipes[(r - 1) * 2]); // 上一段管道的读端已经被子进程使用，父进程无需持有
    if (r < gdata->n_cmds - 1)
        close(pipes[r * 2 + 1]); // 当前段管道的写端即将被子进程使用，父进程无需持有
    r++;
}
// 所有子进程fork完成，关闭最后一段管道的残留读端
if (gdata->n_pipes > 0)
    close(pipes[(gdata->n_pipes - 1) * 2]);
// 统一回收所有子进程退出状态
while (waitpid(-1, &m, 0) > 0)
{
    // 按需处理子进程退出状态即可
}
free(pipes);

额外提示：你代码里的cc变量和r变量完全同步自增，属于冗余变量，直接用r做索引即可，避免后续维护时两个变量计数不一致引入新bug。

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