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

Commit ae68cad7 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

lxc-attach: make sure exit status of command is returned
576b950f · Antonio Terceiro · faec88e5 · 576b950f
Commit 576b950f authored Aug 18, 2019 by Antonio Terceiro
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 3 deletions

lxc_attach.c src/lxc/tools/lxc_attach.c +2 -3

No files found.
--- 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);