diff options
author | DongHun Kwak <dh0128.kwak@samsung.com> | 2019-12-05 15:12:59 +0900 |
---|---|---|
committer | DongHun Kwak <dh0128.kwak@samsung.com> | 2019-12-05 15:12:59 +0900 |
commit | b8cf34c691623e4ec329053cbbf68522a855882d (patch) | |
tree | 34da08632a99677f6b79ecb65e5b655a5b69a67f /boost/process/detail/posix/is_running.hpp | |
parent | 3fdc3e5ee96dca5b11d1694975a65200787eab86 (diff) | |
download | boost-b8cf34c691623e4ec329053cbbf68522a855882d.tar.gz boost-b8cf34c691623e4ec329053cbbf68522a855882d.tar.bz2 boost-b8cf34c691623e4ec329053cbbf68522a855882d.zip |
Imported Upstream version 1.67.0upstream/1.67.0
Diffstat (limited to 'boost/process/detail/posix/is_running.hpp')
-rw-r--r-- | boost/process/detail/posix/is_running.hpp | 57 |
1 files changed, 28 insertions, 29 deletions
diff --git a/boost/process/detail/posix/is_running.hpp b/boost/process/detail/posix/is_running.hpp index e8a009dd22..0d431a9126 100644 --- a/boost/process/detail/posix/is_running.hpp +++ b/boost/process/detail/posix/is_running.hpp @@ -13,37 +13,22 @@ namespace boost { namespace process { namespace detail { namespace posix { - +// Use the "stopped" state (WIFSTOPPED) to indicate "not terminated". +// This bit arrangement of status codes is not guaranteed by POSIX, but (according to comments in +// the glibc <bits/waitstatus.h> header) is the same across systems in practice. constexpr int still_active = 0x7F; -static_assert(!WIFEXITED(still_active), "Internal Error"); +static_assert(!WIFEXITED(still_active) && !WIFSIGNALED(still_active), "Internal Error"); -inline bool is_running(const child_handle &p, int & exit_code) +inline bool is_running(int code) { - int status; - auto ret = ::waitpid(p.pid, &status, WNOHANG|WUNTRACED); - - if (ret == -1) - { - if (errno != ECHILD) //because it no child is running, than this one isn't either, obviously. - ::boost::process::detail::throw_last_error("is_running error"); - - return false; - } - else if (ret == 0) - return true; - else //exited - { - if (WIFEXITED(status)) - exit_code = status; - return false; - } + return !WIFEXITED(code) && !WIFSIGNALED(code); } inline bool is_running(const child_handle &p, int & exit_code, std::error_code &ec) noexcept { int status; - auto ret = ::waitpid(p.pid, &status, WNOHANG|WUNTRACED); - + auto ret = ::waitpid(p.pid, &status, WNOHANG); + if (ret == -1) { if (errno != ECHILD) //because it no child is running, than this one isn't either, obviously. @@ -55,22 +40,36 @@ inline bool is_running(const child_handle &p, int & exit_code, std::error_code & else { ec.clear(); - - if (WIFEXITED(status)) + + if (!is_running(status)) exit_code = status; - + return false; } } -inline bool is_running(int code) +inline bool is_running(const child_handle &p, int & exit_code) { - return !WIFEXITED(code); + std::error_code ec; + bool b = is_running(p, exit_code, ec); + boost::process::detail::throw_error(ec, "waitpid(2) failed in is_running"); + return b; } inline int eval_exit_status(int code) { - return WEXITSTATUS(code); + if (WIFEXITED(code)) + { + return WEXITSTATUS(code); + } + else if (WIFSIGNALED(code)) + { + return WTERMSIG(code); + } + else + { + return code; + } } }}}} |