diff options
author | arton <arton@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-03-22 15:05:39 +0000 |
---|---|---|
committer | arton <arton@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-03-22 15:05:39 +0000 |
commit | 50ff6ed8662699d28403bddd2edf0fc7db0a9b90 (patch) | |
tree | c1ca7e0dc6a7cc0eb416759a398c892b342a4fe3 /win32 | |
parent | f06ec31c54992da9eb1a69e34e69ec846aa25d2b (diff) | |
download | ruby-50ff6ed8662699d28403bddd2edf0fc7db0a9b90.tar.gz |
* win32/win32.c: wait process real termination after reading
exit code. fixes #4518
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@31152 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'win32')
-rw-r--r-- | win32/win32.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/win32/win32.c b/win32/win32.c index 1805fe1015..6c9da16c14 100644 --- a/win32/win32.c +++ b/win32/win32.c @@ -3549,6 +3549,7 @@ poll_child_status(struct ChildRecord *child, int *stat_loc) if (!GetExitCodeProcess(child->hProcess, &exitcode)) { /* If an error occured, return immediatly. */ + error_exit: err = GetLastError(); if (err == ERROR_INVALID_PARAMETER) errno = ECHILD; @@ -3562,7 +3563,12 @@ poll_child_status(struct ChildRecord *child, int *stat_loc) return -1; } if (exitcode != STILL_ACTIVE) { - /* If already died, return immediatly. */ + rb_pid_t pid; + /* If already died, wait process's real termination. */ + if (rb_w32_wait_events_blocking(&child->hProcess, 1, INFINITE) != WAIT_OBJECT_0) { + goto error_exit; + } + pid = child->pid; rb_pid_t pid = child->pid; CloseChildHandle(child); if (stat_loc) *stat_loc = exitcode << 8; |