C语言实现ls-lR|sort|grep管道程序故障排查求助

看起来你的问题出在管道文件描述符没有完全关闭以及子进程等待逻辑不完整这两个关键点上，咱们一步步拆解：

1. 未关闭所有未使用的管道文件描述符 #

每个子进程都会继承父进程打开的所有管道文件描述符，但很多fd是子进程用不到的。如果不关闭这些多余的fd，会导致管道的读/写端一直处于"未完全关闭"的状态——下游进程会一直认为可能还有数据输入，永远不会主动退出。

比如：

• ls子进程只需要用到lssort[1]来输出结果，但它还继承了sortgrep的两个fd，这会导致sortgrep管道的写端始终有一个打开的句柄。即使sort进程已经结束，grep进程也会一直等待，不会退出。
• 同理，sort和grep子进程也需要关闭所有自己用不到的管道fd。

2. wait()调用不完整 #

你只调用了两次wait(0)，但实际fork了三个子进程。第三个子进程的状态会留在系统中变成僵尸进程，同时父进程的等待逻辑不完整，可能导致程序陷入无意义的阻塞。

3. 文件打开的权限问题 #

用open(outfile, O_CREAT | O_RDWR)时，必须指定第三个参数（文件权限，比如0644），否则创建出来的文件权限是随机的，可能导致后续读写异常，甚至无法正常访问文件。

修正后的代码 #

下面是修复后的完整代码片段，我标注了所有关键修改点：

// ls -lR [dir] | sort | grep string [ >outfile]
// -------------^------^------------------------
int lssort[2];
int sortgrep[2];
// 增加管道创建错误检查
if (pipe(lssort) == -1 || pipe(sortgrep) == -1) {
    perror("pipe failed");
    exit(EXIT_FAILURE);
}

// 第一个子进程：ls -lR
if (fork() == 0) {
    // 关闭所有未用到的管道端
    close(lssort[0]);
    close(sortgrep[0]);
    close(sortgrep[1]);
    
    // 用dup2直接重定向stdout，比先close再dup更可靠
    dup2(lssort[1], STDOUT_FILENO);
    close(lssort[1]);
    
    char * argv[] = { "ls", "-lR", parameters.dirIsSet ? dir : NULL, NULL };
    execvp("ls", argv);
    // exec失败后必须处理，避免子进程继续执行父进程逻辑
    perror("execvp ls failed");
    exit(EXIT_FAILURE);
}

// 第二个子进程：sort
if (fork() == 0) {
    // 关闭所有未用到的管道端
    close(lssort[1]);
    close(sortgrep[0]);
    
    // 重定向stdin和stdout
    dup2(lssort[0], STDIN_FILENO);
    close(lssort[0]);
    dup2(sortgrep[1], STDOUT_FILENO);
    close(sortgrep[1]);
    
    char * argv[] = { "sort", NULL };
    execvp("sort", argv);
    perror("execvp sort failed");
    exit(EXIT_FAILURE);
}

// 第三个子进程：grep
if (fork() == 0) {
    // 关闭所有未用到的管道端
    close(lssort[0]);
    close(lssort[1]);
    close(sortgrep[1]);
    
    // 重定向stdin到sortgrep读端
    dup2(sortgrep[0], STDIN_FILENO);
    close(sortgrep[0]);
    
    // 处理输出文件，增加O_TRUNC和正确权限
    if (parameters.outfileIsSet) {
        int fd = open(outfile, O_CREAT | O_RDWR | O_TRUNC, 0644);
        if (fd == -1) {
            perror("Error: Could not open file");
            exit(EXIT_FAILURE);
        }
        dup2(fd, STDOUT_FILENO);
        close(fd);
    }
    
    char * argv[] = { "grep", string, NULL };
    execvp("grep", argv);
    perror("execvp grep failed");
    exit(EXIT_FAILURE);
}

// 父进程关闭所有管道端
close(lssort[0]);
close(lssort[1]);
close(sortgrep[0]);
close(sortgrep[1]);

// 循环等待所有子进程结束，避免僵尸进程
int status;
pid_t pid;
while ((pid = wait(&status)) != -1) {
    // 可选：打印子进程退出状态，方便调试
    printf("Child process %d exited with status %d\n", pid, WEXITSTATUS(status));
}

return 0;

关键修改说明 #

• 用dup2替代dup：dup2可以直接指定要重定向的目标文件描述符，比先close再dup更可靠，能避免中间出现意外的fd占用问题。
• 关闭所有未使用的管道fd：每个子进程都关闭了自己不需要的管道两端，确保管道的读/写端能在所有生产者结束后正确关闭，下游进程能收到EOF并正常退出。
• 完整的子进程等待逻辑：用while循环调用wait，直到所有子进程都退出，彻底避免僵尸进程残留。
• 修复文件打开参数：增加O_TRUNC（覆盖已有文件内容，和shell的>行为一致）以及正确的文件权限0644，符合UNIX文件权限规范。

这样修改后，你的程序应该就能像预期那样正常执行、筛选结果并自动退出了。

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