Improve handling of unexpected 'status' in process_status_msg().
authorBen Pfaff <blp@nicira.com>
Wed, 21 Jan 2009 00:28:38 +0000 (16:28 -0800)
committerBen Pfaff <blp@nicira.com>
Wed, 21 Jan 2009 00:44:14 +0000 (16:44 -0800)
This function was getting passed -1 as 'status' due to a bug elsewhere,
and it was outputting ", core dumped" as the result, which clearly isn't
very helpful.  This improves the situation.

lib/process.c

index c3e3de5ce5c7b9fb200adb9442a870e45bcacdae..a06f5dc13e82e0b401d70c9559b1a73520ce3187 100644 (file)
@@ -313,15 +313,19 @@ process_status_msg(int status)
     struct ds ds = DS_EMPTY_INITIALIZER;
     if (WIFEXITED(status)) {
         ds_put_format(&ds, "exit status %d", WEXITSTATUS(status));
-    } else if (WIFSIGNALED(status)) {
+    } else if (WIFSIGNALED(status) || WIFSTOPPED(status)) {
+        int signr = WIFSIGNALED(status) ? WTERMSIG(status) : WSTOPSIG(status);
         const char *name = NULL;
 #ifdef HAVE_STRSIGNAL
-        name = strsignal(WTERMSIG(status));
+        name = strsignal(signr);
 #endif
-        ds_put_format(&ds, "killed by signal %d", WTERMSIG(status));
+        ds_put_format(&ds, "%s by signal %d",
+                      WIFSIGNALED(status) ? "killed" : "stopped", signr);
         if (name) {
             ds_put_format(&ds, " (%s)", name);
         }
+    } else {
+        ds_put_format(&ds, "terminated abnormally (%x)", status);
     }
     if (WCOREDUMP(status)) {
         ds_put_cstr(&ds, ", core dumped");