[libvirt] [PATCH] Write error check conditionals in more compact form for dispatcher
Replace cases of type = virConnectGetType(conn); if (type == NULL) goto cleanup; With if (!(type = virConnectGetType(conn))) goto cleanup; * daemon/remote.c: Write error checks in compat form --- daemon/remote.c | 795 +++++++++++++++++++------------------------------------ 1 files changed, 278 insertions(+), 517 deletions(-) diff --git a/daemon/remote.c b/daemon/remote.c index 247a226..66df9f7 100644 --- a/daemon/remote.c +++ b/daemon/remote.c @@ -473,9 +473,7 @@ remoteDispatchSupportsFeature(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ret->supported = virDrvSupportsFeature(conn, args->feature); - - if (ret->supported < 0) + if ((ret->supported = virDrvSupportsFeature(conn, args->feature)) < 0) goto cleanup; rv = 0; @@ -502,15 +500,13 @@ remoteDispatchGetType(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - type = virConnectGetType(conn); - if (type == NULL) + if (!(type = virConnectGetType(conn))) goto cleanup; /* We have to strdup because remoteDispatchClientRequest will * free this string after it's been serialised. */ - ret->type = strdup(type); - if (!ret->type) { + if (!(ret->type = strdup(type))) { virReportOOMError(); goto cleanup; } @@ -598,8 +594,7 @@ remoteDispatchGetHostname(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - hostname = virConnectGetHostname(conn); - if (hostname == NULL) + if (!(hostname = virConnectGetHostname(conn))) goto cleanup; ret->hostname = hostname; @@ -628,8 +623,7 @@ remoteDispatchGetUri(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - uri = virConnectGetURI(conn); - if (uri == NULL) + if (!(uri = virConnectGetURI(conn))) goto cleanup; ret->uri = uri; @@ -660,8 +654,7 @@ remoteDispatchGetSysinfo(struct qemud_server *server ATTRIBUTE_UNUSED, } flags = args->flags; - sysinfo = virConnectGetSysinfo(conn, flags); - if (sysinfo == NULL) + if (!(sysinfo = virConnectGetSysinfo(conn, flags))) goto cleanup; ret->sysinfo = sysinfo; @@ -691,8 +684,7 @@ 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 = virConnectGetMaxVcpus(conn, type)) < 0) goto cleanup; rv = 0; @@ -757,8 +749,7 @@ remoteDispatchGetCapabilities(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - caps = virConnectGetCapabilities(conn); - if (caps == NULL) + if (!(caps = virConnectGetCapabilities(conn))) goto cleanup; ret->capabilities = caps; @@ -779,7 +770,7 @@ remoteDispatchNodeGetCellsFreeMemory(struct qemud_server *server ATTRIBUTE_UNUSE remote_node_get_cells_free_memory_args *args, remote_node_get_cells_free_memory_ret *ret) { - int err; + int len; int rv = -1; if (!conn) { @@ -799,13 +790,13 @@ remoteDispatchNodeGetCellsFreeMemory(struct qemud_server *server ATTRIBUTE_UNUSE goto cleanup; } - err = virNodeGetCellsFreeMemory(conn, + len = virNodeGetCellsFreeMemory(conn, (unsigned long long *)ret->freeMems.freeMems_val, args->startCell, args->maxCells); - if (err <= 0) + if (len <= 0) goto cleanup; - ret->freeMems.freeMems_len = err; + ret->freeMems.freeMems_len = len; rv = 0; @@ -835,8 +826,7 @@ remoteDispatchNodeGetFreeMemory(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - freeMem = virNodeGetFreeMemory(conn); - if (freeMem == 0) + if ((freeMem = virNodeGetFreeMemory(conn)) == 0) goto cleanup; ret->freeMem = freeMem; rv = 0; @@ -867,12 +857,10 @@ remoteDispatchDomainGetSchedulerType(struct qemud_server *server ATTRIBUTE_UNUSE goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; - type = virDomainGetSchedulerType(dom, &nparams); - if (type == NULL) + if (!(type = virDomainGetSchedulerType(dom, &nparams))) goto cleanup; ret->type = type; @@ -898,7 +886,7 @@ remoteDispatchDomainGetSchedulerParameters(struct qemud_server *server ATTRIBUTE { virDomainPtr dom = NULL; virSchedParameterPtr params; - int i, r, nparams; + int i, nparams; int rv = -1; if (!conn) { @@ -917,12 +905,10 @@ remoteDispatchDomainGetSchedulerParameters(struct qemud_server *server ATTRIBUTE goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; - r = virDomainGetSchedulerParameters(dom, params, &nparams); - if (r < 0) + if (virDomainGetSchedulerParameters(dom, params, &nparams) < 0) goto cleanup; /* Serialise the scheduler parameters. */ @@ -984,7 +970,7 @@ remoteDispatchDomainSetSchedulerParameters(struct qemud_server *server ATTRIBUTE void *ret ATTRIBUTE_UNUSED) { virDomainPtr dom = NULL; - int i, r, nparams; + int i, nparams; virSchedParameterPtr params; int rv = -1; @@ -1028,12 +1014,10 @@ remoteDispatchDomainSetSchedulerParameters(struct qemud_server *server ATTRIBUTE } } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; - r = virDomainSetSchedulerParameters(dom, params, nparams); - if (r < 0) + if (virDomainSetSchedulerParameters(dom, params, nparams) < 0) goto cleanup; rv = 0; @@ -1066,8 +1050,7 @@ remoteDispatchDomainBlockStats(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; path = args->path; @@ -1109,8 +1092,7 @@ remoteDispatchDomainInterfaceStats(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; path = args->path; @@ -1161,8 +1143,7 @@ remoteDispatchDomainMemoryStats(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; /* Allocate stats array for making dispatch call */ @@ -1219,8 +1200,7 @@ remoteDispatchDomainBlockPeek(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; path = args->path; offset = args->offset; @@ -1275,8 +1255,7 @@ remoteDispatchDomainMemoryPeek(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; offset = args->offset; size = args->size; @@ -1329,8 +1308,7 @@ remoteDispatchDomainAttachDevice(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainAttachDevice(dom, args->xml) < 0) @@ -1363,8 +1341,7 @@ remoteDispatchDomainAttachDeviceFlags(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainAttachDeviceFlags(dom, args->xml, args->flags) < 0) @@ -1397,8 +1374,7 @@ remoteDispatchDomainUpdateDeviceFlags(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainUpdateDeviceFlags(dom, args->xml, args->flags) < 0) @@ -1431,8 +1407,7 @@ remoteDispatchDomainCreate(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainCreate(dom) < 0) @@ -1460,8 +1435,7 @@ remoteDispatchDomainCreateWithFlags(struct qemud_server *server ATTRIBUTE_UNUSED int rv = -1; virDomainPtr dom = NULL; - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainCreateWithFlags(dom, args->flags) < 0) @@ -1496,8 +1470,7 @@ remoteDispatchDomainCreateXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = virDomainCreateXML(conn, args->xml_desc, args->flags); - if (dom == NULL) + if (!(dom = virDomainCreateXML(conn, args->xml_desc, args->flags))) goto cleanup; make_nonnull_domain(&ret->dom, dom); @@ -1529,8 +1502,7 @@ remoteDispatchDomainDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = virDomainDefineXML(conn, args->xml); - if (dom == NULL) + if (!(dom = virDomainDefineXML(conn, args->xml))) goto cleanup; make_nonnull_domain(&ret->dom, dom); @@ -1562,8 +1534,7 @@ remoteDispatchDomainDestroy(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainDestroy(dom) < 0) @@ -1596,8 +1567,7 @@ remoteDispatchDomainDetachDevice(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainDetachDevice(dom, args->xml) < 0) @@ -1630,8 +1600,7 @@ remoteDispatchDomainDetachDeviceFlags(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainDetachDeviceFlags(dom, args->xml, args->flags) < 0) @@ -1664,13 +1633,11 @@ remoteDispatchDomainDumpXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; /* remoteDispatchClientRequest will free this. */ - ret->xml = virDomainGetXMLDesc(dom, args->flags); - if (!ret->xml) + if (!(ret->xml = virDomainGetXMLDesc(dom, args->flags))) goto cleanup; rv = 0; @@ -1700,12 +1667,12 @@ remoteDispatchDomainXmlFromNative(struct qemud_server *server ATTRIBUTE_UNUSED, } /* remoteDispatchClientRequest will free this. */ - ret->domainXml = virConnectDomainXMLFromNative(conn, - args->nativeFormat, - args->nativeConfig, - args->flags); - if (!ret->domainXml) + if (!(ret->domainXml = virConnectDomainXMLFromNative(conn, + args->nativeFormat, + args->nativeConfig, + args->flags))) goto cleanup; + rv = 0; cleanup: @@ -1731,12 +1698,12 @@ remoteDispatchDomainXmlToNative(struct qemud_server *server ATTRIBUTE_UNUSED, } /* remoteDispatchClientRequest will free this. */ - ret->nativeConfig = virConnectDomainXMLToNative(conn, - args->nativeFormat, - args->domainXml, - args->flags); - if (!ret->nativeConfig) + if (!(ret->nativeConfig = virConnectDomainXMLToNative(conn, + args->nativeFormat, + args->domainXml, + args->flags))) goto cleanup; + rv = 0; cleanup: @@ -1763,8 +1730,7 @@ remoteDispatchDomainGetAutostart(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainGetAutostart(dom, &ret->autostart) < 0) @@ -1798,8 +1764,7 @@ remoteDispatchDomainGetInfo(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainGetInfo(dom, &info) < 0) @@ -1838,12 +1803,10 @@ remoteDispatchDomainGetMaxMemory(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; - ret->memory = virDomainGetMaxMemory(dom); - if (ret->memory == 0) + if ((ret->memory = virDomainGetMaxMemory(dom)) == 0) goto cleanup; rv = 0; @@ -1873,12 +1836,10 @@ remoteDispatchDomainGetMaxVcpus(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; - ret->num = virDomainGetMaxVcpus(dom); - if (ret->num < 0) + if ((ret->num = virDomainGetMaxVcpus(dom)) < 0) goto cleanup; rv = 0; @@ -1909,8 +1870,7 @@ remoteDispatchDomainGetSecurityLabel(struct qemud_server *server ATTRIBUTE_UNUSE goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (VIR_ALLOC(seclabel) < 0) { @@ -2000,14 +1960,13 @@ remoteDispatchDomainGetOsType(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; /* remoteDispatchClientRequest will free this */ - ret->type = virDomainGetOSType(dom); - if (ret->type == NULL) + if (!(ret->type = virDomainGetOSType(dom))) goto cleanup; + rv = 0; cleanup: @@ -2038,8 +1997,7 @@ remoteDispatchDomainGetVcpus(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (args->maxinfo > REMOTE_VCPUINFO_MAX) { @@ -2059,10 +2017,9 @@ remoteDispatchDomainGetVcpus(struct qemud_server *server ATTRIBUTE_UNUSED, VIR_ALLOC_N(cpumaps, args->maxinfo * args->maplen) < 0) goto no_memory; - info_len = virDomainGetVcpus(dom, - info, args->maxinfo, - cpumaps, args->maplen); - if (info_len < 0) + if ((info_len = virDomainGetVcpus(dom, + info, args->maxinfo, + cpumaps, args->maplen)) < 0) goto cleanup; /* Allocate the return buffer for info. */ @@ -2120,12 +2077,10 @@ remoteDispatchDomainGetVcpusFlags(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; - ret->num = virDomainGetVcpusFlags(dom, args->flags); - if (ret->num < 0) + if ((ret->num = virDomainGetVcpusFlags(dom, args->flags)) < 0) goto cleanup; rv = 0; @@ -2147,7 +2102,6 @@ remoteDispatchDomainMigratePrepare(struct qemud_server *server ATTRIBUTE_UNUSED, remote_domain_migrate_prepare_args *args, remote_domain_migrate_prepare_ret *ret) { - int r; char *cookie = NULL; int cookielen = 0; char *uri_in; @@ -2169,10 +2123,9 @@ remoteDispatchDomainMigratePrepare(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - r = virDomainMigratePrepare(conn, &cookie, &cookielen, + if (virDomainMigratePrepare(conn, &cookie, &cookielen, uri_in, uri_out, - args->flags, dname, args->resource); - if (r < 0) + args->flags, dname, args->resource) < 0) goto cleanup; /* remoteDispatchClientRequest will free cookie, uri_out and @@ -2205,7 +2158,6 @@ remoteDispatchDomainMigratePerform(struct qemud_server *server ATTRIBUTE_UNUSED, remote_domain_migrate_perform_args *args, void *ret ATTRIBUTE_UNUSED) { - int r; virDomainPtr dom = NULL; char *dname; int rv = -1; @@ -2215,18 +2167,16 @@ remoteDispatchDomainMigratePerform(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; dname = args->dname == NULL ? NULL : *args->dname; - r = virDomainMigratePerform(dom, + if (virDomainMigratePerform(dom, args->cookie.cookie_val, args->cookie.cookie_len, args->uri, - args->flags, dname, args->resource); - if (r < 0) + args->flags, dname, args->resource) < 0) goto cleanup; rv = 0; @@ -2256,12 +2206,11 @@ remoteDispatchDomainMigrateFinish(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ddom = virDomainMigrateFinish(conn, args->dname, - args->cookie.cookie_val, - args->cookie.cookie_len, - args->uri, - args->flags); - if (ddom == NULL) + if (!(ddom = virDomainMigrateFinish(conn, args->dname, + args->cookie.cookie_val, + args->cookie.cookie_len, + args->uri, + args->flags))) goto cleanup; make_nonnull_domain(&ret->ddom, ddom); @@ -2284,7 +2233,6 @@ remoteDispatchDomainMigratePrepare2(struct qemud_server *server ATTRIBUTE_UNUSED remote_domain_migrate_prepare2_args *args, remote_domain_migrate_prepare2_ret *ret) { - int r; char *cookie = NULL; int cookielen = 0; char *uri_in; @@ -2306,11 +2254,10 @@ remoteDispatchDomainMigratePrepare2(struct qemud_server *server ATTRIBUTE_UNUSED goto cleanup; } - r = virDomainMigratePrepare2(conn, &cookie, &cookielen, + if (virDomainMigratePrepare2(conn, &cookie, &cookielen, uri_in, uri_out, args->flags, dname, args->resource, - args->dom_xml); - if (r < 0) + args->dom_xml) < 0) goto cleanup; /* remoteDispatchClientRequest will free cookie, uri_out and @@ -2345,13 +2292,12 @@ remoteDispatchDomainMigrateFinish2(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ddom = virDomainMigrateFinish2(conn, args->dname, - args->cookie.cookie_val, - args->cookie.cookie_len, - args->uri, - args->flags, - args->retcode); - if (ddom == NULL) + if (!(ddom = virDomainMigrateFinish2(conn, args->dname, + args->cookie.cookie_val, + args->cookie.cookie_len, + args->uri, + args->flags, + args->retcode))) goto cleanup; make_nonnull_domain(&ret->ddom, ddom); @@ -2375,7 +2321,6 @@ remoteDispatchDomainMigratePrepareTunnel(struct qemud_server *server ATTRIBUTE_U remote_domain_migrate_prepare_tunnel_args *args, void *ret ATTRIBUTE_UNUSED) { - int r; char *dname; struct qemud_client_stream *stream = NULL; int rv = -1; @@ -2387,16 +2332,14 @@ remoteDispatchDomainMigratePrepareTunnel(struct qemud_server *server ATTRIBUTE_U dname = args->dname == NULL ? NULL : *args->dname; - stream = remoteCreateClientStream(conn, hdr); - if (!stream) { + if (!(stream = remoteCreateClientStream(conn, hdr))) { virReportOOMError(); goto cleanup; } - r = virDomainMigratePrepareTunnel(conn, stream->st, + if (virDomainMigratePrepareTunnel(conn, stream->st, args->flags, dname, args->resource, - args->dom_xml); - if (r < 0) + args->dom_xml) < 0) goto cleanup; if (remoteAddClientStream(client, stream, 0) < 0) @@ -2477,8 +2420,7 @@ remoteDispatchDomainLookupById(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = virDomainLookupByID(conn, args->id); - if (dom == NULL) + if (!(dom = virDomainLookupByID(conn, args->id))) goto cleanup; make_nonnull_domain(&ret->dom, dom); @@ -2510,8 +2452,7 @@ remoteDispatchDomainLookupByName(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = virDomainLookupByName(conn, args->name); - if (dom == NULL) + if (!(dom = virDomainLookupByName(conn, args->name))) goto cleanup; make_nonnull_domain(&ret->dom, dom); @@ -2543,8 +2484,7 @@ remoteDispatchDomainLookupByUuid(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = virDomainLookupByUUID(conn, (unsigned char *) args->uuid); - if (dom == NULL) + if (!(dom = virDomainLookupByUUID(conn, (unsigned char *) args->uuid))) goto cleanup; make_nonnull_domain(&ret->dom, dom); @@ -2575,8 +2515,7 @@ remoteDispatchNumOfDefinedDomains(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ret->num = virConnectNumOfDefinedDomains(conn); - if (ret->num < 0) + if ((ret->num = virConnectNumOfDefinedDomains(conn)) < 0) goto cleanup; rv = 0; @@ -2604,8 +2543,7 @@ remoteDispatchDomainPinVcpu(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (args->cpumap.cpumap_len > REMOTE_CPUMAP_MAX) { @@ -2613,10 +2551,9 @@ remoteDispatchDomainPinVcpu(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - rv = virDomainPinVcpu(dom, args->vcpu, - (unsigned char *) args->cpumap.cpumap_val, - args->cpumap.cpumap_len); - if (rv < 0) + if (virDomainPinVcpu(dom, args->vcpu, + (unsigned char *) args->cpumap.cpumap_val, + args->cpumap.cpumap_len) < 0) goto cleanup; rv = 0; @@ -2646,8 +2583,7 @@ remoteDispatchDomainReboot(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainReboot(dom, args->flags) < 0) @@ -2707,8 +2643,7 @@ remoteDispatchDomainResume(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainResume(dom) < 0) @@ -2741,8 +2676,7 @@ remoteDispatchDomainSave(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainSave(dom, args->to) < 0) @@ -2775,8 +2709,7 @@ remoteDispatchDomainCoreDump(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainCoreDump(dom, args->to, args->flags) < 0) @@ -2809,8 +2742,7 @@ remoteDispatchDomainSetAutostart(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainSetAutostart(dom, args->autostart) < 0) @@ -2843,8 +2775,7 @@ remoteDispatchDomainSetMaxMemory(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainSetMaxMemory(dom, args->memory) < 0) @@ -2877,8 +2808,7 @@ remoteDispatchDomainSetMemory(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainSetMemory(dom, args->memory) < 0) @@ -2911,8 +2841,7 @@ remoteDispatchDomainSetMemoryFlags(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainSetMemoryFlags(dom, args->memory, args->flags) < 0) @@ -2941,7 +2870,7 @@ remoteDispatchDomainSetMemoryParameters(struct qemud_server *server * args, void *ret ATTRIBUTE_UNUSED) { virDomainPtr dom = NULL; - int i, r, nparams; + int i, nparams; virMemoryParameterPtr params; unsigned int flags; int rv = -1; @@ -3007,12 +2936,10 @@ remoteDispatchDomainSetMemoryParameters(struct qemud_server *server } } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; - r = virDomainSetMemoryParameters(dom, params, nparams, flags); - if (r < 0) + if (virDomainSetMemoryParameters(dom, params, nparams, flags) < 0) goto cleanup; rv = 0; @@ -3042,7 +2969,7 @@ remoteDispatchDomainGetMemoryParameters(struct qemud_server *server { virDomainPtr dom = NULL; virMemoryParameterPtr params; - int i, r, nparams; + int i, nparams; unsigned int flags; int rv = -1; @@ -3063,13 +2990,12 @@ remoteDispatchDomainGetMemoryParameters(struct qemud_server *server goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; - r = virDomainGetMemoryParameters(dom, params, &nparams, flags); - if (r < 0) + if (virDomainGetMemoryParameters(dom, params, &nparams, flags) < 0) goto cleanup; + /* In this case, we need to send back the number of parameters * supported */ @@ -3158,7 +3084,7 @@ remoteDispatchDomainSetBlkioParameters(struct qemud_server *server * args, void *ret ATTRIBUTE_UNUSED) { virDomainPtr dom = NULL; - int i, r, nparams; + int i, nparams; virBlkioParameterPtr params; unsigned int flags; int rv = -1; @@ -3224,12 +3150,10 @@ remoteDispatchDomainSetBlkioParameters(struct qemud_server *server } } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; - r = virDomainSetBlkioParameters(dom, params, nparams, flags); - if (r < 0) + if (virDomainSetBlkioParameters(dom, params, nparams, flags) < 0) goto cleanup; rv = 0; @@ -3259,7 +3183,7 @@ remoteDispatchDomainGetBlkioParameters(struct qemud_server *server { virDomainPtr dom = NULL; virBlkioParameterPtr params; - int i, r, nparams; + int i, nparams; unsigned int flags; int rv = -1; @@ -3280,13 +3204,12 @@ remoteDispatchDomainGetBlkioParameters(struct qemud_server *server goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; - r = virDomainGetBlkioParameters(dom, params, &nparams, flags); - if (r < 0) + if (virDomainGetBlkioParameters(dom, params, &nparams, flags) < 0) goto cleanup; + /* In this case, we need to send back the number of parameters * supported */ @@ -3381,8 +3304,7 @@ remoteDispatchDomainSetVcpus(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainSetVcpus(dom, args->nvcpus) < 0) @@ -3415,8 +3337,7 @@ remoteDispatchDomainSetVcpusFlags(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainSetVcpusFlags(dom, args->nvcpus, args->flags) < 0) @@ -3449,8 +3370,7 @@ remoteDispatchDomainShutdown(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainShutdown(dom) < 0) @@ -3483,8 +3403,7 @@ remoteDispatchDomainSuspend(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainSuspend(dom) < 0) @@ -3517,8 +3436,7 @@ remoteDispatchDomainUndefine(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainUndefine(dom) < 0) @@ -3641,8 +3559,7 @@ remoteDispatchDomainManagedSave(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainManagedSave(dom, args->flags) < 0) @@ -3675,12 +3592,10 @@ remoteDispatchDomainHasManagedSaveImage(struct qemud_server *server ATTRIBUTE_UN goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; - ret->ret = virDomainHasManagedSaveImage(dom, args->flags); - if (ret->ret < 0) + if ((ret->ret = virDomainHasManagedSaveImage(dom, args->flags)) < 0) goto cleanup; rv = 0; @@ -3710,8 +3625,7 @@ remoteDispatchDomainManagedSaveRemove(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainManagedSaveRemove(dom, args->flags) < 0) @@ -3789,8 +3703,7 @@ remoteDispatchNetworkCreate(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - net = get_nonnull_network(conn, args->net); - if (net == NULL) + if (!(net = get_nonnull_network(conn, args->net))) goto cleanup; if (virNetworkCreate(net) < 0) @@ -3823,8 +3736,7 @@ remoteDispatchNetworkCreateXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - net = virNetworkCreateXML(conn, args->xml); - if (net == NULL) + if (!(net = virNetworkCreateXML(conn, args->xml))) goto cleanup; make_nonnull_network(&ret->net, net); @@ -3856,8 +3768,7 @@ remoteDispatchNetworkDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - net = virNetworkDefineXML(conn, args->xml); - if (net == NULL) + if (!(net = virNetworkDefineXML(conn, args->xml))) goto cleanup; make_nonnull_network(&ret->net, net); @@ -3889,8 +3800,7 @@ remoteDispatchNetworkDestroy(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - net = get_nonnull_network(conn, args->net); - if (net == NULL) + if (!(net = get_nonnull_network(conn, args->net))) goto cleanup; if (virNetworkDestroy(net) < 0) @@ -3923,13 +3833,11 @@ remoteDispatchNetworkDumpXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - net = get_nonnull_network(conn, args->net); - if (net == NULL) + if (!(net = get_nonnull_network(conn, args->net))) goto cleanup; /* remoteDispatchClientRequest will free this. */ - ret->xml = virNetworkGetXMLDesc(net, args->flags); - if (!ret->xml) + if (!(ret->xml = virNetworkGetXMLDesc(net, args->flags))) goto cleanup; rv = 0; @@ -3959,8 +3867,7 @@ remoteDispatchNetworkGetAutostart(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - net = get_nonnull_network(conn, args->net); - if (net == NULL) + if (!(net = get_nonnull_network(conn, args->net))) goto cleanup; if (virNetworkGetAutostart(net, &ret->autostart) < 0) @@ -3993,13 +3900,11 @@ remoteDispatchNetworkGetBridgeName(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - net = get_nonnull_network(conn, args->net); - if (net == NULL) + if (!(net = get_nonnull_network(conn, args->net))) goto cleanup; /* remoteDispatchClientRequest will free this. */ - ret->name = virNetworkGetBridgeName(net); - if (!ret->name) + if (!(ret->name = virNetworkGetBridgeName(net))) goto cleanup; rv = 0; @@ -4029,8 +3934,7 @@ remoteDispatchNetworkLookupByName(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - net = virNetworkLookupByName(conn, args->name); - if (net == NULL) + if (!(net = virNetworkLookupByName(conn, args->name))) goto cleanup; make_nonnull_network(&ret->net, net); @@ -4062,8 +3966,7 @@ remoteDispatchNetworkLookupByUuid(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - net = virNetworkLookupByUUID(conn, (unsigned char *) args->uuid); - if (net == NULL) + if (!(net = virNetworkLookupByUUID(conn, (unsigned char *) args->uuid))) goto cleanup; make_nonnull_network(&ret->net, net); @@ -4095,8 +3998,7 @@ remoteDispatchNetworkSetAutostart(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - net = get_nonnull_network(conn, args->net); - if (net == NULL) + if (!(net = get_nonnull_network(conn, args->net))) goto cleanup; if (virNetworkSetAutostart(net, args->autostart) < 0) @@ -4129,8 +4031,7 @@ remoteDispatchNetworkUndefine(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - net = get_nonnull_network(conn, args->net); - if (net == NULL) + if (!(net = get_nonnull_network(conn, args->net))) goto cleanup; if (virNetworkUndefine(net) < 0) @@ -4192,8 +4093,7 @@ remoteDispatchNumOfDomains(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ret->num = virConnectNumOfDomains(conn); - if (ret->num < 0) + if ((ret->num = virConnectNumOfDomains(conn)) < 0) goto cleanup; rv = 0; @@ -4220,8 +4120,7 @@ remoteDispatchNumOfNetworks(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ret->num = virConnectNumOfNetworks(conn); - if (ret->num < 0) + if ((ret->num = virConnectNumOfNetworks(conn)) < 0) goto cleanup; rv = 0; @@ -4250,8 +4149,7 @@ remoteDispatchNumOfInterfaces(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ret->num = virConnectNumOfInterfaces(conn); - if (ret->num < 0) + if ((ret->num = virConnectNumOfInterfaces(conn)) < 0) goto cleanup; rv = 0; @@ -4399,8 +4297,7 @@ remoteDispatchInterfaceLookupByName(struct qemud_server *server ATTRIBUTE_UNUSED goto cleanup; } - iface = virInterfaceLookupByName(conn, args->name); - if (iface == NULL) + if (!(iface = virInterfaceLookupByName(conn, args->name))) goto cleanup; make_nonnull_interface(&ret->iface, iface); @@ -4432,8 +4329,7 @@ remoteDispatchInterfaceLookupByMacString(struct qemud_server *server ATTRIBUTE_U goto cleanup; } - iface = virInterfaceLookupByMACString(conn, args->mac); - if (iface == NULL) + if (!(iface = virInterfaceLookupByMACString(conn, args->mac))) goto cleanup; make_nonnull_interface(&ret->iface, iface); @@ -4465,13 +4361,11 @@ remoteDispatchInterfaceGetXmlDesc(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - iface = get_nonnull_interface(conn, args->iface); - if (iface == NULL) + if (!(iface = get_nonnull_interface(conn, args->iface))) goto cleanup; /* remoteDispatchClientRequest will free this. */ - ret->xml = virInterfaceGetXMLDesc(iface, args->flags); - if (!ret->xml) + if (!(ret->xml = virInterfaceGetXMLDesc(iface, args->flags))) goto cleanup; rv = 0; @@ -4501,8 +4395,7 @@ remoteDispatchInterfaceDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - iface = virInterfaceDefineXML(conn, args->xml, args->flags); - if (iface == NULL) + if (!(iface = virInterfaceDefineXML(conn, args->xml, args->flags))) goto cleanup; make_nonnull_interface(&ret->iface, iface); @@ -4534,8 +4427,7 @@ remoteDispatchInterfaceUndefine(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - iface = get_nonnull_interface(conn, args->iface); - if (iface == NULL) + if (!(iface = get_nonnull_interface(conn, args->iface))) goto cleanup; if (virInterfaceUndefine(iface) < 0) @@ -4568,8 +4460,7 @@ remoteDispatchInterfaceCreate(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - iface = get_nonnull_interface(conn, args->iface); - if (iface == NULL) + if (!(iface = get_nonnull_interface(conn, args->iface))) goto cleanup; if (virInterfaceCreate(iface, args->flags) < 0) @@ -4602,8 +4493,7 @@ remoteDispatchInterfaceDestroy(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - iface = get_nonnull_interface(conn, args->iface); - if (iface == NULL) + if (!(iface = get_nonnull_interface(conn, args->iface))) goto cleanup; if (virInterfaceDestroy(iface, args->flags) < 0) @@ -5528,12 +5418,11 @@ remoteDispatchFindStoragePoolSources(struct qemud_server *server ATTRIBUTE_UNUSE goto cleanup; } - ret->xml = - virConnectFindStoragePoolSources(conn, - args->type, - args->srcSpec ? *args->srcSpec : NULL, - args->flags); - if (ret->xml == NULL) + if (!(ret->xml = + virConnectFindStoragePoolSources(conn, + args->type, + args->srcSpec ? *args->srcSpec : NULL, + args->flags))) goto cleanup; rv = 0; @@ -5562,8 +5451,7 @@ remoteDispatchStoragePoolCreate(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; if (virStoragePoolCreate(pool, args->flags) < 0) @@ -5596,8 +5484,7 @@ remoteDispatchStoragePoolCreateXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - pool = virStoragePoolCreateXML(conn, args->xml, args->flags); - if (pool == NULL) + if (!(pool = virStoragePoolCreateXML(conn, args->xml, args->flags))) goto cleanup; make_nonnull_storage_pool(&ret->pool, pool); @@ -5629,8 +5516,7 @@ remoteDispatchStoragePoolDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - pool = virStoragePoolDefineXML(conn, args->xml, args->flags); - if (pool == NULL) + if (!(pool = virStoragePoolDefineXML(conn, args->xml, args->flags))) goto cleanup; make_nonnull_storage_pool(&ret->pool, pool); @@ -5662,8 +5548,7 @@ remoteDispatchStoragePoolBuild(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; if (virStoragePoolBuild(pool, args->flags) < 0) @@ -5697,8 +5582,7 @@ remoteDispatchStoragePoolDestroy(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; if (virStoragePoolDestroy(pool) < 0) @@ -5731,8 +5615,7 @@ remoteDispatchStoragePoolDelete(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; if (virStoragePoolDelete(pool, args->flags) < 0) @@ -5765,8 +5648,7 @@ remoteDispatchStoragePoolRefresh(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; if (virStoragePoolRefresh(pool, args->flags) < 0) @@ -5800,8 +5682,7 @@ remoteDispatchStoragePoolGetInfo(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; if (virStoragePoolGetInfo(pool, &info) < 0) @@ -5839,13 +5720,11 @@ remoteDispatchStoragePoolDumpXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; /* remoteDispatchClientRequest will free this. */ - ret->xml = virStoragePoolGetXMLDesc(pool, args->flags); - if (!ret->xml) + if (!(ret->xml = virStoragePoolGetXMLDesc(pool, args->flags))) goto cleanup; rv = 0; @@ -5875,8 +5754,7 @@ remoteDispatchStoragePoolGetAutostart(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; if (virStoragePoolGetAutostart(pool, &ret->autostart) < 0) @@ -5910,8 +5788,7 @@ remoteDispatchStoragePoolLookupByName(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - pool = virStoragePoolLookupByName(conn, args->name); - if (pool == NULL) + if (!(pool = virStoragePoolLookupByName(conn, args->name))) goto cleanup; make_nonnull_storage_pool(&ret->pool, pool); @@ -5943,8 +5820,7 @@ remoteDispatchStoragePoolLookupByUuid(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - pool = virStoragePoolLookupByUUID(conn, (unsigned char *) args->uuid); - if (pool == NULL) + if (!(pool = virStoragePoolLookupByUUID(conn, (unsigned char *) args->uuid))) goto cleanup; make_nonnull_storage_pool(&ret->pool, pool); @@ -5977,12 +5853,10 @@ remoteDispatchStoragePoolLookupByVolume(struct qemud_server *server ATTRIBUTE_UN goto cleanup; } - vol = get_nonnull_storage_vol(conn, args->vol); - if (vol == NULL) + if (!(vol = get_nonnull_storage_vol(conn, args->vol))) goto cleanup; - pool = virStoragePoolLookupByVolume(vol); - if (pool == NULL) + if (!(pool = virStoragePoolLookupByVolume(vol))) goto cleanup; make_nonnull_storage_pool(&ret->pool, pool); @@ -6016,8 +5890,7 @@ remoteDispatchStoragePoolSetAutostart(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; if (virStoragePoolSetAutostart(pool, args->autostart) < 0) @@ -6050,8 +5923,7 @@ remoteDispatchStoragePoolUndefine(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; if (virStoragePoolUndefine(pool) < 0) @@ -6083,8 +5955,7 @@ remoteDispatchNumOfStoragePools(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ret->num = virConnectNumOfStoragePools(conn); - if (ret->num < 0) + if ((ret->num = virConnectNumOfStoragePools(conn)) < 0) goto cleanup; rv = 0; @@ -6111,8 +5982,7 @@ remoteDispatchNumOfDefinedStoragePools(struct qemud_server *server ATTRIBUTE_UNU goto cleanup; } - ret->num = virConnectNumOfDefinedStoragePools(conn); - if (ret->num < 0) + if ((ret->num = virConnectNumOfDefinedStoragePools(conn)) < 0) goto cleanup; rv = 0; @@ -6147,8 +6017,7 @@ remoteDispatchStoragePoolListVolumes(struct qemud_server *server ATTRIBUTE_UNUSE goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; /* Allocate return buffer. */ @@ -6193,12 +6062,10 @@ remoteDispatchStoragePoolNumOfVolumes(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; - ret->num = virStoragePoolNumOfVolumes(pool); - if (ret->num < 0) + if ((ret->num = virStoragePoolNumOfVolumes(pool)) < 0) goto cleanup; rv = 0; @@ -6236,12 +6103,10 @@ remoteDispatchStorageVolCreateXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; - vol = virStorageVolCreateXML(pool, args->xml, args->flags); - if (vol == NULL) + if (!(vol = virStorageVolCreateXML(pool, args->xml, args->flags))) goto cleanup; make_nonnull_storage_vol(&ret->vol, vol); @@ -6276,17 +6141,14 @@ remoteDispatchStorageVolCreateXmlFrom(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; - clonevol = get_nonnull_storage_vol(conn, args->clonevol); - if (clonevol == NULL) + if (!(clonevol = get_nonnull_storage_vol(conn, args->clonevol))) goto cleanup; - newvol = virStorageVolCreateXMLFrom(pool, args->xml, clonevol, - args->flags); - if (newvol == NULL) + if (!(newvol = virStorageVolCreateXMLFrom(pool, args->xml, clonevol, + args->flags))) goto cleanup; make_nonnull_storage_vol(&ret->vol, newvol); @@ -6321,8 +6183,7 @@ remoteDispatchStorageVolDelete(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - vol = get_nonnull_storage_vol(conn, args->vol); - if (vol == NULL) + if (!(vol = get_nonnull_storage_vol(conn, args->vol))) goto cleanup; if (virStorageVolDelete(vol, args->flags) < 0) @@ -6355,8 +6216,7 @@ remoteDispatchStorageVolWipe(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - vol = get_nonnull_storage_vol(conn, args->vol); - if (vol == NULL) + if (!(vol = get_nonnull_storage_vol(conn, args->vol))) goto cleanup; if (virStorageVolWipe(vol, args->flags) < 0) @@ -6390,8 +6250,7 @@ remoteDispatchStorageVolGetInfo(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - vol = get_nonnull_storage_vol(conn, args->vol); - if (vol == NULL) + if (!(vol = get_nonnull_storage_vol(conn, args->vol))) goto cleanup; if (virStorageVolGetInfo(vol, &info) < 0) @@ -6428,13 +6287,11 @@ remoteDispatchStorageVolDumpXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - vol = get_nonnull_storage_vol(conn, args->vol); - if (vol == NULL) + if (!(vol = get_nonnull_storage_vol(conn, args->vol))) goto cleanup; /* remoteDispatchClientRequest will free this. */ - ret->xml = virStorageVolGetXMLDesc(vol, args->flags); - if (!ret->xml) + if (!(ret->xml = virStorageVolGetXMLDesc(vol, args->flags))) goto cleanup; rv = 0; @@ -6465,13 +6322,11 @@ remoteDispatchStorageVolGetPath(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - vol = get_nonnull_storage_vol(conn, args->vol); - if (vol == NULL) + if (!(vol = get_nonnull_storage_vol(conn, args->vol))) goto cleanup; /* remoteDispatchClientRequest will free this. */ - ret->name = virStorageVolGetPath(vol); - if (!ret->name) + if (!(ret->name = virStorageVolGetPath(vol))) goto cleanup; rv = 0; @@ -6503,12 +6358,10 @@ remoteDispatchStorageVolLookupByName(struct qemud_server *server ATTRIBUTE_UNUSE goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; - vol = virStorageVolLookupByName(pool, args->name); - if (vol == NULL) + if (!(vol = virStorageVolLookupByName(pool, args->name))) goto cleanup; make_nonnull_storage_vol(&ret->vol, vol); @@ -6542,8 +6395,7 @@ remoteDispatchStorageVolLookupByKey(struct qemud_server *server ATTRIBUTE_UNUSED goto cleanup; } - vol = virStorageVolLookupByKey(conn, args->key); - if (vol == NULL) + if (!(vol = virStorageVolLookupByKey(conn, args->key))) goto cleanup; make_nonnull_storage_vol(&ret->vol, vol); @@ -6576,8 +6428,7 @@ remoteDispatchStorageVolLookupByPath(struct qemud_server *server ATTRIBUTE_UNUSE goto cleanup; } - vol = virStorageVolLookupByPath(conn, args->path); - if (vol == NULL) + if (!(vol = virStorageVolLookupByPath(conn, args->path))) goto cleanup; make_nonnull_storage_vol(&ret->vol, vol); @@ -6613,10 +6464,9 @@ remoteDispatchNodeNumOfDevices(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ret->num = virNodeNumOfDevices(conn, - args->cap ? *args->cap : NULL, - args->flags); - if (ret->num < 0) + if ((ret->num = virNodeNumOfDevices(conn, + args->cap ? *args->cap : NULL, + args->flags)) < 0) goto cleanup; rv = 0; @@ -6692,8 +6542,7 @@ remoteDispatchNodeDeviceLookupByName(struct qemud_server *server ATTRIBUTE_UNUSE goto cleanup; } - dev = virNodeDeviceLookupByName(conn, args->name); - if (dev == NULL) + if (!(dev = virNodeDeviceLookupByName(conn, args->name))) goto cleanup; make_nonnull_node_device(&ret->dev, dev); @@ -6726,13 +6575,11 @@ remoteDispatchNodeDeviceDumpXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dev = virNodeDeviceLookupByName(conn, args->name); - if (dev == NULL) + if (!(dev = virNodeDeviceLookupByName(conn, args->name))) goto cleanup; /* remoteDispatchClientRequest will free this. */ - ret->xml = virNodeDeviceGetXMLDesc(dev, args->flags); - if (!ret->xml) + if (!(ret->xml = virNodeDeviceGetXMLDesc(dev, args->flags))) goto cleanup; rv = 0; @@ -6764,8 +6611,7 @@ remoteDispatchNodeDeviceGetParent(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dev = virNodeDeviceLookupByName(conn, args->name); - if (dev == NULL) + if (!(dev = virNodeDeviceLookupByName(conn, args->name))) goto cleanup; parent = virNodeDeviceGetParent(dev); @@ -6815,12 +6661,10 @@ remoteDispatchNodeDeviceNumOfCaps(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dev = virNodeDeviceLookupByName(conn, args->name); - if (dev == NULL) + if (!(dev = virNodeDeviceLookupByName(conn, args->name))) goto cleanup; - ret->num = virNodeDeviceNumOfCaps(dev); - if (ret->num < 0) + if ((ret->num = virNodeDeviceNumOfCaps(dev)) < 0) goto cleanup; rv = 0; @@ -6852,8 +6696,7 @@ remoteDispatchNodeDeviceListCaps(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dev = virNodeDeviceLookupByName(conn, args->name); - if (dev == NULL) + if (!(dev = virNodeDeviceLookupByName(conn, args->name))) goto cleanup; if (args->maxnames > REMOTE_NODE_DEVICE_NAME_LIST_MAX) { @@ -6904,8 +6747,7 @@ remoteDispatchNodeDeviceDettach(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dev = virNodeDeviceLookupByName(conn, args->name); - if (dev == NULL) + if (!(dev = virNodeDeviceLookupByName(conn, args->name))) goto cleanup; if (virNodeDeviceDettach(dev) < 0) @@ -6939,8 +6781,7 @@ remoteDispatchNodeDeviceReAttach(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dev = virNodeDeviceLookupByName(conn, args->name); - if (dev == NULL) + if (!(dev = virNodeDeviceLookupByName(conn, args->name))) goto cleanup; if (virNodeDeviceReAttach(dev) < 0) @@ -6974,8 +6815,7 @@ remoteDispatchNodeDeviceReset(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dev = virNodeDeviceLookupByName(conn, args->name); - if (dev == NULL) + if (!(dev = virNodeDeviceLookupByName(conn, args->name))) goto cleanup; if (virNodeDeviceReset(dev) < 0) @@ -7009,8 +6849,7 @@ remoteDispatchNodeDeviceCreateXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dev = virNodeDeviceCreateXML(conn, args->xml_desc, args->flags); - if (dev == NULL) + if (!(dev = virNodeDeviceCreateXML(conn, args->xml_desc, args->flags))) goto cleanup; make_nonnull_node_device(&ret->dev, dev); @@ -7043,8 +6882,7 @@ remoteDispatchNodeDeviceDestroy(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dev = virNodeDeviceLookupByName(conn, args->name); - if (dev == NULL) + if (!(dev = virNodeDeviceLookupByName(conn, args->name))) goto cleanup; if (virNodeDeviceDestroy(dev) < 0) @@ -7077,12 +6915,10 @@ static int remoteDispatchStorageVolUpload(struct qemud_server *server ATTRIBUTE_ goto cleanup; } - vol = get_nonnull_storage_vol(conn, args->vol); - if (vol == NULL) + if (!(vol = get_nonnull_storage_vol(conn, args->vol))) goto cleanup; - stream = remoteCreateClientStream(conn, hdr); - if (!stream) + if (!(stream = remoteCreateClientStream(conn, hdr))) goto cleanup; if (virStorageVolUpload(vol, stream->st, @@ -7124,12 +6960,10 @@ static int remoteDispatchStorageVolDownload(struct qemud_server *server ATTRIBUT goto cleanup; } - vol = get_nonnull_storage_vol(conn, args->vol); - if (vol == NULL) + if (!(vol = get_nonnull_storage_vol(conn, args->vol))) goto cleanup; - stream = remoteCreateClientStream(conn, hdr); - if (!stream) + if (!(stream = remoteCreateClientStream(conn, hdr))) goto cleanup; if (virStorageVolDownload(vol, stream->st, @@ -7311,8 +7145,7 @@ remoteDispatchNumOfSecrets(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ret->num = virConnectNumOfSecrets(conn); - if (ret->num < 0) + if ((ret->num = virConnectNumOfSecrets(conn)) < 0) goto cleanup; rv = 0; @@ -7384,8 +7217,7 @@ remoteDispatchSecretDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - secret = virSecretDefineXML(conn, args->xml, args->flags); - if (secret == NULL) + if (!(secret = virSecretDefineXML(conn, args->xml, args->flags))) goto cleanup; make_nonnull_secret(&ret->secret, secret); @@ -7418,12 +7250,10 @@ remoteDispatchSecretGetValue(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - secret = get_nonnull_secret(conn, args->secret); - if (secret == NULL) + if (!(secret = get_nonnull_secret(conn, args->secret))) goto cleanup; - value = virSecretGetValue(secret, &value_size, args->flags); - if (value == NULL) + if (!(value = virSecretGetValue(secret, &value_size, args->flags))) goto cleanup; ret->value.value_len = value_size; @@ -7456,11 +7286,9 @@ remoteDispatchSecretGetXmlDesc(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - secret = get_nonnull_secret(conn, args->secret); - if (secret == NULL) + if (!(secret = get_nonnull_secret(conn, args->secret))) goto cleanup; - ret->xml = virSecretGetXMLDesc(secret, args->flags); - if (ret->xml == NULL) + if (!(ret->xml = virSecretGetXMLDesc(secret, args->flags))) goto cleanup; rv = 0; @@ -7490,8 +7318,7 @@ remoteDispatchSecretLookupByUuid(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - secret = virSecretLookupByUUID(conn, (unsigned char *)args->uuid); - if (secret == NULL) + if (!(secret = virSecretLookupByUUID(conn, (unsigned char *)args->uuid))) goto cleanup; make_nonnull_secret(&ret->secret, secret); @@ -7523,8 +7350,7 @@ remoteDispatchSecretSetValue(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - secret = get_nonnull_secret(conn, args->secret); - if (secret == NULL) + if (!(secret = get_nonnull_secret(conn, args->secret))) goto cleanup; if (virSecretSetValue(secret, (const unsigned char *)args->value.value_val, args->value.value_len, args->flags) < 0) @@ -7557,8 +7383,7 @@ remoteDispatchSecretUndefine(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - secret = get_nonnull_secret(conn, args->secret); - if (secret == NULL) + if (!(secret = get_nonnull_secret(conn, args->secret))) goto cleanup; if (virSecretUndefine(secret) < 0) goto cleanup; @@ -7590,8 +7415,7 @@ remoteDispatchSecretLookupByUsage(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - secret = virSecretLookupByUsage(conn, args->usageType, args->usageID); - if (secret == NULL) + if (!(secret = virSecretLookupByUsage(conn, args->usageType, args->usageID))) goto cleanup; make_nonnull_secret(&ret->secret, secret); @@ -7623,13 +7447,10 @@ static int remoteDispatchDomainIsActive(struct qemud_server *server ATTRIBUTE_UN goto cleanup; } - domain = get_nonnull_domain(conn, args->dom); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->dom))) goto cleanup; - ret->active = virDomainIsActive(domain); - - if (ret->active < 0) + if ((ret->active = virDomainIsActive(domain)) < 0) goto cleanup; rv = 0; @@ -7658,13 +7479,10 @@ static int remoteDispatchDomainIsPersistent(struct qemud_server *server ATTRIBUT goto cleanup; } - domain = get_nonnull_domain(conn, args->dom); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->dom))) goto cleanup; - ret->persistent = virDomainIsPersistent(domain); - - if (ret->persistent < 0) + if ((ret->persistent = virDomainIsPersistent(domain)) < 0) goto cleanup; rv = 0; @@ -7693,13 +7511,10 @@ static int remoteDispatchDomainIsUpdated(struct qemud_server *server ATTRIBUTE_U goto cleanup; } - domain = get_nonnull_domain(conn, args->dom); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->dom))) goto cleanup; - ret->updated = virDomainIsUpdated(domain); - - if (ret->updated < 0) + if ((ret->updated = virDomainIsUpdated(domain)) < 0) goto cleanup; rv = 0; @@ -7728,13 +7543,10 @@ static int remoteDispatchInterfaceIsActive(struct qemud_server *server ATTRIBUTE goto cleanup; } - iface = get_nonnull_interface(conn, args->iface); - if (iface == NULL) + if (!(iface = get_nonnull_interface(conn, args->iface))) goto cleanup; - ret->active = virInterfaceIsActive(iface); - - if (ret->active < 0) + if ((ret->active = virInterfaceIsActive(iface)) < 0) goto cleanup; rv = 0; @@ -7763,13 +7575,10 @@ static int remoteDispatchNetworkIsActive(struct qemud_server *server ATTRIBUTE_U goto cleanup; } - network = get_nonnull_network(conn, args->net); - if (network == NULL) + if (!(network = get_nonnull_network(conn, args->net))) goto cleanup; - ret->active = virNetworkIsActive(network); - - if (ret->active < 0) + if ((ret->active = virNetworkIsActive(network)) < 0) goto cleanup; rv = 0; @@ -7798,13 +7607,10 @@ static int remoteDispatchNetworkIsPersistent(struct qemud_server *server ATTRIBU goto cleanup; } - network = get_nonnull_network(conn, args->net); - if (network == NULL) + if (!(network = get_nonnull_network(conn, args->net))) goto cleanup; - ret->persistent = virNetworkIsPersistent(network); - - if (ret->persistent < 0) + if ((ret->persistent = virNetworkIsPersistent(network)) < 0) goto cleanup; rv = 0; @@ -7833,13 +7639,10 @@ static int remoteDispatchStoragePoolIsActive(struct qemud_server *server ATTRIBU goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; - ret->active = virStoragePoolIsActive(pool); - - if (ret->active < 0) + if ((ret->active = virStoragePoolIsActive(pool)) < 0) goto cleanup; rv = 0; @@ -7868,13 +7671,10 @@ static int remoteDispatchStoragePoolIsPersistent(struct qemud_server *server ATT goto cleanup; } - pool = get_nonnull_storage_pool(conn, args->pool); - if (pool == NULL) + if (!(pool = get_nonnull_storage_pool(conn, args->pool))) goto cleanup; - ret->persistent = virStoragePoolIsPersistent(pool); - - if (ret->persistent < 0) + if ((ret->persistent = virStoragePoolIsPersistent(pool)) < 0) goto cleanup; rv = 0; @@ -7903,9 +7703,7 @@ static int remoteDispatchIsSecure(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ret->secure = virConnectIsSecure(conn); - - if (ret->secure < 0) + if ((ret->secure = virConnectIsSecure(conn)) < 0) goto cleanup; rv = 0; @@ -7934,8 +7732,7 @@ remoteDispatchCpuCompare(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - result = virConnectCompareCPU(conn, args->xml, args->flags); - if (result == VIR_CPU_COMPARE_ERROR) + if ((result = virConnectCompareCPU(conn, args->xml, args->flags)) == VIR_CPU_COMPARE_ERROR) goto cleanup; ret->result = result; @@ -7965,11 +7762,10 @@ remoteDispatchCpuBaseline(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - cpu = virConnectBaselineCPU(conn, - (const char **) args->xmlCPUs.xmlCPUs_val, - args->xmlCPUs.xmlCPUs_len, - args->flags); - if (cpu == NULL) + if (!(cpu = virConnectBaselineCPU(conn, + (const char **) args->xmlCPUs.xmlCPUs_val, + args->xmlCPUs.xmlCPUs_len, + args->flags))) goto cleanup; ret->cpu = cpu; @@ -8001,8 +7797,7 @@ remoteDispatchDomainGetJobInfo(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainGetJobInfo(dom, &info) < 0) @@ -8049,8 +7844,7 @@ remoteDispatchDomainAbortJob(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainAbortJob(dom) < 0) @@ -8084,8 +7878,7 @@ remoteDispatchDomainMigrateSetMaxDowntime(struct qemud_server *server ATTRIBUTE_ goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainMigrateSetMaxDowntime(dom, args->downtime, args->flags) < 0) @@ -8118,8 +7911,7 @@ remoteDispatchDomainMigrateSetMaxSpeed(struct qemud_server *server ATTRIBUTE_UNU goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainMigrateSetMaxSpeed(dom, args->bandwidth, args->flags) < 0) @@ -8153,12 +7945,10 @@ remoteDispatchDomainSnapshotCreateXml(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - domain = get_nonnull_domain(conn, args->domain); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->domain))) goto cleanup; - snapshot = virDomainSnapshotCreateXML(domain, args->xml_desc, args->flags); - if (snapshot == NULL) + if (!(snapshot = virDomainSnapshotCreateXML(domain, args->xml_desc, args->flags))) goto cleanup; make_nonnull_domain_snapshot(&ret->snap, snapshot); @@ -8193,17 +7983,14 @@ remoteDispatchDomainSnapshotDumpXml(struct qemud_server *server ATTRIBUTE_UNUSED goto cleanup; } - domain = get_nonnull_domain(conn, args->snap.domain); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->snap.domain))) goto cleanup; - snapshot = get_nonnull_domain_snapshot(domain, args->snap); - if (snapshot == NULL) + if (!(snapshot = get_nonnull_domain_snapshot(domain, args->snap))) goto cleanup; /* remoteDispatchClientRequest will free this. */ - ret->xml = virDomainSnapshotGetXMLDesc(snapshot, args->flags); - if (!ret->xml) + if (!(ret->xml = virDomainSnapshotGetXMLDesc(snapshot, args->flags))) goto cleanup; rv = 0; @@ -8235,12 +8022,10 @@ remoteDispatchDomainSnapshotNum(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - domain = get_nonnull_domain(conn, args->domain); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->domain))) goto cleanup; - ret->num = virDomainSnapshotNum(domain, args->flags); - if (ret->num < 0) + if ((ret->num = virDomainSnapshotNum(domain, args->flags)) < 0) goto cleanup; rv = 0; @@ -8277,8 +8062,7 @@ remoteDispatchDomainSnapshotListNames(struct qemud_server *server ATTRIBUTE_UNUS goto cleanup; } - domain = get_nonnull_domain(conn, args->domain); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->domain))) goto cleanup; /* Allocate return buffer. */ @@ -8325,12 +8109,10 @@ remoteDispatchDomainSnapshotLookupByName(struct qemud_server *server ATTRIBUTE_U goto cleanup; } - domain = get_nonnull_domain(conn, args->domain); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->domain))) goto cleanup; - snapshot = virDomainSnapshotLookupByName(domain, args->name, args->flags); - if (snapshot == NULL) + if (!(snapshot = virDomainSnapshotLookupByName(domain, args->name, args->flags))) goto cleanup; make_nonnull_domain_snapshot(&ret->snap, snapshot); @@ -8365,8 +8147,7 @@ remoteDispatchDomainHasCurrentSnapshot(struct qemud_server *server ATTRIBUTE_UNU goto cleanup; } - domain = get_nonnull_domain(conn, args->domain); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->domain))) goto cleanup; result = virDomainHasCurrentSnapshot(domain, args->flags); @@ -8403,12 +8184,10 @@ remoteDispatchDomainSnapshotCurrent(struct qemud_server *server ATTRIBUTE_UNUSED goto cleanup; } - domain = get_nonnull_domain(conn, args->domain); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->domain))) goto cleanup; - snapshot = virDomainSnapshotCurrent(domain, args->flags); - if (snapshot == NULL) + if (!(snapshot = virDomainSnapshotCurrent(domain, args->flags))) goto cleanup; make_nonnull_domain_snapshot(&ret->snap, snapshot); @@ -8443,12 +8222,10 @@ remoteDispatchDomainRevertToSnapshot(struct qemud_server *server ATTRIBUTE_UNUSE goto cleanup; } - domain = get_nonnull_domain(conn, args->snap.domain); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->snap.domain))) goto cleanup; - snapshot = get_nonnull_domain_snapshot(domain, args->snap); - if (snapshot == NULL) + if (!(snapshot = get_nonnull_domain_snapshot(domain, args->snap))) goto cleanup; if (virDomainRevertToSnapshot(snapshot, args->flags) < 0) @@ -8484,12 +8261,10 @@ remoteDispatchDomainSnapshotDelete(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - domain = get_nonnull_domain(conn, args->snap.domain); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->snap.domain))) goto cleanup; - snapshot = get_nonnull_domain_snapshot(domain, args->snap); - if (snapshot == NULL) + if (!(snapshot = get_nonnull_domain_snapshot(domain, args->snap))) goto cleanup; if (virDomainSnapshotDelete(snapshot, args->flags) < 0) @@ -8577,8 +8352,7 @@ remoteDispatchDomainEventsDeregisterAny(struct qemud_server *server ATTRIBUTE_UN goto cleanup; } - callbackID = client->domainEventCallbackID[args->eventID]; - if (callbackID < 0) { + if ((callbackID = client->domainEventCallbackID[args->eventID]) < 0) { virNetError(VIR_ERR_INTERNAL_ERROR, _("domain event %d not registered"), args->eventID); goto cleanup; } @@ -8614,8 +8388,7 @@ remoteDispatchNwfilterLookupByName(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - nwfilter = virNWFilterLookupByName(conn, args->name); - if (nwfilter == NULL) + if (!(nwfilter = virNWFilterLookupByName(conn, args->name))) goto cleanup; make_nonnull_nwfilter(&ret->nwfilter, nwfilter); @@ -8647,8 +8420,7 @@ remoteDispatchNwfilterLookupByUuid(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - nwfilter = virNWFilterLookupByUUID(conn, (unsigned char *) args->uuid); - if (nwfilter == NULL) + if (!(nwfilter = virNWFilterLookupByUUID(conn, (unsigned char *) args->uuid))) goto cleanup; make_nonnull_nwfilter(&ret->nwfilter, nwfilter); @@ -8681,8 +8453,7 @@ remoteDispatchNwfilterDefineXml(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - nwfilter = virNWFilterDefineXML(conn, args->xml); - if (nwfilter == NULL) + if (!(nwfilter = virNWFilterDefineXML(conn, args->xml))) goto cleanup; make_nonnull_nwfilter(&ret->nwfilter, nwfilter); @@ -8715,8 +8486,7 @@ remoteDispatchNwfilterUndefine(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - nwfilter = get_nonnull_nwfilter(conn, args->nwfilter); - if (nwfilter == NULL) + if (!(nwfilter = get_nonnull_nwfilter(conn, args->nwfilter))) goto cleanup; if (virNWFilterUndefine(nwfilter) < 0) @@ -8795,13 +8565,11 @@ remoteDispatchNwfilterGetXmlDesc(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - nwfilter = get_nonnull_nwfilter(conn, args->nwfilter); - if (nwfilter == NULL) + if (!(nwfilter = get_nonnull_nwfilter(conn, args->nwfilter))) goto cleanup; /* remoteDispatchClientRequest will free this. */ - ret->xml = virNWFilterGetXMLDesc(nwfilter, args->flags); - if (!ret->xml) + if (!(ret->xml = virNWFilterGetXMLDesc(nwfilter, args->flags))) goto cleanup; rv = 0; @@ -8831,8 +8599,7 @@ remoteDispatchNumOfNwfilters(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - ret->num = virConnectNumOfNWFilters(conn); - if (ret->num < 0) + if ((ret->num = virConnectNumOfNWFilters(conn)) < 0) goto cleanup; rv = 0; @@ -8862,8 +8629,7 @@ remoteDispatchDomainGetBlockInfo(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->dom); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->dom))) goto cleanup; if (virDomainGetBlockInfo(dom, args->path, &info, args->flags) < 0) @@ -8900,8 +8666,7 @@ qemuDispatchMonitorCommand(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - domain = get_nonnull_domain(conn, args->domain); - if (domain == NULL) + if (!(domain = get_nonnull_domain(conn, args->domain))) goto cleanup; if (virDomainQemuMonitorCommand(domain, args->cmd, &ret->result, @@ -8928,7 +8693,6 @@ remoteDispatchDomainOpenConsole(struct qemud_server *server ATTRIBUTE_UNUSED, remote_domain_open_console_args *args, void *ret ATTRIBUTE_UNUSED) { - int r; struct qemud_client_stream *stream = NULL; virDomainPtr dom = NULL; int rv = -1; @@ -8938,21 +8702,18 @@ remoteDispatchDomainOpenConsole(struct qemud_server *server ATTRIBUTE_UNUSED, goto cleanup; } - dom = get_nonnull_domain(conn, args->domain); - if (dom == NULL) + if (!(dom = get_nonnull_domain(conn, args->domain))) goto cleanup; - stream = remoteCreateClientStream(conn, hdr); - if (!stream) { + if (!(stream = remoteCreateClientStream(conn, hdr))) { virReportOOMError(); goto cleanup; } - r = virDomainOpenConsole(dom, + if (virDomainOpenConsole(dom, args->devname ? *args->devname : NULL, stream->st, - args->flags); - if (r < 0) + args->flags) < 0) goto cleanup; if (remoteAddClientStream(client, stream, 1) < 0) -- 1.7.4.2
On 04/13/2011 12:14 PM, Daniel P. Berrange wrote:
Replace cases of
type = virConnectGetType(conn); if (type == NULL) goto cleanup;
With
if (!(type = virConnectGetType(conn))) goto cleanup;
* daemon/remote.c: Write error checks in compat form --- daemon/remote.c | 795 +++++++++++++++++++------------------------------------ 1 files changed, 278 insertions(+), 517 deletions(-)
Not quite a 1:2 ration in + to -, but still fairly mechanical. Didn't spot anything obviously wrong. I did see at least one variable rename (from err to int) snuck in; but that's okay. ACK. -- Eric Blake eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org
participants (2)
-
Daniel P. Berrange -
Eric Blake