[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

[libvirt] [PATCH] Remove curly braces on all single-line conditional jumps in dispatcher



Replace all occurrances of

   if (....) {
      goto cleanup;
   }

With

   if (.....)
      goto cleanup;

to save one line of code

* daemon/remote.c: Remove curly braces on single line conditionals
---
 daemon/remote.c |  951 ++++++++++++++++++------------------------------------
 1 files changed, 317 insertions(+), 634 deletions(-)

diff --git a/daemon/remote.c b/daemon/remote.c
index 325ba90..247a226 100644
--- a/daemon/remote.c
+++ b/daemon/remote.c
@@ -427,9 +427,8 @@ remoteDispatchOpen(struct qemud_server *server,
         ? virConnectOpenReadOnly(name)
         : virConnectOpen(name);
 
-    if (client->conn == NULL) {
+    if (client->conn == NULL)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -476,9 +475,8 @@ remoteDispatchSupportsFeature(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     ret->supported = virDrvSupportsFeature(conn, args->feature);
 
-    if (ret->supported < 0) {
+    if (ret->supported < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -505,9 +503,8 @@ remoteDispatchGetType(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     type = virConnectGetType(conn);
-    if (type == NULL) {
+    if (type == NULL)
         goto cleanup;
-    }
 
     /* We have to strdup because remoteDispatchClientRequest will
      * free this string after it's been serialised.
@@ -543,9 +540,8 @@ remoteDispatchGetVersion(struct qemud_server *server ATTRIBUTE_UNUSED,
         goto cleanup;
     }
 
-    if (virConnectGetVersion(conn, &hvVer) < 0) {
+    if (virConnectGetVersion(conn, &hvVer) < 0)
         goto cleanup;
-    }
 
     ret->hv_ver = hvVer;
     rv = 0;
@@ -573,9 +569,8 @@ remoteDispatchGetLibVersion(struct qemud_server *server ATTRIBUTE_UNUSED,
         goto cleanup;
     }
 
-    if (virConnectGetLibVersion(conn, &libVer) < 0) {
+    if (virConnectGetLibVersion(conn, &libVer) < 0)
         goto cleanup;
-    }
 
     ret->lib_ver = libVer;
     rv = 0;
@@ -604,9 +599,8 @@ remoteDispatchGetHostname(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     hostname = virConnectGetHostname(conn);
-    if (hostname == NULL) {
+    if (hostname == NULL)
         goto cleanup;
-    }
 
     ret->hostname = hostname;
     rv = 0;
@@ -635,9 +629,8 @@ remoteDispatchGetUri(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     uri = virConnectGetURI(conn);
-    if (uri == NULL) {
+    if (uri == NULL)
         goto cleanup;
-    }
 
     ret->uri = uri;
     rv = 0;
@@ -668,9 +661,8 @@ remoteDispatchGetSysinfo(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     flags = args->flags;
     sysinfo = virConnectGetSysinfo(conn, flags);
-    if (sysinfo == NULL) {
+    if (sysinfo == NULL)
         goto cleanup;
-    }
 
     ret->sysinfo = sysinfo;
     rv = 0;
@@ -700,9 +692,8 @@ remoteDispatchGetMaxVcpus(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     type = args->type ? *args->type : NULL;
     ret->max_vcpus = virConnectGetMaxVcpus(conn, type);
-    if (ret->max_vcpus < 0) {
+    if (ret->max_vcpus < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -729,9 +720,8 @@ remoteDispatchNodeGetInfo(struct qemud_server *server ATTRIBUTE_UNUSED,
         goto cleanup;
     }
 
-    if (virNodeGetInfo(conn, &info) < 0) {
+    if (virNodeGetInfo(conn, &info) < 0)
         goto cleanup;
-    }
 
     memcpy(ret->model, info.model, sizeof ret->model);
     ret->memory = info.memory;
@@ -768,9 +758,8 @@ remoteDispatchGetCapabilities(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     caps = virConnectGetCapabilities(conn);
-    if (caps == NULL) {
+    if (caps == NULL)
         goto cleanup;
-    }
 
     ret->capabilities = caps;
     rv = 0;
@@ -814,9 +803,8 @@ remoteDispatchNodeGetCellsFreeMemory(struct qemud_server *server ATTRIBUTE_UNUSE
                                     (unsigned long long *)ret->freeMems.freeMems_val,
                                     args->startCell,
                                     args->maxCells);
-    if (err <= 0) {
+    if (err <= 0)
         goto cleanup;
-    }
     ret->freeMems.freeMems_len = err;
 
     rv = 0;
@@ -848,9 +836,8 @@ remoteDispatchNodeGetFreeMemory(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     freeMem = virNodeGetFreeMemory(conn);
-    if (freeMem == 0) {
+    if (freeMem == 0)
         goto cleanup;
-    }
     ret->freeMem = freeMem;
     rv = 0;
 
@@ -881,14 +868,12 @@ remoteDispatchDomainGetSchedulerType(struct qemud_server *server ATTRIBUTE_UNUSE
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     type = virDomainGetSchedulerType(dom, &nparams);
-    if (type == NULL) {
+    if (type == NULL)
         goto cleanup;
-    }
 
     ret->type = type;
     ret->nparams = nparams;
@@ -933,14 +918,12 @@ remoteDispatchDomainGetSchedulerParameters(struct qemud_server *server ATTRIBUTE
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     r = virDomainGetSchedulerParameters(dom, params, &nparams);
-    if (r < 0) {
+    if (r < 0)
         goto cleanup;
-    }
 
     /* Serialise the scheduler parameters. */
     ret->params.params_len = nparams;
@@ -1046,14 +1029,12 @@ remoteDispatchDomainSetSchedulerParameters(struct qemud_server *server ATTRIBUTE
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     r = virDomainSetSchedulerParameters(dom, params, nparams);
-    if (r < 0) {
+    if (r < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1086,14 +1067,12 @@ remoteDispatchDomainBlockStats(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
     path = args->path;
 
-    if (virDomainBlockStats(dom, path, &stats, sizeof stats) < 0) {
+    if (virDomainBlockStats(dom, path, &stats, sizeof stats) < 0)
         goto cleanup;
-    }
 
     ret->rd_req = stats.rd_req;
     ret->rd_bytes = stats.rd_bytes;
@@ -1131,14 +1110,12 @@ remoteDispatchDomainInterfaceStats(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
     path = args->path;
 
-    if (virDomainInterfaceStats(dom, path, &stats, sizeof stats) < 0) {
+    if (virDomainInterfaceStats(dom, path, &stats, sizeof stats) < 0)
         goto cleanup;
-    }
 
     ret->rx_bytes = stats.rx_bytes;
     ret->rx_packets = stats.rx_packets;
@@ -1185,9 +1162,8 @@ remoteDispatchDomainMemoryStats(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     /* Allocate stats array for making dispatch call */
     if (VIR_ALLOC_N(stats, args->maxStats) < 0) {
@@ -1196,9 +1172,8 @@ remoteDispatchDomainMemoryStats(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     nr_stats = virDomainMemoryStats(dom, stats, args->maxStats, 0);
-    if (nr_stats < 0) {
+    if (nr_stats < 0)
         goto cleanup;
-    }
 
     /* Allocate return buffer */
     if (VIR_ALLOC_N(ret->stats.stats_val, args->maxStats) < 0) {
@@ -1245,9 +1220,8 @@ remoteDispatchDomainBlockPeek(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
     path = args->path;
     offset = args->offset;
     size = args->size;
@@ -1266,9 +1240,8 @@ remoteDispatchDomainBlockPeek(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     if (virDomainBlockPeek(dom, path, offset, size,
-                           ret->buffer.buffer_val, flags) < 0) {
+                           ret->buffer.buffer_val, flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1303,9 +1276,8 @@ remoteDispatchDomainMemoryPeek(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
     offset = args->offset;
     size = args->size;
     flags = args->flags;
@@ -1323,9 +1295,8 @@ remoteDispatchDomainMemoryPeek(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     if (virDomainMemoryPeek(dom, offset, size,
-                            ret->buffer.buffer_val, flags) < 0) {
+                            ret->buffer.buffer_val, flags) < 0)
         goto cleanup;
-    }
     if (dom)
         virDomainFree(dom);
 
@@ -1359,13 +1330,11 @@ remoteDispatchDomainAttachDevice(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainAttachDevice(dom, args->xml) < 0) {
+    if (virDomainAttachDevice(dom, args->xml) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1395,13 +1364,11 @@ remoteDispatchDomainAttachDeviceFlags(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainAttachDeviceFlags(dom, args->xml, args->flags) < 0) {
+    if (virDomainAttachDeviceFlags(dom, args->xml, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1431,13 +1398,11 @@ remoteDispatchDomainUpdateDeviceFlags(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainUpdateDeviceFlags(dom, args->xml, args->flags) < 0) {
+    if (virDomainUpdateDeviceFlags(dom, args->xml, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1467,13 +1432,11 @@ remoteDispatchDomainCreate(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainCreate(dom) < 0) {
+    if (virDomainCreate(dom) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1498,13 +1461,11 @@ remoteDispatchDomainCreateWithFlags(struct qemud_server *server ATTRIBUTE_UNUSED
     virDomainPtr dom = NULL;
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainCreateWithFlags(dom, args->flags) < 0) {
+    if (virDomainCreateWithFlags(dom, args->flags) < 0)
         goto cleanup;
-    }
 
     make_nonnull_domain(&ret->dom, dom);
 
@@ -1536,9 +1497,8 @@ remoteDispatchDomainCreateXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = virDomainCreateXML(conn, args->xml_desc, args->flags);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     make_nonnull_domain(&ret->dom, dom);
 
@@ -1570,9 +1530,8 @@ remoteDispatchDomainDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = virDomainDefineXML(conn, args->xml);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     make_nonnull_domain(&ret->dom, dom);
 
@@ -1604,13 +1563,11 @@ remoteDispatchDomainDestroy(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainDestroy(dom) < 0) {
+    if (virDomainDestroy(dom) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1640,13 +1597,11 @@ remoteDispatchDomainDetachDevice(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainDetachDevice(dom, args->xml) < 0) {
+    if (virDomainDetachDevice(dom, args->xml) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1676,13 +1631,11 @@ remoteDispatchDomainDetachDeviceFlags(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainDetachDeviceFlags(dom, args->xml, args->flags) < 0) {
+    if (virDomainDetachDeviceFlags(dom, args->xml, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1712,15 +1665,13 @@ remoteDispatchDomainDumpXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free this. */
     ret->xml = virDomainGetXMLDesc(dom, args->flags);
-    if (!ret->xml) {
+    if (!ret->xml)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1753,9 +1704,8 @@ remoteDispatchDomainXmlFromNative(struct qemud_server *server ATTRIBUTE_UNUSED,
                                                    args->nativeFormat,
                                                    args->nativeConfig,
                                                    args->flags);
-    if (!ret->domainXml) {
+    if (!ret->domainXml)
         goto cleanup;
-    }
     rv = 0;
 
 cleanup:
@@ -1785,9 +1735,8 @@ remoteDispatchDomainXmlToNative(struct qemud_server *server ATTRIBUTE_UNUSED,
                                                     args->nativeFormat,
                                                     args->domainXml,
                                                     args->flags);
-    if (!ret->nativeConfig) {
+    if (!ret->nativeConfig)
         goto cleanup;
-    }
     rv = 0;
 
 cleanup:
@@ -1815,13 +1764,11 @@ remoteDispatchDomainGetAutostart(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainGetAutostart(dom, &ret->autostart) < 0) {
+    if (virDomainGetAutostart(dom, &ret->autostart) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1852,13 +1799,11 @@ remoteDispatchDomainGetInfo(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainGetInfo(dom, &info) < 0) {
+    if (virDomainGetInfo(dom, &info) < 0)
         goto cleanup;
-    }
 
     ret->state = info.state;
     ret->max_mem = info.maxMem;
@@ -1894,14 +1839,12 @@ remoteDispatchDomainGetMaxMemory(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     ret->memory = virDomainGetMaxMemory(dom);
-    if (ret->memory == 0) {
+    if (ret->memory == 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1931,14 +1874,12 @@ remoteDispatchDomainGetMaxVcpus(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     ret->num = virDomainGetMaxVcpus(dom);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -1969,18 +1910,16 @@ remoteDispatchDomainGetSecurityLabel(struct qemud_server *server ATTRIBUTE_UNUSE
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     if (VIR_ALLOC(seclabel) < 0) {
         virReportOOMError();
         goto cleanup;
     }
 
-    if (virDomainGetSecurityLabel(dom, seclabel) < 0) {
+    if (virDomainGetSecurityLabel(dom, seclabel) < 0)
         goto cleanup;
-    }
 
     ret->label.label_len = strlen(seclabel->label) + 1;
     if (VIR_ALLOC_N(ret->label.label_val, ret->label.label_len) < 0) {
@@ -2019,9 +1958,8 @@ remoteDispatchNodeGetSecurityModel(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     memset(&secmodel, 0, sizeof secmodel);
-    if (virNodeGetSecurityModel(conn, &secmodel) < 0) {
+    if (virNodeGetSecurityModel(conn, &secmodel) < 0)
         goto cleanup;
-    }
 
     ret->model.model_len = strlen(secmodel.model) + 1;
     if (VIR_ALLOC_N(ret->model.model_val, ret->model.model_len) < 0) {
@@ -2063,15 +2001,13 @@ remoteDispatchDomainGetOsType(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free this */
     ret->type = virDomainGetOSType(dom);
-    if (ret->type == NULL) {
+    if (ret->type == NULL)
         goto cleanup;
-    }
     rv = 0;
 
 cleanup:
@@ -2103,9 +2039,8 @@ remoteDispatchDomainGetVcpus(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     if (args->maxinfo > REMOTE_VCPUINFO_MAX) {
         virNetError(VIR_ERR_INTERNAL_ERROR, "%s", _("maxinfo > REMOTE_VCPUINFO_MAX"));
@@ -2127,9 +2062,8 @@ remoteDispatchDomainGetVcpus(struct qemud_server *server ATTRIBUTE_UNUSED,
     info_len = virDomainGetVcpus(dom,
                                  info, args->maxinfo,
                                  cpumaps, args->maplen);
-    if (info_len < 0) {
+    if (info_len < 0)
         goto cleanup;
-    }
 
     /* Allocate the return buffer for info. */
     ret->info.info_len = info_len;
@@ -2187,14 +2121,12 @@ remoteDispatchDomainGetVcpusFlags(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     ret->num = virDomainGetVcpusFlags(dom, args->flags);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2240,9 +2172,8 @@ remoteDispatchDomainMigratePrepare(struct qemud_server *server ATTRIBUTE_UNUSED,
     r = virDomainMigratePrepare(conn, &cookie, &cookielen,
                                 uri_in, uri_out,
                                 args->flags, dname, args->resource);
-    if (r < 0) {
+    if (r < 0)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free cookie, uri_out and
      * the string if there is one.
@@ -2285,9 +2216,8 @@ remoteDispatchDomainMigratePerform(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     dname = args->dname == NULL ? NULL : *args->dname;
 
@@ -2296,9 +2226,8 @@ remoteDispatchDomainMigratePerform(struct qemud_server *server ATTRIBUTE_UNUSED,
                                 args->cookie.cookie_len,
                                 args->uri,
                                 args->flags, dname, args->resource);
-    if (r < 0) {
+    if (r < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2332,9 +2261,8 @@ remoteDispatchDomainMigrateFinish(struct qemud_server *server ATTRIBUTE_UNUSED,
                                   args->cookie.cookie_len,
                                   args->uri,
                                   args->flags);
-    if (ddom == NULL) {
+    if (ddom == NULL)
         goto cleanup;
-    }
 
     make_nonnull_domain(&ret->ddom, ddom);
     rv = 0;
@@ -2382,9 +2310,8 @@ remoteDispatchDomainMigratePrepare2(struct qemud_server *server ATTRIBUTE_UNUSED
                                  uri_in, uri_out,
                                  args->flags, dname, args->resource,
                                  args->dom_xml);
-    if (r < 0) {
+    if (r < 0)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free cookie, uri_out and
      * the string if there is one.
@@ -2424,9 +2351,8 @@ remoteDispatchDomainMigrateFinish2(struct qemud_server *server ATTRIBUTE_UNUSED,
                                    args->uri,
                                    args->flags,
                                    args->retcode);
-    if (ddom == NULL) {
+    if (ddom == NULL)
         goto cleanup;
-    }
 
     make_nonnull_domain(&ret->ddom, ddom);
 
@@ -2470,13 +2396,11 @@ remoteDispatchDomainMigratePrepareTunnel(struct qemud_server *server ATTRIBUTE_U
     r = virDomainMigratePrepareTunnel(conn, stream->st,
                                       args->flags, dname, args->resource,
                                       args->dom_xml);
-    if (r < 0) {
+    if (r < 0)
         goto cleanup;
-    }
 
-    if (remoteAddClientStream(client, stream, 0) < 0) {
+    if (remoteAddClientStream(client, stream, 0) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2522,9 +2446,8 @@ remoteDispatchListDefinedDomains(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     len = virConnectListDefinedDomains(conn,
                                        ret->names.names_val, args->maxnames);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -2555,9 +2478,8 @@ remoteDispatchDomainLookupById(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = virDomainLookupByID(conn, args->id);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     make_nonnull_domain(&ret->dom, dom);
 
@@ -2589,9 +2511,8 @@ remoteDispatchDomainLookupByName(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = virDomainLookupByName(conn, args->name);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     make_nonnull_domain(&ret->dom, dom);
 
@@ -2623,9 +2544,8 @@ remoteDispatchDomainLookupByUuid(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = virDomainLookupByUUID(conn, (unsigned char *) args->uuid);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     make_nonnull_domain(&ret->dom, dom);
 
@@ -2656,9 +2576,8 @@ remoteDispatchNumOfDefinedDomains(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     ret->num = virConnectNumOfDefinedDomains(conn);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2686,9 +2605,8 @@ remoteDispatchDomainPinVcpu(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     if (args->cpumap.cpumap_len > REMOTE_CPUMAP_MAX) {
         virNetError(VIR_ERR_INTERNAL_ERROR, "%s", _("cpumap_len > REMOTE_CPUMAP_MAX"));
@@ -2698,9 +2616,8 @@ remoteDispatchDomainPinVcpu(struct qemud_server *server ATTRIBUTE_UNUSED,
     rv = virDomainPinVcpu(dom, args->vcpu,
                           (unsigned char *) args->cpumap.cpumap_val,
                           args->cpumap.cpumap_len);
-    if (rv < 0) {
+    if (rv < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2730,13 +2647,11 @@ remoteDispatchDomainReboot(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainReboot(dom, args->flags) < 0) {
+    if (virDomainReboot(dom, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2764,9 +2679,8 @@ remoteDispatchDomainRestore(struct qemud_server *server ATTRIBUTE_UNUSED,
         goto cleanup;
     }
 
-    if (virDomainRestore(conn, args->from) < 0) {
+    if (virDomainRestore(conn, args->from) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2794,13 +2708,11 @@ remoteDispatchDomainResume(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainResume(dom) < 0) {
+    if (virDomainResume(dom) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2830,13 +2742,11 @@ remoteDispatchDomainSave(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainSave(dom, args->to) < 0) {
+    if (virDomainSave(dom, args->to) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2866,13 +2776,11 @@ remoteDispatchDomainCoreDump(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainCoreDump(dom, args->to, args->flags) < 0) {
+    if (virDomainCoreDump(dom, args->to, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2902,13 +2810,11 @@ remoteDispatchDomainSetAutostart(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainSetAutostart(dom, args->autostart) < 0) {
+    if (virDomainSetAutostart(dom, args->autostart) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2938,13 +2844,11 @@ remoteDispatchDomainSetMaxMemory(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainSetMaxMemory(dom, args->memory) < 0) {
+    if (virDomainSetMaxMemory(dom, args->memory) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -2974,13 +2878,11 @@ remoteDispatchDomainSetMemory(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainSetMemory(dom, args->memory) < 0) {
+    if (virDomainSetMemory(dom, args->memory) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3010,13 +2912,11 @@ remoteDispatchDomainSetMemoryFlags(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainSetMemoryFlags(dom, args->memory, args->flags) < 0) {
+    if (virDomainSetMemoryFlags(dom, args->memory, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3108,14 +3008,12 @@ remoteDispatchDomainSetMemoryParameters(struct qemud_server *server
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     r = virDomainSetMemoryParameters(dom, params, nparams, flags);
-    if (r < 0) {
+    if (r < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3166,14 +3064,12 @@ remoteDispatchDomainGetMemoryParameters(struct qemud_server *server
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     r = virDomainGetMemoryParameters(dom, params, &nparams, flags);
-    if (r < 0) {
+    if (r < 0)
         goto cleanup;
-    }
     /* In this case, we need to send back the number of parameters
      * supported
      */
@@ -3329,14 +3225,12 @@ remoteDispatchDomainSetBlkioParameters(struct qemud_server *server
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     r = virDomainSetBlkioParameters(dom, params, nparams, flags);
-    if (r < 0) {
+    if (r < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3387,14 +3281,12 @@ remoteDispatchDomainGetBlkioParameters(struct qemud_server *server
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     r = virDomainGetBlkioParameters(dom, params, &nparams, flags);
-    if (r < 0) {
+    if (r < 0)
         goto cleanup;
-    }
     /* In this case, we need to send back the number of parameters
      * supported
      */
@@ -3490,13 +3382,11 @@ remoteDispatchDomainSetVcpus(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainSetVcpus(dom, args->nvcpus) < 0) {
+    if (virDomainSetVcpus(dom, args->nvcpus) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3526,13 +3416,11 @@ remoteDispatchDomainSetVcpusFlags(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainSetVcpusFlags(dom, args->nvcpus, args->flags) < 0) {
+    if (virDomainSetVcpusFlags(dom, args->nvcpus, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3562,13 +3450,11 @@ remoteDispatchDomainShutdown(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainShutdown(dom) < 0) {
+    if (virDomainShutdown(dom) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3598,13 +3484,11 @@ remoteDispatchDomainSuspend(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainSuspend(dom) < 0) {
+    if (virDomainSuspend(dom) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3634,13 +3518,11 @@ remoteDispatchDomainUndefine(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainUndefine(dom) < 0) {
+    if (virDomainUndefine(dom) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3683,9 +3565,8 @@ remoteDispatchListDefinedNetworks(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     len = virConnectListDefinedNetworks(conn,
                                       ret->names.names_val, args->maxnames);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -3729,9 +3610,8 @@ remoteDispatchListDomains(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     len = virConnectListDomains(conn,
                                 ret->ids.ids_val, args->maxids);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->ids.ids_len = len;
 
     rv = 0;
@@ -3762,13 +3642,11 @@ remoteDispatchDomainManagedSave(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainManagedSave(dom, args->flags) < 0) {
+    if (virDomainManagedSave(dom, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3798,14 +3676,12 @@ remoteDispatchDomainHasManagedSaveImage(struct qemud_server *server ATTRIBUTE_UN
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     ret->ret = virDomainHasManagedSaveImage(dom, args->flags);
-    if (ret->ret < 0) {
+    if (ret->ret < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3835,13 +3711,11 @@ remoteDispatchDomainManagedSaveRemove(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainManagedSaveRemove(dom, args->flags) < 0) {
+    if (virDomainManagedSaveRemove(dom, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3884,9 +3758,8 @@ remoteDispatchListNetworks(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     len = virConnectListNetworks(conn,
                                  ret->names.names_val, args->maxnames);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -3917,13 +3790,11 @@ remoteDispatchNetworkCreate(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     net = get_nonnull_network(conn, args->net);
-    if (net == NULL) {
+    if (net == NULL)
         goto cleanup;
-    }
 
-    if (virNetworkCreate(net) < 0) {
+    if (virNetworkCreate(net) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -3953,9 +3824,8 @@ remoteDispatchNetworkCreateXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     net = virNetworkCreateXML(conn, args->xml);
-    if (net == NULL) {
+    if (net == NULL)
         goto cleanup;
-    }
 
     make_nonnull_network(&ret->net, net);
 
@@ -3987,9 +3857,8 @@ remoteDispatchNetworkDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     net = virNetworkDefineXML(conn, args->xml);
-    if (net == NULL) {
+    if (net == NULL)
         goto cleanup;
-    }
 
     make_nonnull_network(&ret->net, net);
 
@@ -4021,13 +3890,11 @@ remoteDispatchNetworkDestroy(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     net = get_nonnull_network(conn, args->net);
-    if (net == NULL) {
+    if (net == NULL)
         goto cleanup;
-    }
 
-    if (virNetworkDestroy(net) < 0) {
+    if (virNetworkDestroy(net) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4057,15 +3924,13 @@ remoteDispatchNetworkDumpXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     net = get_nonnull_network(conn, args->net);
-    if (net == NULL) {
+    if (net == NULL)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free this. */
     ret->xml = virNetworkGetXMLDesc(net, args->flags);
-    if (!ret->xml) {
+    if (!ret->xml)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4095,13 +3960,11 @@ remoteDispatchNetworkGetAutostart(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     net = get_nonnull_network(conn, args->net);
-    if (net == NULL) {
+    if (net == NULL)
         goto cleanup;
-    }
 
-    if (virNetworkGetAutostart(net, &ret->autostart) < 0) {
+    if (virNetworkGetAutostart(net, &ret->autostart) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4131,15 +3994,13 @@ remoteDispatchNetworkGetBridgeName(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     net = get_nonnull_network(conn, args->net);
-    if (net == NULL) {
+    if (net == NULL)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free this. */
     ret->name = virNetworkGetBridgeName(net);
-    if (!ret->name) {
+    if (!ret->name)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4169,9 +4030,8 @@ remoteDispatchNetworkLookupByName(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     net = virNetworkLookupByName(conn, args->name);
-    if (net == NULL) {
+    if (net == NULL)
         goto cleanup;
-    }
 
     make_nonnull_network(&ret->net, net);
 
@@ -4203,9 +4063,8 @@ remoteDispatchNetworkLookupByUuid(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     net = virNetworkLookupByUUID(conn, (unsigned char *) args->uuid);
-    if (net == NULL) {
+    if (net == NULL)
         goto cleanup;
-    }
 
     make_nonnull_network(&ret->net, net);
 
@@ -4237,13 +4096,11 @@ remoteDispatchNetworkSetAutostart(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     net = get_nonnull_network(conn, args->net);
-    if (net == NULL) {
+    if (net == NULL)
         goto cleanup;
-    }
 
-    if (virNetworkSetAutostart(net, args->autostart) < 0) {
+    if (virNetworkSetAutostart(net, args->autostart) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4273,13 +4130,11 @@ remoteDispatchNetworkUndefine(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     net = get_nonnull_network(conn, args->net);
-    if (net == NULL) {
+    if (net == NULL)
         goto cleanup;
-    }
 
-    if (virNetworkUndefine(net) < 0) {
+    if (virNetworkUndefine(net) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4309,9 +4164,8 @@ remoteDispatchNumOfDefinedNetworks(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     len = virConnectNumOfDefinedNetworks(conn);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->num = len;
 
     rv = 0;
@@ -4339,9 +4193,8 @@ remoteDispatchNumOfDomains(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     ret->num = virConnectNumOfDomains(conn);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4368,9 +4221,8 @@ remoteDispatchNumOfNetworks(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     ret->num = virConnectNumOfNetworks(conn);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4399,9 +4251,8 @@ remoteDispatchNumOfInterfaces(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     ret->num = virConnectNumOfInterfaces(conn);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4442,9 +4293,8 @@ remoteDispatchListInterfaces(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     len = virConnectListInterfaces(conn,
                                    ret->names.names_val, args->maxnames);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -4475,9 +4325,8 @@ remoteDispatchNumOfDefinedInterfaces(struct qemud_server *server ATTRIBUTE_UNUSE
     }
 
     len = virConnectNumOfDefinedInterfaces(conn);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->num = len;
 
     rv = 0;
@@ -4519,9 +4368,8 @@ remoteDispatchListDefinedInterfaces(struct qemud_server *server ATTRIBUTE_UNUSED
 
     len = virConnectListDefinedInterfaces(conn,
                                           ret->names.names_val, args->maxnames);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -4552,9 +4400,8 @@ remoteDispatchInterfaceLookupByName(struct qemud_server *server ATTRIBUTE_UNUSED
     }
 
     iface = virInterfaceLookupByName(conn, args->name);
-    if (iface == NULL) {
+    if (iface == NULL)
         goto cleanup;
-    }
 
     make_nonnull_interface(&ret->iface, iface);
 
@@ -4586,9 +4433,8 @@ remoteDispatchInterfaceLookupByMacString(struct qemud_server *server ATTRIBUTE_U
     }
 
     iface = virInterfaceLookupByMACString(conn, args->mac);
-    if (iface == NULL) {
+    if (iface == NULL)
         goto cleanup;
-    }
 
     make_nonnull_interface(&ret->iface, iface);
 
@@ -4620,15 +4466,13 @@ remoteDispatchInterfaceGetXmlDesc(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     iface = get_nonnull_interface(conn, args->iface);
-    if (iface == NULL) {
+    if (iface == NULL)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free this. */
     ret->xml = virInterfaceGetXMLDesc(iface, args->flags);
-    if (!ret->xml) {
+    if (!ret->xml)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4658,9 +4502,8 @@ remoteDispatchInterfaceDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     iface = virInterfaceDefineXML(conn, args->xml, args->flags);
-    if (iface == NULL) {
+    if (iface == NULL)
         goto cleanup;
-    }
 
     make_nonnull_interface(&ret->iface, iface);
 
@@ -4692,13 +4535,11 @@ remoteDispatchInterfaceUndefine(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     iface = get_nonnull_interface(conn, args->iface);
-    if (iface == NULL) {
+    if (iface == NULL)
         goto cleanup;
-    }
 
-    if (virInterfaceUndefine(iface) < 0) {
+    if (virInterfaceUndefine(iface) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4728,13 +4569,11 @@ remoteDispatchInterfaceCreate(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     iface = get_nonnull_interface(conn, args->iface);
-    if (iface == NULL) {
+    if (iface == NULL)
         goto cleanup;
-    }
 
-    if (virInterfaceCreate(iface, args->flags) < 0) {
+    if (virInterfaceCreate(iface, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4764,13 +4603,11 @@ remoteDispatchInterfaceDestroy(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     iface = get_nonnull_interface(conn, args->iface);
-    if (iface == NULL) {
+    if (iface == NULL)
         goto cleanup;
-    }
 
-    if (virInterfaceDestroy(iface, args->flags) < 0) {
+    if (virInterfaceDestroy(iface, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -4857,9 +4694,8 @@ remoteDispatchAuthSaslInit(struct qemud_server *server,
                     virStrerror(errno, ebuf, sizeof ebuf));
         goto error;
     }
-    if ((localAddr = virSocketFormatAddrFull(&sa, true, ";")) == NULL) {
+    if ((localAddr = virSocketFormatAddrFull(&sa, true, ";")) == NULL)
         goto error;
-    }
 
     /* Get remote address in form  IPADDR:PORT */
     sa.len = sizeof(sa.data.stor);
@@ -5617,9 +5453,8 @@ remoteDispatchListDefinedStoragePools(struct qemud_server *server ATTRIBUTE_UNUS
 
     len = virConnectListDefinedStoragePools(conn,
                                             ret->names.names_val, args->maxnames);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -5663,9 +5498,8 @@ remoteDispatchListStoragePools(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     len = virConnectListStoragePools(conn,
                                      ret->names.names_val, args->maxnames);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -5699,9 +5533,8 @@ remoteDispatchFindStoragePoolSources(struct qemud_server *server ATTRIBUTE_UNUSE
                                          args->type,
                                          args->srcSpec ? *args->srcSpec : NULL,
                                          args->flags);
-    if (ret->xml == NULL) {
+    if (ret->xml == NULL)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -5730,13 +5563,11 @@ remoteDispatchStoragePoolCreate(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
-    if (virStoragePoolCreate(pool, args->flags) < 0) {
+    if (virStoragePoolCreate(pool, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -5766,9 +5597,8 @@ remoteDispatchStoragePoolCreateXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     pool = virStoragePoolCreateXML(conn, args->xml, args->flags);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     make_nonnull_storage_pool(&ret->pool, pool);
 
@@ -5800,9 +5630,8 @@ remoteDispatchStoragePoolDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     pool = virStoragePoolDefineXML(conn, args->xml, args->flags);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     make_nonnull_storage_pool(&ret->pool, pool);
 
@@ -5834,13 +5663,11 @@ remoteDispatchStoragePoolBuild(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
-    if (virStoragePoolBuild(pool, args->flags) < 0) {
+    if (virStoragePoolBuild(pool, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -5871,13 +5698,11 @@ remoteDispatchStoragePoolDestroy(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
-    if (virStoragePoolDestroy(pool) < 0) {
+    if (virStoragePoolDestroy(pool) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -5907,13 +5732,11 @@ remoteDispatchStoragePoolDelete(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
-    if (virStoragePoolDelete(pool, args->flags) < 0) {
+    if (virStoragePoolDelete(pool, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -5943,13 +5766,11 @@ remoteDispatchStoragePoolRefresh(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
-    if (virStoragePoolRefresh(pool, args->flags) < 0) {
+    if (virStoragePoolRefresh(pool, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -5980,13 +5801,11 @@ remoteDispatchStoragePoolGetInfo(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
-    if (virStoragePoolGetInfo(pool, &info) < 0) {
+    if (virStoragePoolGetInfo(pool, &info) < 0)
         goto cleanup;
-    }
 
     ret->state = info.state;
     ret->capacity = info.capacity;
@@ -6021,15 +5840,13 @@ remoteDispatchStoragePoolDumpXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free this. */
     ret->xml = virStoragePoolGetXMLDesc(pool, args->flags);
-    if (!ret->xml) {
+    if (!ret->xml)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6059,13 +5876,11 @@ remoteDispatchStoragePoolGetAutostart(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
-    if (virStoragePoolGetAutostart(pool, &ret->autostart) < 0) {
+    if (virStoragePoolGetAutostart(pool, &ret->autostart) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6096,9 +5911,8 @@ remoteDispatchStoragePoolLookupByName(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     pool = virStoragePoolLookupByName(conn, args->name);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     make_nonnull_storage_pool(&ret->pool, pool);
 
@@ -6130,9 +5944,8 @@ remoteDispatchStoragePoolLookupByUuid(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     pool = virStoragePoolLookupByUUID(conn, (unsigned char *) args->uuid);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     make_nonnull_storage_pool(&ret->pool, pool);
 
@@ -6165,14 +5978,12 @@ remoteDispatchStoragePoolLookupByVolume(struct qemud_server *server ATTRIBUTE_UN
     }
 
     vol = get_nonnull_storage_vol(conn, args->vol);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
     pool = virStoragePoolLookupByVolume(vol);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     make_nonnull_storage_pool(&ret->pool, pool);
 
@@ -6206,13 +6017,11 @@ remoteDispatchStoragePoolSetAutostart(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
-    if (virStoragePoolSetAutostart(pool, args->autostart) < 0) {
+    if (virStoragePoolSetAutostart(pool, args->autostart) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6242,13 +6051,11 @@ remoteDispatchStoragePoolUndefine(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
-    if (virStoragePoolUndefine(pool) < 0) {
+    if (virStoragePoolUndefine(pool) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6277,9 +6084,8 @@ remoteDispatchNumOfStoragePools(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     ret->num = virConnectNumOfStoragePools(conn);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6306,9 +6112,8 @@ remoteDispatchNumOfDefinedStoragePools(struct qemud_server *server ATTRIBUTE_UNU
     }
 
     ret->num = virConnectNumOfDefinedStoragePools(conn);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6343,9 +6148,8 @@ remoteDispatchStoragePoolListVolumes(struct qemud_server *server ATTRIBUTE_UNUSE
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     /* Allocate return buffer. */
     if (VIR_ALLOC_N(ret->names.names_val, args->maxnames) < 0) {
@@ -6355,9 +6159,8 @@ remoteDispatchStoragePoolListVolumes(struct qemud_server *server ATTRIBUTE_UNUSE
 
     len = virStoragePoolListVolumes(pool,
                                     ret->names.names_val, args->maxnames);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -6391,14 +6194,12 @@ remoteDispatchStoragePoolNumOfVolumes(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     ret->num = virStoragePoolNumOfVolumes(pool);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6436,14 +6237,12 @@ remoteDispatchStorageVolCreateXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     vol = virStorageVolCreateXML(pool, args->xml, args->flags);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
     make_nonnull_storage_vol(&ret->vol, vol);
     rv = 0;
@@ -6478,20 +6277,17 @@ remoteDispatchStorageVolCreateXmlFrom(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     clonevol = get_nonnull_storage_vol(conn, args->clonevol);
-    if (clonevol == NULL) {
+    if (clonevol == NULL)
         goto cleanup;
-    }
 
     newvol = virStorageVolCreateXMLFrom(pool, args->xml, clonevol,
                                         args->flags);
-    if (newvol == NULL) {
+    if (newvol == NULL)
         goto cleanup;
-    }
 
     make_nonnull_storage_vol(&ret->vol, newvol);
     rv = 0;
@@ -6526,13 +6322,11 @@ remoteDispatchStorageVolDelete(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     vol = get_nonnull_storage_vol(conn, args->vol);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
-    if (virStorageVolDelete(vol, args->flags) < 0) {
+    if (virStorageVolDelete(vol, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6562,13 +6356,11 @@ remoteDispatchStorageVolWipe(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     vol = get_nonnull_storage_vol(conn, args->vol);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
-    if (virStorageVolWipe(vol, args->flags) < 0) {
+    if (virStorageVolWipe(vol, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6599,13 +6391,11 @@ remoteDispatchStorageVolGetInfo(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     vol = get_nonnull_storage_vol(conn, args->vol);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
-    if (virStorageVolGetInfo(vol, &info) < 0) {
+    if (virStorageVolGetInfo(vol, &info) < 0)
         goto cleanup;
-    }
 
     ret->type = info.type;
     ret->capacity = info.capacity;
@@ -6639,15 +6429,13 @@ remoteDispatchStorageVolDumpXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     vol = get_nonnull_storage_vol(conn, args->vol);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free this. */
     ret->xml = virStorageVolGetXMLDesc(vol, args->flags);
-    if (!ret->xml) {
+    if (!ret->xml)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6678,15 +6466,13 @@ remoteDispatchStorageVolGetPath(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     vol = get_nonnull_storage_vol(conn, args->vol);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free this. */
     ret->name = virStorageVolGetPath(vol);
-    if (!ret->name) {
+    if (!ret->name)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6718,14 +6504,12 @@ remoteDispatchStorageVolLookupByName(struct qemud_server *server ATTRIBUTE_UNUSE
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     vol = virStorageVolLookupByName(pool, args->name);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
     make_nonnull_storage_vol(&ret->vol, vol);
 
@@ -6759,9 +6543,8 @@ remoteDispatchStorageVolLookupByKey(struct qemud_server *server ATTRIBUTE_UNUSED
     }
 
     vol = virStorageVolLookupByKey(conn, args->key);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
     make_nonnull_storage_vol(&ret->vol, vol);
 
@@ -6794,9 +6577,8 @@ remoteDispatchStorageVolLookupByPath(struct qemud_server *server ATTRIBUTE_UNUSE
     }
 
     vol = virStorageVolLookupByPath(conn, args->path);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
     make_nonnull_storage_vol(&ret->vol, vol);
 
@@ -6834,9 +6616,8 @@ remoteDispatchNodeNumOfDevices(struct qemud_server *server ATTRIBUTE_UNUSED,
     ret->num = virNodeNumOfDevices(conn,
                                    args->cap ? *args->cap : NULL,
                                    args->flags);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6879,9 +6660,8 @@ remoteDispatchNodeListDevices(struct qemud_server *server ATTRIBUTE_UNUSED,
     len = virNodeListDevices(conn,
                              args->cap ? *args->cap : NULL,
                              ret->names.names_val, args->maxnames, args->flags);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -6913,9 +6693,8 @@ remoteDispatchNodeDeviceLookupByName(struct qemud_server *server ATTRIBUTE_UNUSE
     }
 
     dev = virNodeDeviceLookupByName(conn, args->name);
-    if (dev == NULL) {
+    if (dev == NULL)
         goto cleanup;
-    }
 
     make_nonnull_node_device(&ret->dev, dev);
 
@@ -6948,15 +6727,13 @@ remoteDispatchNodeDeviceDumpXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dev = virNodeDeviceLookupByName(conn, args->name);
-    if (dev == NULL) {
+    if (dev == NULL)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free this. */
     ret->xml = virNodeDeviceGetXMLDesc(dev, args->flags);
-    if (!ret->xml) {
+    if (!ret->xml)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -6988,9 +6765,8 @@ remoteDispatchNodeDeviceGetParent(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dev = virNodeDeviceLookupByName(conn, args->name);
-    if (dev == NULL) {
+    if (dev == NULL)
         goto cleanup;
-    }
 
     parent = virNodeDeviceGetParent(dev);
 
@@ -7040,14 +6816,12 @@ remoteDispatchNodeDeviceNumOfCaps(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dev = virNodeDeviceLookupByName(conn, args->name);
-    if (dev == NULL) {
+    if (dev == NULL)
         goto cleanup;
-    }
 
     ret->num = virNodeDeviceNumOfCaps(dev);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7079,9 +6853,8 @@ remoteDispatchNodeDeviceListCaps(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dev = virNodeDeviceLookupByName(conn, args->name);
-    if (dev == NULL) {
+    if (dev == NULL)
         goto cleanup;
-    }
 
     if (args->maxnames > REMOTE_NODE_DEVICE_NAME_LIST_MAX) {
         virNetError(VIR_ERR_INTERNAL_ERROR,
@@ -7097,9 +6870,8 @@ remoteDispatchNodeDeviceListCaps(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     len = virNodeDeviceListCaps(dev, ret->names.names_val,
                                 args->maxnames);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -7133,13 +6905,11 @@ remoteDispatchNodeDeviceDettach(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dev = virNodeDeviceLookupByName(conn, args->name);
-    if (dev == NULL) {
+    if (dev == NULL)
         goto cleanup;
-    }
 
-    if (virNodeDeviceDettach(dev) < 0) {
+    if (virNodeDeviceDettach(dev) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7170,13 +6940,11 @@ remoteDispatchNodeDeviceReAttach(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dev = virNodeDeviceLookupByName(conn, args->name);
-    if (dev == NULL) {
+    if (dev == NULL)
         goto cleanup;
-    }
 
-    if (virNodeDeviceReAttach(dev) < 0) {
+    if (virNodeDeviceReAttach(dev) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7207,13 +6975,11 @@ remoteDispatchNodeDeviceReset(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dev = virNodeDeviceLookupByName(conn, args->name);
-    if (dev == NULL) {
+    if (dev == NULL)
         goto cleanup;
-    }
 
-    if (virNodeDeviceReset(dev) < 0) {
+    if (virNodeDeviceReset(dev) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7244,9 +7010,8 @@ remoteDispatchNodeDeviceCreateXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dev = virNodeDeviceCreateXML(conn, args->xml_desc, args->flags);
-    if (dev == NULL) {
+    if (dev == NULL)
         goto cleanup;
-    }
 
     make_nonnull_node_device(&ret->dev, dev);
 
@@ -7279,13 +7044,11 @@ remoteDispatchNodeDeviceDestroy(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dev = virNodeDeviceLookupByName(conn, args->name);
-    if (dev == NULL) {
+    if (dev == NULL)
         goto cleanup;
-    }
 
-    if (virNodeDeviceDestroy(dev) < 0) {
+    if (virNodeDeviceDestroy(dev) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7315,24 +7078,20 @@ static int remoteDispatchStorageVolUpload(struct qemud_server *server ATTRIBUTE_
     }
 
     vol = get_nonnull_storage_vol(conn, args->vol);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
     stream = remoteCreateClientStream(conn, hdr);
-    if (!stream) {
+    if (!stream)
         goto cleanup;
-    }
 
     if (virStorageVolUpload(vol, stream->st,
                             args->offset, args->length,
-                            args->flags) < 0) {
+                            args->flags) < 0)
         goto cleanup;
-    }
 
-    if (remoteAddClientStream(client, stream, 0) < 0) {
+    if (remoteAddClientStream(client, stream, 0) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7366,24 +7125,20 @@ static int remoteDispatchStorageVolDownload(struct qemud_server *server ATTRIBUT
     }
 
     vol = get_nonnull_storage_vol(conn, args->vol);
-    if (vol == NULL) {
+    if (vol == NULL)
         goto cleanup;
-    }
 
     stream = remoteCreateClientStream(conn, hdr);
-    if (!stream) {
+    if (!stream)
         goto cleanup;
-    }
 
     if (virStorageVolDownload(vol, stream->st,
                               args->offset, args->length,
-                              args->flags) < 0) {
+                              args->flags) < 0)
         goto cleanup;
-    }
 
-    if (remoteAddClientStream(client, stream, 1) < 0) {
+    if (remoteAddClientStream(client, stream, 1) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7429,9 +7184,8 @@ remoteDispatchDomainEventsRegister(struct qemud_server *server ATTRIBUTE_UNUSED,
                                                        NULL,
                                                        VIR_DOMAIN_EVENT_ID_LIFECYCLE,
                                                        VIR_DOMAIN_EVENT_CALLBACK(remoteRelayDomainEventLifecycle),
-                                                       client, NULL)) < 0) {
+                                                       client, NULL)) < 0)
         goto cleanup;
-    }
 
     client->domainEventCallbackID[VIR_DOMAIN_EVENT_ID_LIFECYCLE] = callbackID;
 
@@ -7465,9 +7219,8 @@ remoteDispatchDomainEventsDeregister(struct qemud_server *server ATTRIBUTE_UNUSE
     }
 
     if (virConnectDomainEventDeregisterAny(conn,
-                                           client->domainEventCallbackID[VIR_DOMAIN_EVENT_ID_LIFECYCLE]) < 0) {
+                                           client->domainEventCallbackID[VIR_DOMAIN_EVENT_ID_LIFECYCLE]) < 0)
         goto cleanup;
-    }
 
     client->domainEventCallbackID[VIR_DOMAIN_EVENT_ID_LIFECYCLE] = -1;
     rv = 0;
@@ -7559,9 +7312,8 @@ remoteDispatchNumOfSecrets(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     ret->num = virConnectNumOfSecrets(conn);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7601,9 +7353,8 @@ remoteDispatchListSecrets(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     len = virConnectListSecrets(conn, ret->uuids.uuids_val,
                                 args->maxuuids);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->uuids.uuids_len = len;
 
     rv = 0;
@@ -7634,9 +7385,8 @@ remoteDispatchSecretDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     secret = virSecretDefineXML(conn, args->xml, args->flags);
-    if (secret == NULL) {
+    if (secret == NULL)
         goto cleanup;
-    }
 
     make_nonnull_secret(&ret->secret, secret);
     rv = 0;
@@ -7669,14 +7419,12 @@ remoteDispatchSecretGetValue(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     secret = get_nonnull_secret(conn, args->secret);
-    if (secret == NULL) {
+    if (secret == NULL)
         goto cleanup;
-    }
 
     value = virSecretGetValue(secret, &value_size, args->flags);
-    if (value == NULL) {
+    if (value == NULL)
         goto cleanup;
-    }
 
     ret->value.value_len = value_size;
     ret->value.value_val = (char *)value;
@@ -7709,13 +7457,11 @@ remoteDispatchSecretGetXmlDesc(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     secret = get_nonnull_secret(conn, args->secret);
-    if (secret == NULL) {
+    if (secret == NULL)
         goto cleanup;
-    }
     ret->xml = virSecretGetXMLDesc(secret, args->flags);
-    if (ret->xml == NULL) {
+    if (ret->xml == NULL)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7745,9 +7491,8 @@ remoteDispatchSecretLookupByUuid(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     secret = virSecretLookupByUUID(conn, (unsigned char *)args->uuid);
-    if (secret == NULL) {
+    if (secret == NULL)
         goto cleanup;
-    }
 
     make_nonnull_secret(&ret->secret, secret);
 
@@ -7779,13 +7524,11 @@ remoteDispatchSecretSetValue(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     secret = get_nonnull_secret(conn, args->secret);
-    if (secret == NULL) {
+    if (secret == NULL)
         goto cleanup;
-    }
     if (virSecretSetValue(secret, (const unsigned char *)args->value.value_val,
-                          args->value.value_len, args->flags) < 0) {
+                          args->value.value_len, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7815,12 +7558,10 @@ remoteDispatchSecretUndefine(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     secret = get_nonnull_secret(conn, args->secret);
-    if (secret == NULL) {
+    if (secret == NULL)
         goto cleanup;
-    }
-    if (virSecretUndefine(secret) < 0) {
+    if (virSecretUndefine(secret) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7850,9 +7591,8 @@ remoteDispatchSecretLookupByUsage(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     secret = virSecretLookupByUsage(conn, args->usageType, args->usageID);
-    if (secret == NULL) {
+    if (secret == NULL)
         goto cleanup;
-    }
 
     make_nonnull_secret(&ret->secret, secret);
 
@@ -7884,15 +7624,13 @@ static int remoteDispatchDomainIsActive(struct qemud_server *server ATTRIBUTE_UN
     }
 
     domain = get_nonnull_domain(conn, args->dom);
-    if (domain == NULL) {
+    if (domain == NULL)
         goto cleanup;
-    }
 
     ret->active = virDomainIsActive(domain);
 
-    if (ret->active < 0) {
+    if (ret->active < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7921,15 +7659,13 @@ static int remoteDispatchDomainIsPersistent(struct qemud_server *server ATTRIBUT
     }
 
     domain = get_nonnull_domain(conn, args->dom);
-    if (domain == NULL) {
+    if (domain == NULL)
         goto cleanup;
-    }
 
     ret->persistent = virDomainIsPersistent(domain);
 
-    if (ret->persistent < 0) {
+    if (ret->persistent < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7958,15 +7694,13 @@ static int remoteDispatchDomainIsUpdated(struct qemud_server *server ATTRIBUTE_U
     }
 
     domain = get_nonnull_domain(conn, args->dom);
-    if (domain == NULL) {
+    if (domain == NULL)
         goto cleanup;
-    }
 
     ret->updated = virDomainIsUpdated(domain);
 
-    if (ret->updated < 0) {
+    if (ret->updated < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -7995,15 +7729,13 @@ static int remoteDispatchInterfaceIsActive(struct qemud_server *server ATTRIBUTE
     }
 
     iface = get_nonnull_interface(conn, args->iface);
-    if (iface == NULL) {
+    if (iface == NULL)
         goto cleanup;
-    }
 
     ret->active = virInterfaceIsActive(iface);
 
-    if (ret->active < 0) {
+    if (ret->active < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -8032,15 +7764,13 @@ static int remoteDispatchNetworkIsActive(struct qemud_server *server ATTRIBUTE_U
     }
 
     network = get_nonnull_network(conn, args->net);
-    if (network == NULL) {
+    if (network == NULL)
         goto cleanup;
-    }
 
     ret->active = virNetworkIsActive(network);
 
-    if (ret->active < 0) {
+    if (ret->active < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -8069,15 +7799,13 @@ static int remoteDispatchNetworkIsPersistent(struct qemud_server *server ATTRIBU
     }
 
     network = get_nonnull_network(conn, args->net);
-    if (network == NULL) {
+    if (network == NULL)
         goto cleanup;
-    }
 
     ret->persistent = virNetworkIsPersistent(network);
 
-    if (ret->persistent < 0) {
+    if (ret->persistent < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -8106,15 +7834,13 @@ static int remoteDispatchStoragePoolIsActive(struct qemud_server *server ATTRIBU
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     ret->active = virStoragePoolIsActive(pool);
 
-    if (ret->active < 0) {
+    if (ret->active < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -8143,15 +7869,13 @@ static int remoteDispatchStoragePoolIsPersistent(struct qemud_server *server ATT
     }
 
     pool = get_nonnull_storage_pool(conn, args->pool);
-    if (pool == NULL) {
+    if (pool == NULL)
         goto cleanup;
-    }
 
     ret->persistent = virStoragePoolIsPersistent(pool);
 
-    if (ret->persistent < 0) {
+    if (ret->persistent < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -8181,9 +7905,8 @@ static int remoteDispatchIsSecure(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     ret->secure = virConnectIsSecure(conn);
 
-    if (ret->secure < 0) {
+    if (ret->secure < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -8212,9 +7935,8 @@ remoteDispatchCpuCompare(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     result = virConnectCompareCPU(conn, args->xml, args->flags);
-    if (result == VIR_CPU_COMPARE_ERROR) {
+    if (result == VIR_CPU_COMPARE_ERROR)
         goto cleanup;
-    }
 
     ret->result = result;
     rv = 0;
@@ -8247,9 +7969,8 @@ remoteDispatchCpuBaseline(struct qemud_server *server ATTRIBUTE_UNUSED,
                                 (const char **) args->xmlCPUs.xmlCPUs_val,
                                 args->xmlCPUs.xmlCPUs_len,
                                 args->flags);
-    if (cpu == NULL) {
+    if (cpu == NULL)
         goto cleanup;
-    }
 
     ret->cpu = cpu;
 
@@ -8281,13 +8002,11 @@ remoteDispatchDomainGetJobInfo(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainGetJobInfo(dom, &info) < 0) {
+    if (virDomainGetJobInfo(dom, &info) < 0)
         goto cleanup;
-    }
 
     ret->type = info.type;
     ret->timeElapsed = info.timeElapsed;
@@ -8331,13 +8050,11 @@ remoteDispatchDomainAbortJob(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainAbortJob(dom) < 0) {
+    if (virDomainAbortJob(dom) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -8368,13 +8085,11 @@ remoteDispatchDomainMigrateSetMaxDowntime(struct qemud_server *server ATTRIBUTE_
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainMigrateSetMaxDowntime(dom, args->downtime, args->flags) < 0) {
+    if (virDomainMigrateSetMaxDowntime(dom, args->downtime, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -8404,13 +8119,11 @@ remoteDispatchDomainMigrateSetMaxSpeed(struct qemud_server *server ATTRIBUTE_UNU
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainMigrateSetMaxSpeed(dom, args->bandwidth, args->flags) < 0) {
+    if (virDomainMigrateSetMaxSpeed(dom, args->bandwidth, args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -8441,14 +8154,12 @@ remoteDispatchDomainSnapshotCreateXml(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     domain = get_nonnull_domain(conn, args->domain);
-    if (domain == NULL) {
+    if (domain == NULL)
         goto cleanup;
-    }
 
     snapshot = virDomainSnapshotCreateXML(domain, args->xml_desc, args->flags);
-    if (snapshot == NULL) {
+    if (snapshot == NULL)
         goto cleanup;
-    }
 
     make_nonnull_domain_snapshot(&ret->snap, snapshot);
 
@@ -8525,14 +8236,12 @@ remoteDispatchDomainSnapshotNum(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     domain = get_nonnull_domain(conn, args->domain);
-    if (domain == NULL) {
+    if (domain == NULL)
         goto cleanup;
-    }
 
     ret->num = virDomainSnapshotNum(domain, args->flags);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -8569,9 +8278,8 @@ remoteDispatchDomainSnapshotListNames(struct qemud_server *server ATTRIBUTE_UNUS
     }
 
     domain = get_nonnull_domain(conn, args->domain);
-    if (domain == NULL) {
+    if (domain == NULL)
         goto cleanup;
-    }
 
     /* Allocate return buffer. */
     if (VIR_ALLOC_N(ret->names.names_val, args->nameslen) < 0) {
@@ -8583,9 +8291,8 @@ remoteDispatchDomainSnapshotListNames(struct qemud_server *server ATTRIBUTE_UNUS
                                      ret->names.names_val,
                                      args->nameslen,
                                      args->flags);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -8619,14 +8326,12 @@ remoteDispatchDomainSnapshotLookupByName(struct qemud_server *server ATTRIBUTE_U
     }
 
     domain = get_nonnull_domain(conn, args->domain);
-    if (domain == NULL) {
+    if (domain == NULL)
         goto cleanup;
-    }
 
     snapshot = virDomainSnapshotLookupByName(domain, args->name, args->flags);
-    if (snapshot == NULL) {
+    if (snapshot == NULL)
         goto cleanup;
-    }
 
     make_nonnull_domain_snapshot(&ret->snap, snapshot);
 
@@ -8661,14 +8366,12 @@ remoteDispatchDomainHasCurrentSnapshot(struct qemud_server *server ATTRIBUTE_UNU
     }
 
     domain = get_nonnull_domain(conn, args->domain);
-    if (domain == NULL) {
+    if (domain == NULL)
         goto cleanup;
-    }
 
     result = virDomainHasCurrentSnapshot(domain, args->flags);
-    if (result < 0) {
+    if (result < 0)
         goto cleanup;
-    }
 
     ret->result = result;
 
@@ -8701,14 +8404,12 @@ remoteDispatchDomainSnapshotCurrent(struct qemud_server *server ATTRIBUTE_UNUSED
     }
 
     domain = get_nonnull_domain(conn, args->domain);
-    if (domain == NULL) {
+    if (domain == NULL)
         goto cleanup;
-    }
 
     snapshot = virDomainSnapshotCurrent(domain, args->flags);
-    if (snapshot == NULL) {
+    if (snapshot == NULL)
         goto cleanup;
-    }
 
     make_nonnull_domain_snapshot(&ret->snap, snapshot);
 
@@ -8839,9 +8540,8 @@ remoteDispatchDomainEventsRegisterAny(struct qemud_server *server ATTRIBUTE_UNUS
                                                        NULL,
                                                        args->eventID,
                                                        domainEventCallbacks[args->eventID],
-                                                       client, NULL)) < 0) {
+                                                       client, NULL)) < 0)
         goto cleanup;
-    }
 
     client->domainEventCallbackID[args->eventID] = callbackID;
 
@@ -8883,9 +8583,8 @@ remoteDispatchDomainEventsDeregisterAny(struct qemud_server *server ATTRIBUTE_UN
         goto cleanup;
     }
 
-    if (virConnectDomainEventDeregisterAny(conn, callbackID) < 0) {
+    if (virConnectDomainEventDeregisterAny(conn, callbackID) < 0)
         goto cleanup;
-    }
 
     client->domainEventCallbackID[args->eventID] = -1;
     rv = 0;
@@ -8916,9 +8615,8 @@ remoteDispatchNwfilterLookupByName(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     nwfilter = virNWFilterLookupByName(conn, args->name);
-    if (nwfilter == NULL) {
+    if (nwfilter == NULL)
         goto cleanup;
-    }
 
     make_nonnull_nwfilter(&ret->nwfilter, nwfilter);
 
@@ -8950,9 +8648,8 @@ remoteDispatchNwfilterLookupByUuid(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     nwfilter = virNWFilterLookupByUUID(conn, (unsigned char *) args->uuid);
-    if (nwfilter == NULL) {
+    if (nwfilter == NULL)
         goto cleanup;
-    }
 
     make_nonnull_nwfilter(&ret->nwfilter, nwfilter);
 
@@ -8985,9 +8682,8 @@ remoteDispatchNwfilterDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     nwfilter = virNWFilterDefineXML(conn, args->xml);
-    if (nwfilter == NULL) {
+    if (nwfilter == NULL)
         goto cleanup;
-    }
 
     make_nonnull_nwfilter(&ret->nwfilter, nwfilter);
 
@@ -9020,13 +8716,11 @@ remoteDispatchNwfilterUndefine(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     nwfilter = get_nonnull_nwfilter(conn, args->nwfilter);
-    if (nwfilter == NULL) {
+    if (nwfilter == NULL)
         goto cleanup;
-    }
 
-    if (virNWFilterUndefine(nwfilter) < 0) {
+    if (virNWFilterUndefine(nwfilter) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -9069,9 +8763,8 @@ remoteDispatchListNwfilters(struct qemud_server *server ATTRIBUTE_UNUSED,
 
     len = virConnectListNWFilters(conn,
                                   ret->names.names_val, args->maxnames);
-    if (len < 0) {
+    if (len < 0)
         goto cleanup;
-    }
     ret->names.names_len = len;
 
     rv = 0;
@@ -9103,15 +8796,13 @@ remoteDispatchNwfilterGetXmlDesc(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     nwfilter = get_nonnull_nwfilter(conn, args->nwfilter);
-    if (nwfilter == NULL) {
+    if (nwfilter == NULL)
         goto cleanup;
-    }
 
     /* remoteDispatchClientRequest will free this. */
     ret->xml = virNWFilterGetXMLDesc(nwfilter, args->flags);
-    if (!ret->xml) {
+    if (!ret->xml)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -9141,9 +8832,8 @@ remoteDispatchNumOfNwfilters(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     ret->num = virConnectNumOfNWFilters(conn);
-    if (ret->num < 0) {
+    if (ret->num < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -9173,13 +8863,11 @@ remoteDispatchDomainGetBlockInfo(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->dom);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
-    if (virDomainGetBlockInfo(dom, args->path, &info, args->flags) < 0) {
+    if (virDomainGetBlockInfo(dom, args->path, &info, args->flags) < 0)
         goto cleanup;
-    }
 
     ret->capacity = info.capacity;
     ret->allocation = info.allocation;
@@ -9213,14 +8901,12 @@ qemuDispatchMonitorCommand(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     domain = get_nonnull_domain(conn, args->domain);
-    if (domain == NULL) {
+    if (domain == NULL)
         goto cleanup;
-    }
 
     if (virDomainQemuMonitorCommand(domain, args->cmd, &ret->result,
-                                    args->flags) < 0) {
+                                    args->flags) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
@@ -9253,9 +8939,8 @@ remoteDispatchDomainOpenConsole(struct qemud_server *server ATTRIBUTE_UNUSED,
     }
 
     dom = get_nonnull_domain(conn, args->domain);
-    if (dom == NULL) {
+    if (dom == NULL)
         goto cleanup;
-    }
 
     stream = remoteCreateClientStream(conn, hdr);
     if (!stream) {
@@ -9267,13 +8952,11 @@ remoteDispatchDomainOpenConsole(struct qemud_server *server ATTRIBUTE_UNUSED,
                              args->devname ? *args->devname : NULL,
                              stream->st,
                              args->flags);
-    if (r < 0) {
+    if (r < 0)
         goto cleanup;
-    }
 
-    if (remoteAddClientStream(client, stream, 1) < 0) {
+    if (remoteAddClientStream(client, stream, 1) < 0)
         goto cleanup;
-    }
 
     rv = 0;
 
-- 
1.7.4.2


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]