[libvirt] [PATCH] lxc: errors after the handshake won't be reported

Cédric Bosdonnat cbosdonnat at suse.com
Tue Jul 19 14:25:34 UTC 2016


Any error happening after the hand shake in the lxc controller
will not result in a failure as errors are checked during the handshake.
Move the handshake after the last possible error.
---
 src/lxc/lxc_controller.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/lxc/lxc_controller.c b/src/lxc/lxc_controller.c
index f55aadc..825b4d4 100644
--- a/src/lxc/lxc_controller.c
+++ b/src/lxc/lxc_controller.c
@@ -2444,13 +2444,13 @@ virLXCControllerRun(virLXCControllerPtr ctrl)
     if (lxcControllerClearCapabilities() < 0)
         goto cleanup;
 
-    if (virLXCControllerDaemonHandshake(ctrl) < 0)
-        goto cleanup;
-
     for (i = 0; i < ctrl->nconsoles; i++)
         if (virLXCControllerConsoleSetNonblocking(&(ctrl->consoles[i])) < 0)
             goto cleanup;
 
+    if (virLXCControllerDaemonHandshake(ctrl) < 0)
+        goto cleanup;
+
     /* We must not hold open a dbus connection for life
      * of LXC instance, since dbus-daemon is limited to
      * only a few 100 connections by default
-- 
2.6.6




More information about the libvir-list mailing list