tools/libxl: Initialise rc on error paths of libxl_domain_remus_start()
authorAndrew Cooper <andrew.cooper3@citrix.com>
Wed, 1 Oct 2014 10:09:17 +0000 (11:09 +0100)
committerIan Campbell <ian.campbell@citrix.com>
Wed, 1 Oct 2014 11:08:52 +0000 (12:08 +0100)
Coverity-ID: 1242320
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
CC: Ian Campbell <Ian.Campbell@citrix.com>
CC: Ian Jackson <Ian.Jackson@eu.citrix.com>
CC: Wei Liu <wei.liu2@citrix.com>
CC: Shriram Rajagopalan <rshriram@cs.ubc.ca>
CC: Yang Hongyang <yanghy@cn.fujitsu.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
tools/libxl/libxl.c

index 9f629c48af98b0de943a1cac2b84ea27a306c44f..3f0a7ffba21ce171d2bd4c08df6ed3f5fca2bee5 100644 (file)
@@ -816,6 +816,7 @@ int libxl_domain_remus_start(libxl_ctx *ctx, libxl_domain_remus_info *info,
          !libxl_defbool_val(info->diskbuf))) {
         LOG(ERROR, "Unsafe mode must be enabled to replicate to /dev/null,"
                    "disable network buffering and disk replication");
+        rc = ERROR_FAIL;
         goto out;
     }
 
@@ -839,6 +840,7 @@ int libxl_domain_remus_start(libxl_ctx *ctx, libxl_domain_remus_info *info,
     if (libxl_defbool_val(info->netbuf)) {
         if (!libxl__netbuffer_enabled(gc)) {
             LOG(ERROR, "Remus: No support for network buffering");
+            rc = ERROR_FAIL;
             goto out;
         }
         rds->device_kind_flags |= (1 << LIBXL__DEVICE_KIND_VIF);