From 39b5ca92bccba916cd9b418c9f61984fbe6831e7 Mon Sep 17 00:00:00 2001 From: Antonio Terceiro Date: Sun, 18 Aug 2019 17:30:32 -0300 Subject: [PATCH] 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 Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=934983 --- src/lxc/tools/lxc_attach.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/lxc/tools/lxc_attach.c b/src/lxc/tools/lxc_attach.c index e2a5b307a..a9334b936 100644 --- a/src/lxc/tools/lxc_attach.c +++ b/src/lxc/tools/lxc_attach.c @@ -367,10 +367,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); -- 2.47.3