Use WIFEXITED and WEXITSTATUS macros. If there are systems out there

that want to run sudo that still don't support these we can try to
deal with that later.

1 files changed, 3 insertions, 2 deletions

diff --git a/visudo.c b/visudo.c
index cf3dfdba6..0c657d4be 100644
--- a/visudo.c
+++ b/visudo.c
@@ -638,8 +638,9 @@ run_command(path, argv)
 
     (void) sigprocmask(SIG_SETMASK, &oset, NULL);
 
-    /* XXX - should use WEXITSTATUS() */
-    return(pid == -1 ? -1 : (status >> 8));
+    if (pid == -1 || !WIFEXITED(status))
+	return(-1);
+    return(WEXITSTATUS(status));
 }
 
 static int