FUJITA Tomonori wrote:
The temporary solution is fine by me.
okay
+ do {
[...]
+ ret_sel = select(fds[0]+1, &rfds, NULL, NULL, &tv); + } while (ret_sel < 0 && errno == EINTR); + if (ret_sel <= 0) { /* error or timeout */ + eprintf("timeout on redirect callback, \ + terminating child pid %d\n", pid); + kill(pid, SIGTERM); + }
Why this is necessary before waitpid()? I thought that we need to make sure that fds[0] is readable (with a timeout) before calling read() for fds[0].
if the timeout expires and fds[0] isn't readable, we don't want to wait to the child process anymore, so we must terminate it - else we could hang on waitpid forever, something we wanted to avoid in the first place.
Or. Or. -- To unsubscribe from this list: send the line "unsubscribe stgt" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
