@@ -783,7 +773,10 @@ virExec(virCommandPtr cmd) if (virCommandHandshakeChild(cmd) < 0) goto fork_error;
- if (sigaction(SIGPIPE, &waxon, NULL) < 0) { + memset(&sig_action, 0, sizeof(sig_action)); + sig_action.sa_handler = SIG_DFL; + sigemptyset(&sig_action.sa_mask); + if (sigaction(SIGPIPE, &sig_action, NULL) < 0) { virReportSystemError(errno, "%s", _("Could not re-enable SIGPIPE")); goto fork_error;
We can just delete this sigaction() call entirely. The execve() call will purge the no-op handler function we registered in virFork(), resetting it back to SIG_DFL, so no need todo it manually here.
ok, i'll adjust the code and push another patch. --- Best wishes Wang Yechao