From 68f5ff0ec4ba02712e806df963ba65838522e2bb Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Wed, 26 Jul 2023 10:18:14 +0200
Subject: process: portable exit code processing

---
 src/process.c | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

(limited to 'src/process.c')

diff --git a/src/process.c b/src/process.c
index 3fc866f..1e2583b 100644
--- a/src/process.c
+++ b/src/process.c
@@ -40,12 +40,20 @@ static int wait_for_child(pid_t pid, int *ec)
 		return ret;
 	}
 
-	if (WIFEXITED(status))
+	/* The child process reports the lowest 8 bits of its exit code, which
+	 * are treated as an unsigned integer on Linux.
+	 *
+	 * If it was killed by a signal, indicate that by negating the signal
+	 * number. */
+
+	if (WIFEXITED(status)) {
 		*ec = WEXITSTATUS(status);
-	else if (WIFSIGNALED(status))
-		*ec = status; /* Same as $?. */
-	else
-		*ec = -1;
+	} else if (WIFSIGNALED(status)) {
+		*ec = -WTERMSIG(status);
+	} else {
+		log_err("This shouldn't happen: %d\n", status);
+		*ec = 1;
+	}
 
 	return 0;
 }
-- 
cgit v1.2.3