lxc-attach: make sure exit status of command is returned

Commit ae68cad763d5b39a6a9e51de2acd1ad128b720ca introduced a regression that
makes lxc-attach ignore the exit status of the executed command. This was first
identified in 3.0.4 LTS, while it worked on 3.0.3.

  # lxc-attach --version
  3.0.4
  # lxc-attach -n test false; echo $?
  0

Signed-off-by: Antonio Terceiro <terceiro@debian.org>
Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=934983

diff --git a/src/lxc/tools/lxc_attach.c b/src/lxc/tools/lxc_attach.c
index 917cca2821dc82f3f4784af64e2dc04a743b0dba..c3e1e6182a9c1c4ec37908010da1983d05deb48b 100644 (file)
--- a/src/lxc/tools/lxc_attach.c
+++ b/src/lxc/tools/lxc_attach.c
@@ -385,10 +385,9 @@ int main(int argc, char *argv[])
                ret = lxc_wait_for_pid_status(pid);
                if (ret < 0)
                        goto out;
-
-               if (WIFEXITED(ret))
-                       wexit = WEXITSTATUS(ret);
        }
+       if (WIFEXITED(ret))
+               wexit = WEXITSTATUS(ret);
 
 out:
        lxc_container_put(c);