scrotExecApp: check for WIFEXITED first

according to waitpid(3) manpage: | This macro (WEXITSTATUS) should be employed only if WIFEXITED returned true.
resurrecting-open-source-projects · May 12, 2023 · fafced4 · fafced4
1 parent 9c9ba63
commit fafced4
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/scrot.c b/src/scrot.c
@@ -575,7 +575,7 @@ static void scrotExecApp(Imlib_Image image, struct tm *tm, char *filenameIM,
 
     if (ret == -1)
         warn("The child process could not be created");
-    else if (WEXITSTATUS(ret) == 127)
+    else if (WIFEXITED(ret) && WEXITSTATUS(ret) == 127)
         warnx("scrot could not execute the command: %s", execStr);
     free(execStr);
 }