From 64a8a8ab08e938fb7158d4f2ab7ca370ba512bfb Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Sun, 3 May 2015 13:57:16 +0200 Subject: dispatcher: repeat waitpid() call on EINTR Also, no use of first trying to kill() with signal zero. Just send SIGKILL right away. --- callouts/nm-dispatcher.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/callouts/nm-dispatcher.c b/callouts/nm-dispatcher.c index 6fb40cf9f3..77f2153d62 100644 --- a/callouts/nm-dispatcher.c +++ b/callouts/nm-dispatcher.c @@ -294,9 +294,12 @@ script_timeout_cb (gpointer user_data) g_warning ("Script '%s' took too long; killing it.", script->script); - if (kill (script->pid, 0) == 0) - kill (script->pid, SIGKILL); - (void) waitpid (script->pid, NULL, 0); + kill (script->pid, SIGKILL); +again: + if (waitpid (script->pid, NULL, 0) == -1) { + if (errno == EINTR) + goto again; + } script->error = g_strdup_printf ("Script '%s' timed out.", script->script); script->result = DISPATCH_RESULT_TIMEOUT; -- cgit v1.2.1