waitpid / wexitstatus返回0而不是正确的返回码

Question

我有下面的帮助函数，用于执行命令并在posix系统上获取返回值。 我曾经使用过popen ，但是如果它运行并且在popen / pclose有机会完成其工作之前退出，则无法获得具有popen的应用程序的返回代码。

以下辅助函数创建进程fork，使用execvp运行所需的外部进程，然后父进程使用waitpid获取返回代码。 我看到奇怪的情况，它拒绝运行。

当使用wait = true调用时，无论如何， waitpid都应该返回应用程序的退出代码。 但是，我看到stdout输出指定返回代码应该是非零，但返回代码为零。 在常规shell中测试外部进程，然后echo $? 返回非零，因此外部进程没有返回正确的代码也不是问题。 如果有任何帮助，正在运行的外部进程是mount(8) （是的，我知道我可以使用mount(2)但是除了这一点之外）。

我提前为代码转储道歉。 大多数是调试/日志记录：

inline int ForkAndRun(const std::string &command, const std::vector<std::string> &args, bool wait = false, std::string *output = NULL)
{
    std::string debug;

    std::vector<char*> argv;
    for(size_t i = 0; i < args.size(); ++i)
    {
        argv.push_back(const_cast<char*>(args[i].c_str()));
        debug += "\"";
        debug += args[i];
        debug += "\" ";
    }
    argv.push_back((char*)NULL);

    neosmart::logger.Debug("Executing %s", debug.c_str());

    int pipefd[2];

    if (pipe(pipefd) != 0)
    {
        neosmart::logger.Error("Failed to create pipe descriptor when trying to launch %s", debug.c_str());
        return EXIT_FAILURE;
    }

    pid_t pid = fork();

    if (pid == 0)
    {
        close(pipefd[STDIN_FILENO]); //child isn't going to be reading
        dup2(pipefd[STDOUT_FILENO], STDOUT_FILENO);
        close(pipefd[STDOUT_FILENO]); //now that it's been dup2'd
        dup2(pipefd[STDOUT_FILENO], STDERR_FILENO);

        if (execvp(command.c_str(), &argv[0]) != 0)
        {
            exit(EXIT_FAILURE);
        }
        return 0;
    }
    else if (pid < 0)
    {
        neosmart::logger.Error("Failed to fork when trying to launch %s", debug.c_str());
        return EXIT_FAILURE;
    }
    else
    {
        close(pipefd[STDOUT_FILENO]);

        int exitCode = 0;

        if (wait)
        {
            waitpid(pid, &exitCode, wait ? __WALL : (WNOHANG | WUNTRACED));

            std::string result;
            char buffer[128];
            ssize_t bytesRead;
            while ((bytesRead = read(pipefd[STDIN_FILENO], buffer, sizeof(buffer)-1)) != 0)
            {
                buffer[bytesRead] = '\0';
                result += buffer;
            }

            if (wait)
            {
                if ((WIFEXITED(exitCode)) == 0)
                {
                    neosmart::logger.Error("Failed to run command %s", debug.c_str());
                    neosmart::logger.Info("Output:\n%s", result.c_str());
                }
                else
                {
                    neosmart::logger.Debug("Output:\n%s", result.c_str());
                    exitCode = WEXITSTATUS(exitCode);
                    if (exitCode != 0)
                    {
                        neosmart::logger.Info("Return code %d", (exitCode));
                    }
                }
            }

            if (output)
            {
                result.swap(*output);
            }
        }

        close(pipefd[STDIN_FILENO]);

        return exitCode;
    }
}

请注意，使用正确的参数运行命令，函数继续运行没有任何问题， WIFEXITED返回TRUE 。 但是， WEXITSTATUS返回0，应该返回其他内容。

Answer 1

可能不是你的主要问题，但我认为我看到一个小问题。 在你的孩子过程中，你有......

dup2(pipefd[STDOUT_FILENO], STDOUT_FILENO);
close(pipefd[STDOUT_FILENO]); //now that it's been dup2'd
dup2(pipefd[STDOUT_FILENO], STDERR_FILENO); //but wait, this pipe is closed!

但我认为你想要的是：

dup2(pipefd[STDOUT_FILENO], STDOUT_FILENO);
dup2(pipefd[STDOUT_FILENO], STDERR_FILENO);
close(pipefd[STDOUT_FILENO]); //now that it's been dup2'd for both, can close

我对Linux中的分支和管道没有多少经验，但我最近写了一个类似的功能。 如果您愿意，可以查看要比较的代码。 我知道我的功能有效。

execAndRedirect.cpp

Answer 2

我使用的猫鼬图书馆和grepping我的代码SIGCHLD透露，使用mg_start在制定从猫鼬结果SIGCHLD到SIG_IGN 。

在waitpid手册页中 ，在Linux 上 ，设置为SIG_IGN的SIGCHLD不会创建僵尸进程，因此如果进程已成功运行并退出，则waitpid将失败 - 但如果尚未运行则将运行OK。 这是我的代码零星失败的原因。

在将mg_start调用为无效函数后，只需重新设置SIGCHLD完成任何操作，足以防止僵尸记录被立即删除。

根据@ Geoff_Montee的建议 ，我的STDERR重定向中存在一个错误，但这不是问题的原因，因为execvp不会将返回值存储在STDERR甚至STDOUT ，而是存储在与父进程关联的内核对象中（僵尸记录）。

@jilles关于C ++中vector非连续性的警告不适用于C ++ 03及更高版本（仅对C ++ 98有效，但在实践中，大多数C ++ 98编译器确实使用了连续存储），并且与此问题无关。 但是，在阻塞和检查waitpid的输出之前从管道读取的建议是正确的。

Answer 3

我发现pclose不会阻塞并等待进程结束，这与文档相反（这是在CentOS 6上）。 我发现我需要调用pclose然后调用waitpid(pid,&status,0); 获得真正的回报价值。