C语言fork+pipe实现管道命令功能异常排查求助

问题排查与修复方案 #

代码逻辑错误 #

现有的代码存在三个核心逻辑问题：

1. 第一个子进程未关闭不使用的管道读端p[0]，会导致管道资源无法正常释放，EOF信号传递异常
2. 父进程fork第一个子进程后直接调用wait(NULL)等待子进程退出，违背了管道并行传输的设计逻辑：如果第一个命令的输出超过管道默认缓冲区大小，会直接触发死锁；同时也会导致more这类需要交互的分页命令行为异常
3. 存在冗余的管道描述符关闭操作，多次关闭已经释放的文件描述符属于不规范写法，可能引发未知问题

输出单列的额外说明 #

你观察到的ls -la | more输出为垂直单列属于ls命令的默认特性：当ls检测到标准输出不是终端设备时，会自动切换为单列输出格式，你可以直接执行ls | cat验证该现象。如果需要强制多列输出，可以给ls增加-C参数，执行ls -laC | more即可。

修复后代码 #

#include <stdio.h>
#include <unistd.h>
#include <sys/wait.h>
#include <stdlib.h>

void handle_pipe_cmds(char **args1, char **args2){
    char ** word1 = parse_space(*args1);
    char ** word2 = parse_space(*args2);

    int p[2];
    if(pipe(p) == -1){
        perror("pipe创建失败");
        return;
    }
    
    // 启动第一个子进程：执行前序命令，写入管道
    pid_t pid1 = fork();
    if (pid1 == -1) {
        perror("第一个子进程fork失败");
        return;
    } else if (pid1 == 0) {
        close(p[0]); // 关闭不用的读端
        dup2(p[1], STDOUT_FILENO);
        close(p[1]);
        execvp(word1[0], word1);
        perror(word1[0]);
        exit(1);
    }

    // 启动第二个子进程：执行后序命令，读取管道
    pid_t pid2 = fork();
    if (pid2 == -1) {
        perror("第二个子进程fork失败");
        return;
    } else if (pid2 == 0) {
        close(p[1]); // 关闭不用的写端
        dup2(p[0], STDIN_FILENO);
        close(p[0]);
        execvp(word2[0], word2);
        perror(word2[0]);
        exit(1);
    }

    // 父进程关闭所有管道描述符
    close(p[0]);
    close(p[1]);

    // 等待两个子进程执行完成
    waitpid(pid1, NULL, 0);
    waitpid(pid2, NULL, 0);
}

核心修改点 #

• 移除了父进程第一次fork后的等待逻辑，两个子进程全部启动后再统一等待，避免死锁
• 每个子进程都主动关闭了不需要使用的管道端，保证管道结束信号能正常传递
• 增加了系统调用的错误校验，用perror输出错误信息方便定位
• 统一在父进程末尾关闭所有管道描述符，消除冗余操作

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