fix: Improve portability of compiler exit status check

WEXITSTATUS(status) should only be used when WIFEXITED(status) is true.
The previous code works on Linux "by accident".

diff --git a/src/ccache/execute.cpp b/src/ccache/execute.cpp
index c75cbc61e8232b6d3db3ae3087482f016f48a719..34ab6545d0b4889a4250934645244da4c7300593 100644 (file)
--- a/src/ccache/execute.cpp
+++ b/src/ccache/execute.cpp
@@ -354,11 +354,11 @@ execute(Context& ctx,
     ctx.compiler_pid = 0;
   }
 
-  if (WEXITSTATUS(status) == 0 && WIFSIGNALED(status)) {
+  if (WIFSIGNALED(status)) {
     return -1;
   }
 
-  return WEXITSTATUS(status);
+  return WIFEXITED(status) ? WEXITSTATUS(status) : -1;
 }
 
 void