C++中pipe与execlp调用子进程，第二次无输出问题修复请求

问题描述 #

尝试使用pipe()和execlp()从子进程获取字符串，编写了getHelp2函数。主进程中两次调用该函数时，第一次运行正常，但第二次无效果。已尝试恢复stdin、stdout、stderr，使用fcntl及cin.clear()仍无效，怀疑stdout未正确恢复。

原代码：

vector<string> getHelp2(string program_name){
    string command = "man " + program_name;
    int stdin_copy = dup(STDIN_FILENO);
    int stdout_copy = dup(STDOUT_FILENO);
    int stderr_copy = dup(STDERR_FILENO);
    int pipefd[2] = {0, };
    pipe(pipefd);
    pid_t pid = fork();
    if(pid == 0){
        close(pipefd[0]);
        dup2(pipefd[1], STDOUT_FILENO);
        dup2(pipefd[1], STDERR_FILENO);
        fcntl(pipefd[1], F_SETFD, FD_CLOEXEC);
        execlp("man", "man", program_name.data() ,NULL);
        
    }
    else{
        dup2(pipefd[0], STDIN_FILENO);
        waitpid(pid, 0, 0);
        close(pipefd[1]);
        string str;
        string substr;
        getline(cin, substr);
        getline(cin, substr);
        while(getline(cin, substr)){
            str+=substr+"\n";
        }
        dup2(stdin_copy, STDIN_FILENO);
        dup2(stdout_copy, STDOUT_FILENO);
        dup2(stderr_copy, STDERR_FILENO);
        close(pipefd[0]);
        cin.clear();

修复方案及代码 #

问题根源在于文件描述符泄漏和子进程异常处理缺失，以下是修复后的完整代码：

#include <vector>
#include <string>
#include <unistd.h>
#include <sys/wait.h>
#include <fcntl.h>
#include <iostream>
#include <sstream>
#include <cstdio>

using namespace std;

vector<string> getHelp2(string program_name) {
    vector<string> result;
    int pipefd[2];

    // 创建管道失败直接返回空结果
    if (pipe(pipefd) == -1) {
        perror("pipe failed");
        return result;
    }

    pid_t pid = fork();
    if (pid == -1) {
        perror("fork failed");
        close(pipefd[0]);
        close(pipefd[1]);
        return result;
    }

    if (pid == 0) {
        // 子进程：重定向输出到管道写端
        close(pipefd[0]); // 关闭不需要的管道读端
        dup2(pipefd[1], STDOUT_FILENO);
        dup2(pipefd[1], STDERR_FILENO);
        close(pipefd[1]); // 重定向完成后关闭原管道写端，避免泄漏

        // 执行man命令，失败则立即退出子进程
        execlp("man", "man", program_name.c_str(), NULL);
        perror("execlp failed");
        _exit(EXIT_FAILURE); // 必须退出，防止执行父进程代码逻辑
    } else {
        // 父进程：从管道读端读取子进程输出
        close(pipefd[1]); // 关闭不需要的管道写端

        // 直接从管道文件描述符读取，规避cin缓冲异常问题
        char buffer[4096];
        ssize_t bytes_read;
        stringstream ss;
        while ((bytes_read = read(pipefd[0], buffer, sizeof(buffer))) > 0) {
            ss.write(buffer, bytes_read);
        }
        close(pipefd[0]);

        // 等待子进程执行完毕
        int status;
        waitpid(pid, &status, 0);

        // 将读取到的内容按行分割存入vector
        string line;
        while (getline(ss, line)) {
            result.push_back(line);
        }
    }

    return result;
}

关键修复点说明 #

1. 移除不必要的标准流复制操作：
   原代码复制stdin/stdout/stderr后未关闭副本，导致文件描述符泄漏。修复后直接从管道读端读取数据，无需修改父进程标准流，彻底避免了标准流恢复的问题。

2. 严格管理管道文件描述符：

   • 子进程重定向完成后关闭原管道写端，避免文件描述符泄漏；
   • 父进程立即关闭管道写端，确保子进程退出时管道读端能正常收到EOF信号。

3. 补充子进程异常退出逻辑：
   若execlp执行失败（如指定程序不存在），子进程必须调用_exit退出，防止其继续执行父进程的代码逻辑，引发不可预期的错误。

4. 改用底层文件读取替代cin：
   直接使用read系统调用读取管道数据，规避了cin缓冲机制带来的状态异常问题，同时读取效率更高、逻辑更可靠。

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