lxc.init: correctly exit with the app's error code

Based on the comments in the code (and the have_status flag), the intent
here (and IMO, the desired behavior) should be for init.lxc to propagate
the actual exit code from the real application process up through.
Otherwise, it is swallowed and nobody can access it.

The bug being fixed here is that ret held the correct exit code, but when
it went around the loop again (to wait for other children) ret is
clobbered. Let's save the desired exit status somewhere else, so it can't
get clobbered, and we propagate things correctly.

Signed-off-by: Tycho Andersen <tycho@tycho.ws>

diff --git a/src/lxc/lxc_init.c b/src/lxc/lxc_init.c
index 2879a9399fc611f48830977a6e59ec5276dc990d..78811de4b560425b37fafee796401b669f22ddeb 100644 (file)
--- a/src/lxc/lxc_init.c
+++ b/src/lxc/lxc_init.c
@@ -202,7 +202,7 @@ int main(int argc, char *argv[])
        struct sigaction act;
        struct lxc_log log;
        sigset_t mask, omask;
-       int have_status = 0, shutdown = 0;
+       int have_status = 0, exit_with = 1, shutdown = 0;
 
        if (arguments_parse(&my_args, argc, argv))
                exit(EXIT_FAILURE);
@@ -420,14 +420,14 @@ int main(int argc, char *argv[])
                 * pid) and continue to wait for the end of the orphan group.
                 */
                if (waited_pid == pid && !have_status) {
-                       ret = lxc_error_set_and_log(waited_pid, status);
+                       exit_with = lxc_error_set_and_log(waited_pid, status);
                        have_status = 1;
                }
        }
 out:
        if (ret < 0)
                exit(EXIT_FAILURE);
-       exit(ret);
+       exit(exit_with);
 }
 
 static void print_usage(const struct option longopts[])