JasonWoof Got questions, comments, patches, etc.? Contact Jason Woofenden
Add error message when child exits whit error
authorRoberto E. Vargas Caballero <k0ga@shike2.com>
Mon, 4 Aug 2014 20:07:04 +0000 (22:07 +0200)
committerRoberto E. Vargas Caballero <k0ga@shike2.com>
Mon, 4 Aug 2014 20:07:04 +0000 (22:07 +0200)
Master proccess was not showing any error message when the child
died with an error, and it was very confusing for the user (for
example with incorrect -e command).

st.c

diff --git a/st.c b/st.c
index a2703f4..45bc89d 100644 (file)
--- a/st.c
+++ b/st.c
@@ -1176,16 +1176,15 @@ execsh(void) {
 
 void
 sigchld(int a) {
 
 void
 sigchld(int a) {
-       int stat = 0;
+       int stat, ret;
 
        if(waitpid(pid, &stat, 0) < 0)
                die("Waiting for pid %hd failed: %s\n", pid, strerror(errno));
 
 
        if(waitpid(pid, &stat, 0) < 0)
                die("Waiting for pid %hd failed: %s\n", pid, strerror(errno));
 
-       if(WIFEXITED(stat)) {
-               exit(WEXITSTATUS(stat));
-       } else {
-               exit(EXIT_FAILURE);
-       }
+       ret = WIFEXITED(stat) ? WEXITSTATUS(stat) : EXIT_FAILURE;
+       if (ret != EXIT_SUCCESS)
+               die("child finished with error '%d'\n", stat);
+       exit(EXIT_SUCCESS);
 }
 
 void
 }
 
 void