--- tests/commandhelper.c | 2 +- tests/commandtest.c | 34 ++++++++++----------- tests/cputest.c | 18 +++++------ tests/domainconftest.c | 2 +- tests/domainsnapshotxml2xmltest.c | 6 ++-- tests/esxutilstest.c | 4 +-- tests/fchosttest.c | 8 ++--- tests/fdstreamtest.c | 4 +-- tests/jsontest.c | 4 +-- tests/libvirtdconftest.c | 4 +-- tests/lxcconf2xmltest.c | 4 +-- tests/lxcxml2xmltest.c | 2 +- tests/metadatatest.c | 4 +-- tests/networkxml2conftest.c | 2 +- tests/networkxml2xmltest.c | 2 +- tests/networkxml2xmlupdatetest.c | 6 ++-- tests/nodeinfotest.c | 8 ++--- tests/nwfilterxml2xmltest.c | 2 +- tests/objecteventtest.c | 16 +++++----- tests/openvzutilstest.c | 4 +-- tests/qemuagenttest.c | 18 +++++------ tests/qemuargv2xmltest.c | 2 +- tests/qemucapabilitiestest.c | 6 ++-- tests/qemuhelptest.c | 2 +- tests/qemuhotplugtest.c | 4 +-- tests/qemumonitorjsontest.c | 62 +++++++++++++++++++------------------- tests/qemumonitortestutils.c | 24 +++++++-------- tests/qemuxml2argvtest.c | 12 ++++---- tests/qemuxml2xmltest.c | 2 +- tests/qemuxmlnstest.c | 2 +- tests/secretxml2xmltest.c | 4 +-- tests/securityselinuxlabeltest.c | 6 ++-- tests/securityselinuxtest.c | 4 +-- tests/sexpr2xmltest.c | 2 +- tests/storagebackendsheepdogtest.c | 6 ++-- tests/storagepoolxml2xmltest.c | 2 +- tests/storagevolxml2argvtest.c | 4 +-- tests/storagevolxml2xmltest.c | 2 +- tests/sysinfotest.c | 4 +-- tests/test_conf.c | 2 +- tests/testutils.c | 2 +- tests/testutilslxc.c | 2 +- tests/testutilsqemu.c | 12 ++++---- tests/testutilsxen.c | 2 +- tests/virauthconfigtest.c | 2 +- tests/virbitmaptest.c | 18 +++++------ tests/virbuftest.c | 4 +-- tests/vircapstest.c | 4 +-- tests/vircgroupmock.c | 4 +-- tests/vircgrouptest.c | 24 +++++++-------- tests/virdbustest.c | 12 ++++---- tests/virendiantest.c | 4 +-- tests/virhashtest.c | 20 ++++++------ tests/virhostdevtest.c | 20 ++++++------ tests/viridentitytest.c | 6 ++-- tests/viriscsitest.c | 4 +-- tests/virkeycodetest.c | 4 +-- tests/virkeyfiletest.c | 2 +- tests/virkmodtest.c | 8 ++--- tests/virlockspacetest.c | 14 ++++----- tests/virnetdevbandwidthtest.c | 2 +- tests/virnetmessagetest.c | 10 +++--- tests/virnetsockettest.c | 14 ++++----- tests/virnettlscontexttest.c | 2 +- tests/virnettlssessiontest.c | 2 +- tests/virpcimock.c | 18 +++++------ tests/virpcitest.c | 22 +++++++------- tests/virportallocatortest.c | 4 +-- tests/virscsitest.c | 6 ++-- tests/virshtest.c | 2 +- tests/virstoragetest.c | 6 ++-- tests/virstringtest.c | 8 ++--- tests/viruritest.c | 2 +- tests/virusbmock.c | 2 +- tests/virusbtest.c | 4 +-- tests/vmwarevertest.c | 2 +- tests/vmx2xmltest.c | 8 ++--- tests/xmconfigtest.c | 2 +- tests/xml2sexprtest.c | 2 +- tests/xml2vmxtest.c | 8 ++--- 80 files changed, 300 insertions(+), 300 deletions(-) diff --git a/tests/commandhelper.c b/tests/commandhelper.c index 32b8512..796b89d 100644 --- a/tests/commandhelper.c +++ b/tests/commandhelper.c @@ -156,7 +156,7 @@ int main(int argc, char **argv) { return EXIT_SUCCESS; -error: + error: return EXIT_FAILURE; } diff --git a/tests/commandtest.c b/tests/commandtest.c index 0cba8d9..c5c3a9a 100644 --- a/tests/commandtest.c +++ b/tests/commandtest.c @@ -93,7 +93,7 @@ static int checkoutput(const char *testname) ret = 0; -cleanup: + cleanup: if (actualname) unlink(actualname); VIR_FREE(actuallog); @@ -123,7 +123,7 @@ static int test0(const void *unused ATTRIBUTE_UNUSED) virResetLastError(); ret = 0; -cleanup: + cleanup: virCommandFree(cmd); return ret; } @@ -152,7 +152,7 @@ static int test1(const void *unused ATTRIBUTE_UNUSED) goto cleanup; ret = 0; -cleanup: + cleanup: virCommandFree(cmd); return ret; } @@ -221,7 +221,7 @@ static int test3(const void *unused ATTRIBUTE_UNUSED) ret = checkoutput("test3"); -cleanup: + cleanup: virCommandFree(cmd); /* coverity[double_close] */ VIR_FORCE_CLOSE(newfd1); @@ -263,7 +263,7 @@ static int test4(const void *unused ATTRIBUTE_UNUSED) ret = checkoutput("test4"); -cleanup: + cleanup: virCommandFree(cmd); if (pidfile) unlink(pidfile); @@ -512,7 +512,7 @@ static int test13(const void *unused ATTRIBUTE_UNUSED) ret = checkoutput("test13"); -cleanup: + cleanup: virCommandFree(cmd); VIR_FREE(outactual); return ret; @@ -584,7 +584,7 @@ static int test14(const void *unused ATTRIBUTE_UNUSED) ret = checkoutput("test14"); -cleanup: + cleanup: virCommandFree(cmd); VIR_FREE(outactual); VIR_FREE(erractual); @@ -615,7 +615,7 @@ static int test15(const void *unused ATTRIBUTE_UNUSED) ret = checkoutput("test15"); -cleanup: + cleanup: VIR_FREE(cwd); virCommandFree(cmd); @@ -661,7 +661,7 @@ static int test16(const void *unused ATTRIBUTE_UNUSED) ret = checkoutput("test16"); -cleanup: + cleanup: virCommandFree(cmd); VIR_FORCE_CLOSE(fd); VIR_FREE(outactual); @@ -718,7 +718,7 @@ static int test17(const void *unused ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virCommandFree(cmd); VIR_FREE(outbuf); VIR_FREE(errbuf); @@ -766,7 +766,7 @@ static int test18(const void *unused ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virCommandFree(cmd); if (pidfile) unlink(pidfile); @@ -806,7 +806,7 @@ static int test19(const void *unused ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virCommandFree(cmd); return ret; } @@ -841,7 +841,7 @@ static int test20(const void *unused ATTRIBUTE_UNUSED) } ret = checkoutput("test20"); -cleanup: + cleanup: virCommandFree(cmd); VIR_FREE(buf); return ret; @@ -900,7 +900,7 @@ static int test21(const void *unused ATTRIBUTE_UNUSED) } ret = checkoutput("test21"); -cleanup: + cleanup: VIR_FREE(outbuf); VIR_FREE(errbuf); virCommandFree(cmd); @@ -957,7 +957,7 @@ test22(const void *unused ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virCommandFree(cmd); return ret; } @@ -1016,7 +1016,7 @@ test23(const void *unused ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: return ret; } @@ -1178,7 +1178,7 @@ mymain(void) } virMutexUnlock(&test->lock); -cleanup: + cleanup: if (test->running) virThreadJoin(&test->thread); diff --git a/tests/cputest.c b/tests/cputest.c index c03538a..8903f82 100644 --- a/tests/cputest.c +++ b/tests/cputest.c @@ -96,7 +96,7 @@ cpuTestLoadXML(const char *arch, const char *name) cpu = virCPUDefParseXML(ctxt->node, ctxt, VIR_CPU_TYPE_AUTO); -cleanup: + cleanup: xmlXPathFreeContext(ctxt); xmlFreeDoc(doc); VIR_FREE(xml); @@ -136,14 +136,14 @@ cpuTestLoadMultiXML(const char *arch, *count = n; -cleanup: + cleanup: VIR_FREE(xml); VIR_FREE(nodes); xmlXPathFreeContext(ctxt); xmlFreeDoc(doc); return cpus; -cleanup_cpus: + cleanup_cpus: for (i = 0; i < n; i++) virCPUDefFree(cpus[i]); VIR_FREE(cpus); @@ -181,7 +181,7 @@ cpuTestCompareXML(const char *arch, ret = 0; -cleanup: + cleanup: VIR_FREE(xml); VIR_FREE(expected); VIR_FREE(actual); @@ -247,7 +247,7 @@ cpuTestCompare(const void *arg) ret = 0; -cleanup: + cleanup: virCPUDefFree(host); virCPUDefFree(cpu); return ret; @@ -307,7 +307,7 @@ cpuTestGuestData(const void *arg) ret = cpuTestCompareXML(data->arch, guest, result, 0); -cleanup: + cleanup: VIR_FREE(result); cpuDataFree(guestData); virCPUDefFree(host); @@ -374,7 +374,7 @@ cpuTestBaseline(const void *arg) ret = 0; -cleanup: + cleanup: if (cpus) { for (i = 0; i < ncpus; i++) virCPUDefFree(cpus[i]); @@ -408,7 +408,7 @@ cpuTestUpdate(const void *arg) ret = cpuTestCompareXML(data->arch, cpu, result, VIR_DOMAIN_XML_UPDATE_CPU); -cleanup: + cleanup: virCPUDefFree(host); virCPUDefFree(cpu); VIR_FREE(result); @@ -449,7 +449,7 @@ cpuTestHasFeature(const void *arg) ret = 0; -cleanup: + cleanup: cpuDataFree(hostData); virCPUDefFree(host); return ret; diff --git a/tests/domainconftest.c b/tests/domainconftest.c index c87da5d..3d6ebe1 100644 --- a/tests/domainconftest.c +++ b/tests/domainconftest.c @@ -78,7 +78,7 @@ static int testGetFilesystem(const void *opaque) ret = 0; -cleanup: + cleanup: virDomainDefFree(def); VIR_FREE(xmlData); VIR_FREE(filename); diff --git a/tests/domainsnapshotxml2xmltest.c b/tests/domainsnapshotxml2xmltest.c index 82925c5..6d20181 100644 --- a/tests/domainsnapshotxml2xmltest.c +++ b/tests/domainsnapshotxml2xmltest.c @@ -65,7 +65,7 @@ testFilterXML(char *xml) ret = virBufferContentAndReset(&buf); -cleanup: + cleanup: virBufferFreeAndReset(&buf); virStringFreeList(xmlLines); return ret; @@ -123,7 +123,7 @@ testCompareXMLToXMLFiles(const char *inxml, ret = 0; -cleanup: + cleanup: VIR_FREE(inXmlData); VIR_FREE(outXmlData); VIR_FREE(actual); @@ -226,7 +226,7 @@ mymain(void) DO_TEST_IN("description_only", NULL); DO_TEST_IN("name_only", NULL); -cleanup: + cleanup: if (testSnapshotXMLVariableLineRegex) regfree(testSnapshotXMLVariableLineRegex); VIR_FREE(testSnapshotXMLVariableLineRegex); diff --git a/tests/esxutilstest.c b/tests/esxutilstest.c index 0576366..41c0807 100644 --- a/tests/esxutilstest.c +++ b/tests/esxutilstest.c @@ -75,14 +75,14 @@ testParseDatastorePath(const void *data ATTRIBUTE_UNUSED) } } - cleanup: + cleanup: VIR_FREE(datastoreName); VIR_FREE(directoryName); VIR_FREE(directoryAndFileName); return result; - failure: + failure: result = -1; goto cleanup; diff --git a/tests/fchosttest.c b/tests/fchosttest.c index b5291f0..87b3a2a 100644 --- a/tests/fchosttest.c +++ b/tests/fchosttest.c @@ -107,7 +107,7 @@ test3(const void *data ATTRIBUTE_UNUSED) goto cleanup; ret = 0; -cleanup: + cleanup: VIR_FREE(wwnn); VIR_FREE(wwpn); VIR_FREE(fabric_wwn); @@ -133,7 +133,7 @@ test4(const void *data ATTRIBUTE_UNUSED) goto cleanup; ret = 0; -cleanup: + cleanup: VIR_FREE(hostname); return ret; } @@ -153,7 +153,7 @@ test5(const void *data ATTRIBUTE_UNUSED) goto cleanup; ret = 0; -cleanup: + cleanup: VIR_FREE(hostname); return ret; } @@ -180,7 +180,7 @@ mymain(void) if (virtTestRun("test5", test5, NULL) < 0) ret = -1; -cleanup: + cleanup: VIR_FREE(fchost_prefix); return ret; } diff --git a/tests/fdstreamtest.c b/tests/fdstreamtest.c index 92e7add..d52b77b 100644 --- a/tests/fdstreamtest.c +++ b/tests/fdstreamtest.c @@ -147,7 +147,7 @@ static int testFDStreamReadCommon(const char *scratchdir, bool blocking) } ret = 0; -cleanup: + cleanup: if (st) virStreamFree(st); VIR_FORCE_CLOSE(fd); @@ -290,7 +290,7 @@ static int testFDStreamWriteCommon(const char *scratchdir, bool blocking) goto cleanup; ret = 0; -cleanup: + cleanup: if (st) virStreamFree(st); VIR_FORCE_CLOSE(fd); diff --git a/tests/jsontest.c b/tests/jsontest.c index 933c41e..a2a42e3 100644 --- a/tests/jsontest.c +++ b/tests/jsontest.c @@ -49,7 +49,7 @@ testJSONFromString(const void *data) ret = 0; -cleanup: + cleanup: virJSONValueFree(json); return ret; } @@ -122,7 +122,7 @@ testJSONAddRemove(const void *data) } ret = 0; -cleanup: + cleanup: virJSONValueFree(json); virJSONValueFree(name); VIR_FREE(result); diff --git a/tests/libvirtdconftest.c b/tests/libvirtdconftest.c index bc22087..13cfa60 100644 --- a/tests/libvirtdconftest.c +++ b/tests/libvirtdconftest.c @@ -153,7 +153,7 @@ testCorrupt(const void *opaque) break; } -cleanup: + cleanup: VIR_FREE(newdata); daemonConfigFree(conf); return ret; @@ -232,7 +232,7 @@ mymain(void) ret = -1; } -cleanup: + cleanup: VIR_FREE(filename); VIR_FREE(filedata); VIR_FREE(params); diff --git a/tests/lxcconf2xmltest.c b/tests/lxcconf2xmltest.c index e799893..6255942 100644 --- a/tests/lxcconf2xmltest.c +++ b/tests/lxcconf2xmltest.c @@ -53,7 +53,7 @@ testCompareXMLToConfigFiles(const char *xml, ret = 0; -fail: + fail: VIR_FREE(expectxml); VIR_FREE(actualxml); VIR_FREE(config); @@ -82,7 +82,7 @@ testCompareXMLToConfigHelper(const void *data) result = testCompareXMLToConfigFiles(xml, config, info->expectError); -cleanup: + cleanup: VIR_FREE(xml); VIR_FREE(config); return result; diff --git a/tests/lxcxml2xmltest.c b/tests/lxcxml2xmltest.c index 4e73d7f..8144989 100644 --- a/tests/lxcxml2xmltest.c +++ b/tests/lxcxml2xmltest.c @@ -101,7 +101,7 @@ testCompareXMLToXMLHelper(const void *data) } ret = 0; -cleanup: + cleanup: VIR_FREE(xml_in); VIR_FREE(xml_out); return ret; diff --git a/tests/metadatatest.c b/tests/metadatatest.c index 0896c52..91fc944 100644 --- a/tests/metadatatest.c +++ b/tests/metadatatest.c @@ -76,7 +76,7 @@ getMetadataFromXML(virDomainPtr dom) ret = virXMLNodeToString(node->doc, node); -cleanup: + cleanup: VIR_FREE(xml); xmlFreeDoc(doc); xmlXPathFreeContext(ctxt); @@ -156,7 +156,7 @@ verifyMetadata(virDomainPtr dom, ret = true; -cleanup: + cleanup: VIR_FREE(metadataXML); VIR_FREE(metadataAPI); diff --git a/tests/networkxml2conftest.c b/tests/networkxml2conftest.c index 9b6b869..4f1d934 100644 --- a/tests/networkxml2conftest.c +++ b/tests/networkxml2conftest.c @@ -93,7 +93,7 @@ testCompareXMLToConfHelper(const void *data) result = testCompareXMLToConfFiles(inxml, outxml, info->caps); -cleanup: + cleanup: VIR_FREE(inxml); VIR_FREE(outxml); diff --git a/tests/networkxml2xmltest.c b/tests/networkxml2xmltest.c index 1977675..c6e0f6f 100644 --- a/tests/networkxml2xmltest.c +++ b/tests/networkxml2xmltest.c @@ -74,7 +74,7 @@ testCompareXMLToXMLHelper(const void *data) result = testCompareXMLToXMLFiles(inxml, outxml, info->flags); -cleanup: + cleanup: VIR_FREE(inxml); VIR_FREE(outxml); diff --git a/tests/networkxml2xmlupdatetest.c b/tests/networkxml2xmlupdatetest.c index 6263dac..f34638a 100644 --- a/tests/networkxml2xmlupdatetest.c +++ b/tests/networkxml2xmlupdatetest.c @@ -56,7 +56,7 @@ testCompareXMLToXMLFiles(const char *netxml, const char *updatexml, ret = 0; -fail: + fail: if (expectFailure) { if (ret == 0) { virReportError(VIR_ERR_INTERNAL_ERROR, "%s", "Failed to fail."); @@ -66,7 +66,7 @@ fail: ret = 0; } } -error: + error: VIR_FREE(netXmlData); VIR_FREE(updateXmlData); VIR_FREE(outXmlData); @@ -109,7 +109,7 @@ testCompareXMLToXMLHelper(const void *data) info->command, info->section, info->parentIndex, info->expectFailure); -cleanup: + cleanup: VIR_FREE(netxml); VIR_FREE(updatexml); VIR_FREE(outxml); diff --git a/tests/nodeinfotest.c b/tests/nodeinfotest.c index 6a175cb..a00a0b7 100644 --- a/tests/nodeinfotest.c +++ b/tests/nodeinfotest.c @@ -73,7 +73,7 @@ linuxTestCompareFiles(const char *cpuinfofile, ret = 0; -fail: + fail: VIR_FREE(expectData); VIR_FREE(actualData); return ret; @@ -157,7 +157,7 @@ linuxCPUStatsCompareFiles(const char *cpustatfile, ret = 0; -fail: + fail: virBufferFreeAndReset(&buf); VIR_FORCE_FCLOSE(cpustat); VIR_FREE(expectData); @@ -193,7 +193,7 @@ linuxTestNodeInfo(const void *data) result = linuxTestCompareFiles(cpuinfo, sysfs_dir, output); -cleanup: + cleanup: VIR_FREE(cpuinfo); VIR_FREE(output); VIR_FREE(sysfs_dir); @@ -223,7 +223,7 @@ linuxTestNodeCPUStats(const void *data) result = linuxCPUStatsCompareFiles(cpustatfile, testData->ncpus, outfile); -fail: + fail: VIR_FREE(cpustatfile); VIR_FREE(outfile); return result; diff --git a/tests/nwfilterxml2xmltest.c b/tests/nwfilterxml2xmltest.c index 5cebb9f..ba1ead1 100644 --- a/tests/nwfilterxml2xmltest.c +++ b/tests/nwfilterxml2xmltest.c @@ -85,7 +85,7 @@ testCompareXMLToXMLHelper(const void *data) result = testCompareXMLToXMLFiles(inxml, outxml, tp->expect_warning); -cleanup: + cleanup: VIR_FREE(inxml); VIR_FREE(outxml); diff --git a/tests/objecteventtest.c b/tests/objecteventtest.c index 76dc378..919f559 100644 --- a/tests/objecteventtest.c +++ b/tests/objecteventtest.c @@ -155,7 +155,7 @@ testDomainCreateXMLOld(const void *data) registered = false; ret = 0; -cleanup: + cleanup: if (registered) virConnectDomainEventDeregister(test->conn, domainLifecycleCb); if (dom) { @@ -196,7 +196,7 @@ testDomainCreateXMLNew(const void *data) id = -1; ret = 0; -cleanup: + cleanup: if (id >= 0) virConnectDomainEventDeregisterAny(test->conn, id); if (dom) { @@ -267,7 +267,7 @@ testDomainCreateXMLMixed(const void *data) id2 = -1; ret = 0; -cleanup: + cleanup: if (id1 >= 0) virConnectDomainEventDeregisterAny(test->conn, id1); if (id2 >= 0) @@ -327,7 +327,7 @@ testDomainDefine(const void *data) } -cleanup: + cleanup: virConnectDomainEventDeregisterAny(test->conn, id); if (dom != NULL) virDomainFree(dom); @@ -388,7 +388,7 @@ testDomainStartStopEvent(const void *data) goto cleanup; ret = 0; -cleanup: + cleanup: virConnectDomainEventDeregisterAny(test->conn, id); virDomainFree(dom); if (dom2) @@ -426,7 +426,7 @@ testNetworkCreateXML(const void *data) goto cleanup; } -cleanup: + cleanup: virConnectNetworkEventDeregisterAny(test->conn, id); virNetworkDestroy(net); @@ -478,7 +478,7 @@ testNetworkDefine(const void *data) } -cleanup: + cleanup: virConnectNetworkEventDeregisterAny(test->conn, id); virNetworkFree(net); @@ -512,7 +512,7 @@ testNetworkStartStopEvent(const void *data) ret = -1; goto cleanup; } -cleanup: + cleanup: virConnectNetworkEventDeregisterAny(test->conn, id); return ret; diff --git a/tests/openvzutilstest.c b/tests/openvzutilstest.c index 3858b42..ed2c098 100644 --- a/tests/openvzutilstest.c +++ b/tests/openvzutilstest.c @@ -64,7 +64,7 @@ testReadConfigParam(const void *data ATTRIBUTE_UNUSED) result = 0; -cleanup: + cleanup: VIR_FREE(conf); VIR_FREE(value); @@ -132,7 +132,7 @@ testReadNetworkConf(const void *data ATTRIBUTE_UNUSED) result = 0; -cleanup: + cleanup: VIR_FREE(actual); virDomainDefFree(def); diff --git a/tests/qemuagenttest.c b/tests/qemuagenttest.c index b5bcbe3..1131d98 100644 --- a/tests/qemuagenttest.c +++ b/tests/qemuagenttest.c @@ -75,7 +75,7 @@ testQemuAgentFSFreeze(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -125,7 +125,7 @@ testQemuAgentFSThaw(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -155,7 +155,7 @@ testQemuAgentFSTrim(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -201,7 +201,7 @@ testQemuAgentSuspend(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -264,7 +264,7 @@ qemuAgentShutdownTestMonitorHandler(qemuMonitorTestPtr test, ret = 0; -cleanup: + cleanup: virJSONValueFree(val); return ret; @@ -349,7 +349,7 @@ testQemuAgentShutdown(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -467,7 +467,7 @@ testQemuAgentCPU(const void *data) ret = 0; -cleanup: + cleanup: VIR_FREE(cpuinfo); qemuMonitorTestFree(test); return ret; @@ -511,7 +511,7 @@ testQemuAgentArbitraryCommand(const void *data) ret = 0; -cleanup: + cleanup: VIR_FREE(reply); qemuMonitorTestFree(test); return ret; @@ -572,7 +572,7 @@ testQemuAgentTimeout(const void *data) ret = 0; -cleanup: + cleanup: VIR_FREE(reply); qemuMonitorTestFree(test); return ret; diff --git a/tests/qemuargv2xmltest.c b/tests/qemuargv2xmltest.c index 9eb3c49..6d7e23e 100644 --- a/tests/qemuargv2xmltest.c +++ b/tests/qemuargv2xmltest.c @@ -114,7 +114,7 @@ testCompareXMLToArgvHelper(const void *data) result = testCompareXMLToArgvFiles(xml, args, !!info->extraFlags); -cleanup: + cleanup: VIR_FREE(xml); VIR_FREE(args); return result; diff --git a/tests/qemucapabilitiestest.c b/tests/qemucapabilitiestest.c index 3b34f78..a291460 100644 --- a/tests/qemucapabilitiestest.c +++ b/tests/qemucapabilitiestest.c @@ -80,7 +80,7 @@ testQemuFeedMonitor(char *replies, return test; -error: + error: qemuMonitorTestFree(test); return NULL; } @@ -126,7 +126,7 @@ testQemuGetCaps(char *caps) xmlXPathFreeContext(ctxt); return qemuCaps; -error: + error: VIR_FREE(nodes); virObjectUnref(qemuCaps); xmlFreeDoc(xml); @@ -203,7 +203,7 @@ testQemuCaps(const void *opaque) goto cleanup; ret = 0; -cleanup: + cleanup: VIR_FREE(repliesFile); VIR_FREE(capsFile); VIR_FREE(replies); diff --git a/tests/qemuhelptest.c b/tests/qemuhelptest.c index 6e0a5f8..6eb8096 100644 --- a/tests/qemuhelptest.c +++ b/tests/qemuhelptest.c @@ -118,7 +118,7 @@ static int testHelpStrParsing(const void *data) } ret = 0; -cleanup: + cleanup: VIR_FREE(path); VIR_FREE(help); virObjectUnref(flags); diff --git a/tests/qemuhotplugtest.c b/tests/qemuhotplugtest.c index 4ef81e0..eaf424d 100644 --- a/tests/qemuhotplugtest.c +++ b/tests/qemuhotplugtest.c @@ -91,7 +91,7 @@ qemuHotplugCreateObjects(virDomainXMLOptionPtr xmlopt, goto cleanup; ret = 0; -cleanup: + cleanup: return ret; } @@ -296,7 +296,7 @@ testQemuHotplug(const void *data) ret = testQemuHotplugUpdate(vm, dev); } -cleanup: + cleanup: VIR_FREE(domain_filename); VIR_FREE(device_filename); VIR_FREE(result_filename); diff --git a/tests/qemumonitorjsontest.c b/tests/qemumonitorjsontest.c index 6b48296..f80d03e 100644 --- a/tests/qemumonitorjsontest.c +++ b/tests/qemumonitorjsontest.c @@ -201,7 +201,7 @@ testQemuMonitorJSONGetStatus(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -303,7 +303,7 @@ testQemuMonitorJSONGetVersion(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); VIR_FREE(package); return ret; @@ -381,7 +381,7 @@ testQemuMonitorJSONGetMachines(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); for (i = 0; i < ninfo; i++) qemuMonitorMachineInfoFree(info[i]); @@ -448,7 +448,7 @@ testQemuMonitorJSONGetCPUDefinitions(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); for (i = 0; i < ncpus; i++) VIR_FREE(cpus[i]); @@ -513,7 +513,7 @@ testQemuMonitorJSONGetCommands(const void *data) #undef CHECK ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); for (i = 0; i < ncommands; i++) VIR_FREE(commands[i]); @@ -568,7 +568,7 @@ testQemuMonitorJSONGetTPMModels(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); virStringFreeList(tpmmodels); return ret; @@ -670,7 +670,7 @@ testQemuMonitorJSONGetCommandLineOptionParameters(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); virStringFreeList(params); return ret; @@ -752,7 +752,7 @@ testQemuMonitorJSONAttachChardev(const void *data) #undef CHECK_FAIL #undef DO_CHECK -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -776,7 +776,7 @@ testQemuMonitorJSONDetachChardev(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -849,7 +849,7 @@ testQemuMonitorJSONGetListPaths(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); for (i = 0; i < npaths; i++) qemuMonitorJSONListPathFree(paths[i]); @@ -899,7 +899,7 @@ testQemuMonitorJSONGetObjectProperty(const void *data) } ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -957,7 +957,7 @@ testQemuMonitorJSONSetObjectProperty(const void *data) } ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1017,7 +1017,7 @@ testQemuMonitorJSONGetDeviceAliases(const void *data) } } -cleanup: + cleanup: virStringFreeList(aliases); qemuMonitorTestFree(test); return ret; @@ -1077,7 +1077,7 @@ testQemuMonitorJSONCPU(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1105,7 +1105,7 @@ testQemuMonitorJSONSimpleFunc(const void *opaque) goto cleanup; ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1242,7 +1242,7 @@ testQemuMonitorJSONqemuMonitorJSONGetCPUInfo(const void *data) ret = 0; -cleanup: + cleanup: VIR_FREE(cpupids); qemuMonitorTestFree(test); return ret; @@ -1279,7 +1279,7 @@ testQemuMonitorJSONqemuMonitorJSONGetBalloonInfo(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1331,7 +1331,7 @@ testQemuMonitorJSONqemuMonitorJSONGetVirtType(const void *data) } ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1402,7 +1402,7 @@ testQemuMonitorJSONqemuMonitorJSONGetBlockInfo(const void *data) } ret = 0; -cleanup: + cleanup: virHashFree(blockDevices); virHashFree(expectedBlockDevices); qemuMonitorTestFree(test); @@ -1615,7 +1615,7 @@ testQemuMonitorJSONqemuMonitorJSONGetBlockStatsInfo(const void *data) #undef CHECK #undef CHECK0 -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1651,7 +1651,7 @@ testQemuMonitorJSONqemuMonitorJSONGetMigrationCacheSize(const void *data) ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1700,7 +1700,7 @@ testQemuMonitorJSONqemuMonitorJSONGetMigrationStatus(const void *data) } ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1739,7 +1739,7 @@ testQemuMonitorJSONqemuMonitorJSONGetSpiceMigrationStatus(const void *data) } ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1803,7 +1803,7 @@ testQemuMonitorJSONqemuMonitorJSONGetPtyPaths(const void *data) } ret = 0; -cleanup: + cleanup: virHashFree(paths); virHashFree(expectedPaths); qemuMonitorTestFree(test); @@ -1847,7 +1847,7 @@ testQemuMonitorJSONqemuMonitorJSONSetBlockIoThrottle(const void *data) goto cleanup; ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1883,7 +1883,7 @@ testQemuMonitorJSONqemuMonitorJSONGetTargetArch(const void *data) } ret = 0; -cleanup: + cleanup: VIR_FREE(arch); qemuMonitorTestFree(test); return ret; @@ -1929,7 +1929,7 @@ testQemuMonitorJSONqemuMonitorJSONGetMigrationCapability(const void *data) goto cleanup; ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1954,7 +1954,7 @@ testQemuMonitorJSONqemuMonitorJSONSendKey(const void *data) goto cleanup; ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -1997,7 +1997,7 @@ testQemuMonitorJSONqemuMonitorJSONGetDumpGuestMemoryCapability(const void *data) } ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); return ret; } @@ -2069,7 +2069,7 @@ testQemuMonitorJSONGetCPUData(const void *opaque) } ret = 0; -cleanup: + cleanup: VIR_FREE(jsonFile); VIR_FREE(dataFile); VIR_FREE(jsonStr); @@ -2118,7 +2118,7 @@ testQemuMonitorJSONGetNonExistingCPUData(const void *opaque) } ret = 0; -cleanup: + cleanup: qemuMonitorTestFree(test); cpuDataFree(cpuData); return ret; diff --git a/tests/qemumonitortestutils.c b/tests/qemumonitortestutils.c index 9dae9a5..30c2153 100644 --- a/tests/qemumonitortestutils.c +++ b/tests/qemumonitortestutils.c @@ -157,7 +157,7 @@ qemuMonitorReportError(qemuMonitorTestPtr test, const char *errmsg, ...) ret = qemuMonitorTestAddReponse(test, jsonmsg); -cleanup: + cleanup: va_end(msgargs); VIR_FREE(msg); VIR_FREE(jsonmsg); @@ -269,7 +269,7 @@ qemuMonitorTestIO(virNetSocketPtr sock, VIR_EVENT_HANDLE_ERROR)) err = true; -cleanup: + cleanup: if (err) { virNetSocketRemoveIOCallback(sock); virNetSocketClose(sock); @@ -404,7 +404,7 @@ qemuMonitorTestAddHandler(qemuMonitorTestPtr test, return 0; -error: + error: if (freecb) (freecb)(opaque); VIR_FREE(item); @@ -493,7 +493,7 @@ qemuMonitorTestProcessCommandDefault(qemuMonitorTestPtr test, else ret = qemuMonitorTestAddReponse(test, data->response); -cleanup: + cleanup: VIR_FREE(cmdcopy); virJSONValueFree(val); return ret; @@ -563,7 +563,7 @@ qemuMonitorTestProcessGuestAgentSync(qemuMonitorTestPtr test, ret = qemuMonitorTestAddReponse(test, retmsg); -cleanup: + cleanup: virJSONValueFree(val); VIR_FREE(retmsg); return ret; @@ -653,7 +653,7 @@ qemuMonitorTestProcessCommandWithArgs(qemuMonitorTestPtr test, /* arguments checked out, return the response */ ret = qemuMonitorTestAddReponse(test, data->response); -cleanup: + cleanup: VIR_FREE(argstr); virJSONValueFree(val); return ret; @@ -706,7 +706,7 @@ qemuMonitorTestAddItemParams(qemuMonitorTestPtr test, qemuMonitorTestProcessCommandWithArgs, data, qemuMonitorTestHandlerDataFree); -error: + error: va_end(args); qemuMonitorTestHandlerDataFree(data); return -1; @@ -804,10 +804,10 @@ qemuMonitorCommonTestNew(virDomainXMLOptionPtr xmlopt, if (virNetSocketListen(test->server, 1) < 0) goto error; -cleanup: + cleanup: return test; -error: + error: VIR_FREE(path); VIR_FREE(tmpdir_template); qemuMonitorTestFree(test); @@ -854,7 +854,7 @@ qemuMonitorCommonTestInit(qemuMonitorTestPtr test) return 0; -error: + error: return -1; } @@ -913,7 +913,7 @@ qemuMonitorTestNew(bool json, return test; -error: + error: virDomainChrSourceDefClear(&src); qemuMonitorTestFree(test); return NULL; @@ -944,7 +944,7 @@ qemuMonitorTestNewAgent(virDomainXMLOptionPtr xmlopt) return test; -error: + error: virDomainChrSourceDefClear(&src); qemuMonitorTestFree(test); return NULL; diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c index a6c7c80..f6800c8 100644 --- a/tests/qemuxml2argvtest.c +++ b/tests/qemuxml2argvtest.c @@ -103,7 +103,7 @@ fakeStoragePoolLookupByName(virConnectPtr conn, ret = virGetStoragePool(conn, name, fakeUUID, NULL, NULL); -cleanup: + cleanup: VIR_FREE(xmlpath); return ret; } @@ -140,11 +140,11 @@ fakeStorageVolLookupByName(virStoragePoolPtr pool, ret = virGetStorageVol(pool->conn, pool->name, volinfo[1], volinfo[0], NULL, NULL); -cleanup: + cleanup: virStringFreeList(volinfo); return ret; -fallback: + fallback: ret = virGetStorageVol(pool->conn, pool->name, name, "block", NULL, NULL); goto cleanup; } @@ -203,7 +203,7 @@ fakeStoragePoolGetXMLDesc(virStoragePoolPtr pool, goto cleanup; } -cleanup: + cleanup: VIR_FREE(xmlpath); return xmlbuf; @@ -403,7 +403,7 @@ static int testCompareXMLToArgvFiles(const char *xml, ret = 0; -out: + out: VIR_FREE(log); VIR_FREE(expectargv); VIR_FREE(actualargv); @@ -444,7 +444,7 @@ testCompareXMLToArgvHelper(const void *data) info->migrateFrom, info->migrateFd, flags); -cleanup: + cleanup: VIR_FREE(xml); VIR_FREE(args); return result; diff --git a/tests/qemuxml2xmltest.c b/tests/qemuxml2xmltest.c index a1f7a4f..5dc3afd 100644 --- a/tests/qemuxml2xmltest.c +++ b/tests/qemuxml2xmltest.c @@ -103,7 +103,7 @@ testCompareXMLToXMLHelper(const void *data) ret = 0; -cleanup: + cleanup: VIR_FREE(xml_in); VIR_FREE(xml_out); return ret; diff --git a/tests/qemuxmlnstest.c b/tests/qemuxmlnstest.c index 2a4d746..30bb723 100644 --- a/tests/qemuxmlnstest.c +++ b/tests/qemuxmlnstest.c @@ -193,7 +193,7 @@ testCompareXMLToArgvHelper(const void *data) info->migrateFrom, info->migrateFd, info->json, info->expectError); -cleanup: + cleanup: VIR_FREE(xml); VIR_FREE(args); return result; diff --git a/tests/secretxml2xmltest.c b/tests/secretxml2xmltest.c index be9ef64..c9d879e 100644 --- a/tests/secretxml2xmltest.c +++ b/tests/secretxml2xmltest.c @@ -35,7 +35,7 @@ testCompareXMLToXMLFiles(const char *inxml, const char *outxml) ret = 0; -fail: + fail: VIR_FREE(inXmlData); VIR_FREE(outXmlData); VIR_FREE(actual); @@ -67,7 +67,7 @@ testCompareXMLToXMLHelper(const void *data) result = testCompareXMLToXMLFiles(inxml, outxml); -cleanup: + cleanup: VIR_FREE(inxml); VIR_FREE(outxml); diff --git a/tests/securityselinuxlabeltest.c b/tests/securityselinuxlabeltest.c index e6a0c29..ce5e6a8 100644 --- a/tests/securityselinuxlabeltest.c +++ b/tests/securityselinuxlabeltest.c @@ -139,7 +139,7 @@ testSELinuxLoadFileList(const char *testname, ret = 0; -cleanup: + cleanup: VIR_FORCE_FCLOSE(fp); VIR_FREE(path); VIR_FREE(line); @@ -200,7 +200,7 @@ testSELinuxLoadDef(const char *testname) testSELinuxMungePath(&def->os.initrd) < 0) goto cleanup; -cleanup: + cleanup: VIR_FREE(xmlfile); VIR_FREE(xmlstr); return def; @@ -298,7 +298,7 @@ testSELinuxLabeling(const void *opaque) ret = 0; -cleanup: + cleanup: if (testSELinuxDeleteDisks(files, nfiles) < 0) VIR_WARN("unable to fully clean up"); diff --git a/tests/securityselinuxtest.c b/tests/securityselinuxtest.c index 4ea2b72..3b5c3e5 100644 --- a/tests/securityselinuxtest.c +++ b/tests/securityselinuxtest.c @@ -93,7 +93,7 @@ testBuildDomainDef(bool dynamic, return def; -error: + error: virDomainDefFree(def); return NULL; } @@ -257,7 +257,7 @@ testSELinuxGenLabel(const void *opaque) ret = 0; -cleanup: + cleanup: context_free(con); context_free(imgcon); virDomainDefFree(def); diff --git a/tests/sexpr2xmltest.c b/tests/sexpr2xmltest.c index 34cdc6d..15fe187 100644 --- a/tests/sexpr2xmltest.c +++ b/tests/sexpr2xmltest.c @@ -108,7 +108,7 @@ testCompareHelper(const void *data) result = testCompareFiles(xml, args, info->version); -cleanup: + cleanup: VIR_FREE(xml); VIR_FREE(args); diff --git a/tests/storagebackendsheepdogtest.c b/tests/storagebackendsheepdogtest.c index d4fa479..e219acb 100644 --- a/tests/storagebackendsheepdogtest.c +++ b/tests/storagebackendsheepdogtest.c @@ -74,7 +74,7 @@ test_node_info_parser(collie_test test, char *poolxml) pool->allocation == test.expected_allocation) ret = 0; - cleanup: + cleanup: VIR_FREE(output); VIR_FREE(poolXmlData); virStoragePoolDefFree(pool); @@ -118,7 +118,7 @@ test_vdi_list_parser(collie_test test, char *poolxml, char *volxml) vol->allocation == test.expected_allocation) ret = 0; - cleanup: + cleanup: VIR_FREE(output); VIR_FREE(poolXmlData); VIR_FREE(volXmlData); @@ -203,7 +203,7 @@ mymain(void) ret = 0; - cleanup: + cleanup: VIR_FREE(poolxml); VIR_FREE(volxml); return ret == 0 ? EXIT_SUCCESS : EXIT_FAILURE; diff --git a/tests/storagepoolxml2xmltest.c b/tests/storagepoolxml2xmltest.c index d54f2e5..971fe3b 100644 --- a/tests/storagepoolxml2xmltest.c +++ b/tests/storagepoolxml2xmltest.c @@ -67,7 +67,7 @@ testCompareXMLToXMLHelper(const void *data) result = testCompareXMLToXMLFiles(inxml, outxml); -cleanup: + cleanup: VIR_FREE(inxml); VIR_FREE(outxml); diff --git a/tests/storagevolxml2argvtest.c b/tests/storagevolxml2argvtest.c index 7362823..11d70e1 100644 --- a/tests/storagevolxml2argvtest.c +++ b/tests/storagevolxml2argvtest.c @@ -120,7 +120,7 @@ testCompareXMLToArgvFiles(bool shouldFail, ret = 0; -cleanup: + cleanup: virStoragePoolDefFree(pool); virStoragePoolDefFree(inputpool); virStorageVolDefFree(vol); @@ -181,7 +181,7 @@ testCompareXMLToArgvHelper(const void *data) cmdline, info->flags, info->imgformat); -cleanup: + cleanup: VIR_FREE(poolxml); VIR_FREE(volxml); VIR_FREE(inputvolxml); diff --git a/tests/storagevolxml2xmltest.c b/tests/storagevolxml2xmltest.c index 1705aef..234a2f1 100644 --- a/tests/storagevolxml2xmltest.c +++ b/tests/storagevolxml2xmltest.c @@ -86,7 +86,7 @@ testCompareXMLToXMLHelper(const void *data) result = testCompareXMLToXMLFiles(poolxml, inxml, outxml); -cleanup: + cleanup: VIR_FREE(poolxml); VIR_FREE(inxml); VIR_FREE(outxml); diff --git a/tests/sysinfotest.c b/tests/sysinfotest.c index c8675f9..5ba3298 100644 --- a/tests/sysinfotest.c +++ b/tests/sysinfotest.c @@ -89,7 +89,7 @@ testSysinfo(const void *data) result = 0; -cleanup: + cleanup: VIR_FREE(sysfsExpectData); virSysinfoDefFree(ret); virBufferFreeAndReset(&buf); @@ -123,7 +123,7 @@ sysinfotest_run(const char *test, ret = EXIT_SUCCESS; -error: + error: VIR_FREE(testdata.decoder); VIR_FREE(testdata.sysinfo); VIR_FREE(testdata.cpuinfo); diff --git a/tests/test_conf.c b/tests/test_conf.c index acf41c6..05704df 100644 --- a/tests/test_conf.c +++ b/tests/test_conf.c @@ -42,7 +42,7 @@ int main(int argc, char **argv) exit_code = EXIT_SUCCESS; -cleanup: + cleanup: VIR_FREE(buffer); return exit_code; } diff --git a/tests/testutils.c b/tests/testutils.c index 179c72a..9767a78 100644 --- a/tests/testutils.c +++ b/tests/testutils.c @@ -884,7 +884,7 @@ virCapsPtr virTestGenericCapsInit(void) return caps; -error: + error: virObjectUnref(caps); return NULL; } diff --git a/tests/testutilslxc.c b/tests/testutilslxc.c index a1574d3..968d8d0 100644 --- a/tests/testutilslxc.c +++ b/tests/testutilslxc.c @@ -49,7 +49,7 @@ virCapsPtr testLXCCapsInit(void) return caps; -error: + error: virObjectUnref(caps); return NULL; } diff --git a/tests/testutilsqemu.c b/tests/testutilsqemu.c index f7810f6..a8884ba 100644 --- a/tests/testutilsqemu.c +++ b/tests/testutilsqemu.c @@ -80,7 +80,7 @@ static int testQemuAddPPC64Guest(virCapsPtr caps) return 0; -error: + error: /* No way to free a guest? */ virCapabilitiesFreeMachines(machines, 1); return -1; @@ -110,7 +110,7 @@ static int testQemuAddPPCGuest(virCapsPtr caps) return 0; -error: + error: /* No way to free a guest? */ virCapabilitiesFreeMachines(machines, 1); return -1; @@ -140,7 +140,7 @@ static int testQemuAddS390Guest(virCapsPtr caps) return 0; -error: + error: virCapabilitiesFreeMachines(machines, ARRAY_CARDINALITY(s390_machines)); return -1; } @@ -170,7 +170,7 @@ static int testQemuAddArmGuest(virCapsPtr caps) return 0; -error: + error: virCapabilitiesFreeMachines(capsmachines, ARRAY_CARDINALITY(machines)); return -1; } @@ -198,7 +198,7 @@ static int testQemuAddAARCH64Guest(virCapsPtr caps) return 0; -error: + error: virCapabilitiesFreeMachines(capsmachines, ARRAY_CARDINALITY(machines)); return -1; } @@ -348,7 +348,7 @@ virCapsPtr testQemuCapsInit(void) return caps; -cleanup: + cleanup: virCapabilitiesFreeMachines(machines, nmachines); virObjectUnref(caps); return NULL; diff --git a/tests/testutilsxen.c b/tests/testutilsxen.c index f3216e9..3c8b60f 100644 --- a/tests/testutilsxen.c +++ b/tests/testutilsxen.c @@ -64,7 +64,7 @@ virCapsPtr testXenCapsInit(void) return caps; -cleanup: + cleanup: virCapabilitiesFreeMachines(machines, nmachines); virObjectUnref(caps); return NULL; diff --git a/tests/virauthconfigtest.c b/tests/virauthconfigtest.c index 594becc..d9e05fd 100644 --- a/tests/virauthconfigtest.c +++ b/tests/virauthconfigtest.c @@ -78,7 +78,7 @@ static int testAuthLookup(const void *args) } ret = 0; -cleanup: + cleanup: return ret; } diff --git a/tests/virbitmaptest.c b/tests/virbitmaptest.c index 64fa769..cc34216 100644 --- a/tests/virbitmaptest.c +++ b/tests/virbitmaptest.c @@ -57,7 +57,7 @@ test1(const void *data ATTRIBUTE_UNUSED) ret = 0; -error: + error: virBitmapFree(bitmap); return ret; } @@ -138,7 +138,7 @@ test2(const void *data ATTRIBUTE_UNUSED) ret = 0; -error: + error: virBitmapFree(bitmap); VIR_FREE(bitsString2); return ret; @@ -166,7 +166,7 @@ test3(const void *data ATTRIBUTE_UNUSED) goto error; ret = 0; -error: + error: virBitmapFree(bitmap); return ret; } @@ -261,7 +261,7 @@ test4(const void *data ATTRIBUTE_UNUSED) virBitmapFree(bitmap); return 0; -error: + error: virBitmapFree(bitmap); return -1; } @@ -308,7 +308,7 @@ test5(const void *v ATTRIBUTE_UNUSED) goto error; ret = 0; -error: + error: virBitmapFree(bitmap); VIR_FREE(data2); return ret; @@ -392,7 +392,7 @@ test6(const void *v ATTRIBUTE_UNUSED) ret = 0; -error: + error: virBitmapFree(bitmap); VIR_FREE(str); return ret; @@ -433,7 +433,7 @@ test7(const void *v ATTRIBUTE_UNUSED) return 0; -error: + error: virBitmapFree(bitmap); return -1; } @@ -459,7 +459,7 @@ test8(const void *v ATTRIBUTE_UNUSED) goto cleanup; ret = 0; -cleanup: + cleanup: virBitmapFree(bitmap); return ret; } @@ -491,7 +491,7 @@ test9(const void *opaque ATTRIBUTE_UNUSED) goto cleanup; ret = 0; -cleanup: + cleanup: virBitmapFree(bitmap); return ret; diff --git a/tests/virbuftest.c b/tests/virbuftest.c index 9eb7f66..2398890 100644 --- a/tests/virbuftest.c +++ b/tests/virbuftest.c @@ -48,7 +48,7 @@ static int testBufInfiniteLoop(const void *data) virBufferAsprintf(buf, "%s", addstr); ret = 0; -out: + out: bufret = virBufferContentAndReset(buf); if (!bufret) { TEST_ERROR("Buffer had error set"); @@ -194,7 +194,7 @@ static int testBufTrim(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virBufferFreeAndReset(buf); VIR_FREE(result); return ret; diff --git a/tests/vircapstest.c b/tests/vircapstest.c index 4264e9e..821a92b 100644 --- a/tests/vircapstest.c +++ b/tests/vircapstest.c @@ -74,7 +74,7 @@ buildNUMATopology(int seq) return caps; -error: + error: virCapabilitiesClearHostNUMACellCPUTopology(cell_cpus, MAX_CPUS_IN_CELL); VIR_FREE(cell_cpus); virObjectUnref(caps); @@ -108,7 +108,7 @@ test_virCapabilitiesGetCpusForNodemask(const void *data ATTRIBUTE_UNUSED) ret = 0; -error: + error: virObjectUnref(caps); virBitmapFree(nodemask); virBitmapFree(cpumap); diff --git a/tests/vircgroupmock.c b/tests/vircgroupmock.c index ae88984..6e0a0e7 100644 --- a/tests/vircgroupmock.c +++ b/tests/vircgroupmock.c @@ -168,7 +168,7 @@ static int make_file(const char *path, goto cleanup; ret = 0; -cleanup: + cleanup: if (fd != -1 && close(fd) < 0) ret = -1; free(filepath); @@ -368,7 +368,7 @@ static int make_controller(const char *path, mode_t mode) } ret = 0; -cleanup: + cleanup: return ret; } diff --git a/tests/vircgrouptest.c b/tests/vircgrouptest.c index caddc6a..dd078c1 100644 --- a/tests/vircgrouptest.c +++ b/tests/vircgrouptest.c @@ -180,7 +180,7 @@ static int testCgroupNewForSelf(const void *args ATTRIBUTE_UNUSED) ret = validateCgroup(cgroup, "", mountsFull, links, placement); -cleanup: + cleanup: virCgroupFree(&cgroup); return ret; } @@ -266,7 +266,7 @@ static int testCgroupNewForPartition(const void *args ATTRIBUTE_UNUSED) } ret = validateCgroup(cgroup, "/virtualmachines.partition", mountsFull, links, placementFull); -cleanup: + cleanup: virCgroupFree(&cgroup); return ret; } @@ -315,7 +315,7 @@ static int testCgroupNewForPartitionNested(const void *args ATTRIBUTE_UNUSED) ret = validateCgroup(cgroup, "/deployment.partition/production.partition", mountsFull, links, placementFull); -cleanup: + cleanup: virCgroupFree(&cgroup); return ret; } @@ -369,7 +369,7 @@ static int testCgroupNewForPartitionNestedDeep(const void *args ATTRIBUTE_UNUSED ret = validateCgroup(cgroup, "/user/berrange.user/production.partition", mountsFull, links, placementFull); -cleanup: + cleanup: virCgroupFree(&cgroup); return ret; } @@ -405,7 +405,7 @@ static int testCgroupNewForPartitionDomain(const void *args ATTRIBUTE_UNUSED) ret = validateCgroup(domaincgroup, "/production.partition/foo.libvirt-lxc", mountsFull, links, placement); -cleanup: + cleanup: virCgroupFree(&partitioncgroup); virCgroupFree(&domaincgroup); return ret; @@ -456,7 +456,7 @@ static int testCgroupNewForPartitionDomainEscaped(const void *args ATTRIBUTE_UNU */ ret = validateCgroup(domaincgroup, "/_cgroup.evil/net_cls.evil/__evil.evil/_cpu.foo.libvirt-lxc", mountsFull, links, placement); -cleanup: + cleanup: virCgroupFree(&partitioncgroup3); virCgroupFree(&partitioncgroup2); virCgroupFree(&partitioncgroup1); @@ -485,7 +485,7 @@ static int testCgroupNewForSelfAllInOne(const void *args ATTRIBUTE_UNUSED) ret = validateCgroup(cgroup, "", mountsAllInOne, linksAllInOne, placement); -cleanup: + cleanup: virCgroupFree(&cgroup); return ret; } @@ -513,7 +513,7 @@ static int testCgroupNewForSelfLogind(const void *args ATTRIBUTE_UNUSED) ret = validateCgroup(cgroup, "", mountsLogind, linksLogind, placement); -cleanup: + cleanup: virCgroupFree(&cgroup); return ret; } @@ -590,7 +590,7 @@ static int testCgroupGetPercpuStats(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virCgroupFree(&cgroup); return ret; } @@ -622,7 +622,7 @@ static int testCgroupGetMemoryUsage(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virCgroupFree(&cgroup); return ret; } @@ -672,7 +672,7 @@ static int testCgroupGetBlkioIoServiced(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virCgroupFree(&cgroup); return ret; } @@ -745,7 +745,7 @@ static int testCgroupGetBlkioIoDeviceServiced(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virCgroupFree(&cgroup); return ret; } diff --git a/tests/virdbustest.c b/tests/virdbustest.c index 2269c8d..1870f35 100644 --- a/tests/virdbustest.c +++ b/tests/virdbustest.c @@ -117,7 +117,7 @@ static int testMessageSimple(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: VIR_FREE(out_string); VIR_FREE(out_signature); VIR_FREE(out_objectpath); @@ -168,7 +168,7 @@ static int testMessageVariant(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: VIR_FREE(out_str1); VIR_FREE(out_str2); dbus_message_unref(msg); @@ -221,7 +221,7 @@ static int testMessageArray(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: VIR_FREE(out_str1); VIR_FREE(out_str2); dbus_message_unref(msg); @@ -311,7 +311,7 @@ static int testMessageArrayRef(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: VIR_FREE(out_int32); VIR_FREE(out_str1); VIR_FREE(out_str2); @@ -386,7 +386,7 @@ static int testMessageStruct(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: VIR_FREE(out_string); VIR_FREE(out_signature); VIR_FREE(out_objectpath); @@ -454,7 +454,7 @@ static int testMessageDict(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: VIR_FREE(out_str1); VIR_FREE(out_str2); VIR_FREE(out_key1); diff --git a/tests/virendiantest.c b/tests/virendiantest.c index 4f672ce..2097c28 100644 --- a/tests/virendiantest.c +++ b/tests/virendiantest.c @@ -51,7 +51,7 @@ test1(const void *data ATTRIBUTE_UNUSED) goto cleanup; ret = 0; -cleanup: + cleanup: return ret; } @@ -82,7 +82,7 @@ test2(const void *data ATTRIBUTE_UNUSED) goto cleanup; ret = 0; -cleanup: + cleanup: return ret; } diff --git a/tests/virhashtest.c b/tests/virhashtest.c index eead479..340dc84 100644 --- a/tests/virhashtest.c +++ b/tests/virhashtest.c @@ -121,7 +121,7 @@ testHashGrow(const void *data) ret = 0; -cleanup: + cleanup: virHashFree(hash); return ret; } @@ -163,7 +163,7 @@ testHashUpdate(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virHashFree(hash); return ret; } @@ -195,7 +195,7 @@ testHashRemove(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virHashFree(hash); return ret; } @@ -290,7 +290,7 @@ testHashRemoveForEach(const void *data) ret = 0; -cleanup: + cleanup: virHashFree(hash); return ret; } @@ -322,7 +322,7 @@ testHashSteal(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virHashFree(hash); return ret; } @@ -393,7 +393,7 @@ testHashForEach(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virHashFree(hash); return ret; } @@ -453,7 +453,7 @@ testHashRemoveSet(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virHashFree(hash); return ret; } @@ -494,7 +494,7 @@ testHashSearch(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virHashFree(hash); return ret; } @@ -577,7 +577,7 @@ testHashGetItems(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: VIR_FREE(array); virHashFree(hash); return ret; @@ -655,7 +655,7 @@ testHashEqual(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virHashFree(hash1); virHashFree(hash2); return ret; diff --git a/tests/virhostdevtest.c b/tests/virhostdevtest.c index 56f2f7a..089014e 100644 --- a/tests/virhostdevtest.c +++ b/tests/virhostdevtest.c @@ -116,7 +116,7 @@ myInit(void) return 0; -cleanup: + cleanup: myCleanup(); return -1; } @@ -139,7 +139,7 @@ virHostdevHostSupportsPassthroughKVM(void) ret = true; # endif -cleanup: + cleanup: VIR_FORCE_CLOSE(kvmfd); return ret; @@ -206,7 +206,7 @@ testVirHostdevPreparePCIHostdevs_unmanaged(const void *oaque ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: return ret; } @@ -240,7 +240,7 @@ testVirHostdevReAttachPCIHostdevs_unmanaged(const void *oaque ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: return ret; } @@ -286,7 +286,7 @@ testVirHostdevPreparePCIHostdevs_managed(const void *oaque ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: return ret; } @@ -318,7 +318,7 @@ testVirHostdevReAttachPCIHostdevs_managed(const void *oaque ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: return ret; } @@ -339,7 +339,7 @@ testVirHostdevDetachPCINodeDevice(const void *oaque ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: return ret; } static int @@ -355,7 +355,7 @@ testVirHostdevResetPCINodeDevice(const void *oaque ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: return ret; } @@ -376,7 +376,7 @@ testVirHostdevReAttachPCINodeDevice(const void *oaque ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: return ret; } @@ -403,7 +403,7 @@ testVirHostdevUpdateActivePCIHostdevs(const void *oaque ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: return ret; } diff --git a/tests/viridentitytest.c b/tests/viridentitytest.c index 3e79ad8..b1a1db0 100644 --- a/tests/viridentitytest.c +++ b/tests/viridentitytest.c @@ -91,7 +91,7 @@ static int testIdentityAttrs(const void *data ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virObjectUnref(ident); return ret; } @@ -158,7 +158,7 @@ static int testIdentityEqual(const void *data ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virObjectUnref(identa); virObjectUnref(identb); return ret; @@ -195,7 +195,7 @@ static int testIdentityGetSystem(const void *data) } ret = 0; -cleanup: + cleanup: virObjectUnref(ident); return ret; } diff --git a/tests/viriscsitest.c b/tests/viriscsitest.c index ebe830a..c697a4a 100644 --- a/tests/viriscsitest.c +++ b/tests/viriscsitest.c @@ -119,7 +119,7 @@ testISCSIGetSession(const void *data) ret = 0; -cleanup: + cleanup: virCommandSetDryRun(NULL, NULL, NULL); VIR_FREE(actual_session); return ret; @@ -165,7 +165,7 @@ testISCSIScanTargets(const void *data) ret = 0; -cleanup: + cleanup: virCommandSetDryRun(NULL, NULL, NULL); for (i = 0; i < ntargets; i++) VIR_FREE(targets[i]); diff --git a/tests/virkeycodetest.c b/tests/virkeycodetest.c index 77378ec..a429fbc 100644 --- a/tests/virkeycodetest.c +++ b/tests/virkeycodetest.c @@ -61,7 +61,7 @@ static int testKeycodeMapping(const void *data ATTRIBUTE_UNUSED) #undef TRANSLATE ret = 0; -cleanup: + cleanup: return ret; } @@ -88,7 +88,7 @@ static int testKeycodeStrings(const void *data ATTRIBUTE_UNUSED) #undef TRANSLATE ret = 0; -cleanup: + cleanup: return ret; } diff --git a/tests/virkeyfiletest.c b/tests/virkeyfiletest.c index b149ce1..c31d2aa 100644 --- a/tests/virkeyfiletest.c +++ b/tests/virkeyfiletest.c @@ -101,7 +101,7 @@ static int testParse(const void *args ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virKeyFileFree(kf); return ret; } diff --git a/tests/virkmodtest.c b/tests/virkmodtest.c index 9473352..9fa40c3 100644 --- a/tests/virkmodtest.c +++ b/tests/virkmodtest.c @@ -52,7 +52,7 @@ testKModConfig(const void *args ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: VIR_FREE(outbuf); return ret; } @@ -80,7 +80,7 @@ checkOutput(virBufferPtr buf, const char *exp_cmd) ret = 0; -cleanup: + cleanup: VIR_FREE(actual_cmd); return ret; } @@ -109,7 +109,7 @@ testKModLoad(const void *args) ret = 0; -cleanup: + cleanup: virCommandSetDryRun(NULL, NULL, NULL); VIR_FREE(errbuf); return ret; @@ -138,7 +138,7 @@ testKModUnload(const void *args) ret = 0; -cleanup: + cleanup: virCommandSetDryRun(NULL, NULL, NULL); VIR_FREE(errbuf); return ret; diff --git a/tests/virlockspacetest.c b/tests/virlockspacetest.c index 650b598..4db9c0f 100644 --- a/tests/virlockspacetest.c +++ b/tests/virlockspacetest.c @@ -54,7 +54,7 @@ static int testLockSpaceCreate(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virLockSpaceFree(lockspace); rmdir(LOCKSPACE_DIR); return ret; @@ -88,7 +88,7 @@ static int testLockSpaceResourceLifecycle(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virLockSpaceFree(lockspace); rmdir(LOCKSPACE_DIR); return ret; @@ -134,7 +134,7 @@ static int testLockSpaceResourceLockExcl(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virLockSpaceFree(lockspace); rmdir(LOCKSPACE_DIR); return ret; @@ -172,7 +172,7 @@ static int testLockSpaceResourceLockExclAuto(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virLockSpaceFree(lockspace); rmdir(LOCKSPACE_DIR); return ret; @@ -226,7 +226,7 @@ static int testLockSpaceResourceLockShr(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virLockSpaceFree(lockspace); rmdir(LOCKSPACE_DIR); return ret; @@ -286,7 +286,7 @@ static int testLockSpaceResourceLockShrAuto(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virLockSpaceFree(lockspace); rmdir(LOCKSPACE_DIR); return ret; @@ -332,7 +332,7 @@ static int testLockSpaceResourceLockPath(const void *args ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virLockSpaceFree(lockspace); rmdir(LOCKSPACE_DIR); return ret; diff --git a/tests/virnetdevbandwidthtest.c b/tests/virnetdevbandwidthtest.c index f5d252c..3f68194 100644 --- a/tests/virnetdevbandwidthtest.c +++ b/tests/virnetdevbandwidthtest.c @@ -100,7 +100,7 @@ testVirNetDevBandwidthSet(const void *data) } ret = 0; -cleanup: + cleanup: virCommandSetDryRun(NULL, NULL, NULL); virNetDevBandwidthFree(band); virBufferFreeAndReset(&buf); diff --git a/tests/virnetmessagetest.c b/tests/virnetmessagetest.c index 60f9602..440b7bf 100644 --- a/tests/virnetmessagetest.c +++ b/tests/virnetmessagetest.c @@ -82,7 +82,7 @@ static int testMessageHeaderEncode(const void *args ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virNetMessageFree(msg); return ret; } @@ -178,7 +178,7 @@ static int testMessageHeaderDecode(const void *args ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virNetMessageFree(msg); return ret; } @@ -274,7 +274,7 @@ static int testMessagePayloadEncode(const void *args ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: if (err.message) VIR_FREE(*err.message); if (err.str1) @@ -449,7 +449,7 @@ static int testMessagePayloadDecode(const void *args ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: xdr_free((xdrproc_t)xdr_virNetMessageError, (void*)&err); virNetMessageFree(msg); return ret; @@ -516,7 +516,7 @@ static int testMessagePayloadStreamEncode(const void *args ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virNetMessageFree(msg); return ret; } diff --git a/tests/virnetsockettest.c b/tests/virnetsockettest.c index a04e576..5d91f26 100644 --- a/tests/virnetsockettest.c +++ b/tests/virnetsockettest.c @@ -141,7 +141,7 @@ checkProtocols(bool *hasIPv4, bool *hasIPv6, ret = 0; -cleanup: + cleanup: VIR_FORCE_CLOSE(s4); VIR_FORCE_CLOSE(s6); return ret; @@ -194,7 +194,7 @@ static int testSocketTCPAccept(const void *opaque) ret = 0; -cleanup: + cleanup: virObjectUnref(ssock); for (i = 0; i < nlsock; i++) virObjectUnref(lsock[i]); @@ -245,7 +245,7 @@ static int testSocketUNIXAccept(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: VIR_FREE(path); virObjectUnref(lsock); virObjectUnref(ssock); @@ -323,7 +323,7 @@ static int testSocketUNIXAddrs(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: VIR_FREE(path); virObjectUnref(lsock); virObjectUnref(ssock); @@ -356,7 +356,7 @@ static int testSocketCommandNormal(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virObjectUnref(csock); return ret; } @@ -379,7 +379,7 @@ static int testSocketCommandFail(const void *data ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virObjectUnref(csock); return ret; } @@ -448,7 +448,7 @@ static int testSocketSSH(const void *opaque) ret = 0; -cleanup: + cleanup: virObjectUnref(csock); return ret; } diff --git a/tests/virnettlscontexttest.c b/tests/virnettlscontexttest.c index c66ae58..51a0369 100644 --- a/tests/virnettlscontexttest.c +++ b/tests/virnettlscontexttest.c @@ -101,7 +101,7 @@ static int testTLSContextInit(const void *opaque) ret = 0; -cleanup: + cleanup: virObjectUnref(ctxt); return ret; } diff --git a/tests/virnettlssessiontest.c b/tests/virnettlssessiontest.c index 3397865..1e2683c 100644 --- a/tests/virnettlssessiontest.c +++ b/tests/virnettlssessiontest.c @@ -223,7 +223,7 @@ static int testTLSSessionInit(const void *opaque) ret = 0; -cleanup: + cleanup: virObjectUnref(serverCtxt); virObjectUnref(clientCtxt); virObjectUnref(serverSess); diff --git a/tests/virpcimock.c b/tests/virpcimock.c index b42e8bf..033b7e9 100644 --- a/tests/virpcimock.c +++ b/tests/virpcimock.c @@ -219,7 +219,7 @@ pci_read_file(const char *path, goto cleanup; ret = 0; -cleanup: + cleanup: VIR_FREE(newpath); realclose(fd); return ret; @@ -283,7 +283,7 @@ add_fd(int fd, const char *path) callbacks[nCallbacks++].fd = fd; ret = 0; -cleanup: + cleanup: return ret; } @@ -307,7 +307,7 @@ remove_fd(int fd) } ret = 0; -cleanup: + cleanup: return ret; } @@ -558,7 +558,7 @@ pci_driver_bind(struct pciDriver *driver, dev->driver = driver; ret = 0; -cleanup: + cleanup: VIR_FREE(devpath); VIR_FREE(driverpath); return ret; @@ -592,7 +592,7 @@ pci_driver_unbind(struct pciDriver *driver, dev->driver = NULL; ret = 0; -cleanup: + cleanup: VIR_FREE(devpath); VIR_FREE(driverpath); return ret; @@ -649,7 +649,7 @@ pci_driver_handle_bind(const char *path) } ret = pci_driver_bind(driver, dev); -cleanup: + cleanup: return ret; } @@ -666,7 +666,7 @@ pci_driver_handle_unbind(const char *path) } ret = pci_driver_unbind(dev->driver, dev); -cleanup: + cleanup: return ret; } static int @@ -720,7 +720,7 @@ pci_driver_handle_new_id(const char *path) } ret = 0; -cleanup: + cleanup: return ret; } @@ -761,7 +761,7 @@ pci_driver_handle_remove_id(const char *path) } ret = 0; -cleanup: + cleanup: return ret; } diff --git a/tests/virpcitest.c b/tests/virpcitest.c index 1df8d0a..4a8e4a9 100644 --- a/tests/virpcitest.c +++ b/tests/virpcitest.c @@ -52,7 +52,7 @@ testVirPCIDeviceCheckDriver(virPCIDevicePtr dev, const char *expected) } ret = 0; -cleanup: + cleanup: VIR_FREE(path); VIR_FREE(driver); return ret; @@ -77,7 +77,7 @@ testVirPCIDeviceNew(const void *opaque ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virPCIDeviceFree(dev); return ret; } @@ -122,7 +122,7 @@ testVirPCIDeviceDetach(const void *oaque ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: for (i = 0; i < nDev; i++) virPCIDeviceFree(dev[i]); virObjectUnref(activeDevs); @@ -156,7 +156,7 @@ testVirPCIDeviceReset(const void *opaque ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: for (i = 0; i < nDev; i++) virPCIDeviceFree(dev[i]); virObjectUnref(activeDevs); @@ -205,7 +205,7 @@ testVirPCIDeviceReattach(const void *opaque ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virObjectUnref(activeDevs); virObjectUnref(inactiveDevs); return ret; @@ -233,7 +233,7 @@ testVirPCIDeviceIsAssignable(const void *opaque) ret = 0; virPCIDeviceFree(dev); -cleanup: + cleanup: return ret; } @@ -253,7 +253,7 @@ testVirPCIDeviceDetachSingle(const void *opaque) goto cleanup; ret = 0; -cleanup: + cleanup: virPCIDeviceFree(dev); return ret; } @@ -284,7 +284,7 @@ testVirPCIDeviceDetachFail(const void *opaque) virPCIDeviceGetStubDriver(dev)); } -cleanup: + cleanup: virPCIDeviceFree(dev); return ret; } @@ -305,7 +305,7 @@ testVirPCIDeviceReattachSingle(const void *opaque) goto cleanup; ret = 0; -cleanup: + cleanup: virPCIDeviceFree(dev); return ret; } @@ -325,7 +325,7 @@ testVirPCIDeviceCheckDriverTest(const void *opaque) goto cleanup; ret = 0; -cleanup: + cleanup: virPCIDeviceFree(dev); return ret; } @@ -345,7 +345,7 @@ testVirPCIDeviceUnbind(const void *opaque) goto cleanup; ret = 0; -cleanup: + cleanup: virPCIDeviceFree(dev); return ret; } diff --git a/tests/virportallocatortest.c b/tests/virportallocatortest.c index fa701ba..48d2c9a 100644 --- a/tests/virportallocatortest.c +++ b/tests/virportallocatortest.c @@ -184,7 +184,7 @@ static int testAllocAll(const void *args ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virObjectUnref(alloc); return ret; } @@ -237,7 +237,7 @@ static int testAllocReuse(const void *args ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: virObjectUnref(alloc); return ret; } diff --git a/tests/virscsitest.c b/tests/virscsitest.c index 586c41b..b917e47 100644 --- a/tests/virscsitest.c +++ b/tests/virscsitest.c @@ -47,7 +47,7 @@ test1(const void *data ATTRIBUTE_UNUSED) goto cleanup; ret = 0; -cleanup: + cleanup: VIR_FREE(name); return ret; } @@ -151,7 +151,7 @@ test2(const void *data ATTRIBUTE_UNUSED) goto cleanup; ret = 0; -cleanup: + cleanup: VIR_FREE(sgname); if (free_dev) virSCSIDeviceFree(dev); @@ -180,7 +180,7 @@ mymain(void) if (virtTestRun("test2", test2, NULL) < 0) ret = -1; -cleanup: + cleanup: VIR_FREE(virscsi_prefix); return ret; } diff --git a/tests/virshtest.c b/tests/virshtest.c index f7edc02..3fdae92 100644 --- a/tests/virshtest.c +++ b/tests/virshtest.c @@ -78,7 +78,7 @@ testCompareOutputLit(const char *expectData, result = 0; -cleanup: + cleanup: VIR_FREE(actualData); return result; diff --git a/tests/virstoragetest.c b/tests/virstoragetest.c index 38581db..2890651 100644 --- a/tests/virstoragetest.c +++ b/tests/virstoragetest.c @@ -186,14 +186,14 @@ testPrepImages(void) #endif ret = 0; -cleanup: + cleanup: VIR_FREE(buf); virCommandFree(cmd); if (ret) testCleanupImages(); return ret; -skip: + skip: fputs("qemu-img is too old; skipping this test\n", stderr); ret = EXIT_AM_SKIP; goto cleanup; @@ -306,7 +306,7 @@ testStorageChain(const void *args) } ret = 0; -cleanup: + cleanup: virStorageFileFreeMetadata(meta); return ret; } diff --git a/tests/virstringtest.c b/tests/virstringtest.c index 97cf75f..a4ae966 100644 --- a/tests/virstringtest.c +++ b/tests/virstringtest.c @@ -80,7 +80,7 @@ static int testSplit(const void *args) } ret = 0; -cleanup: + cleanup: virStringFreeList(got); return ret; @@ -103,7 +103,7 @@ static int testJoin(const void *args) } ret = 0; -cleanup: + cleanup: VIR_FREE(got); return ret; @@ -192,7 +192,7 @@ testStrdup(const void *data ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: for (i = 0; i < ARRAY_CARDINALITY(array); i++) VIR_FREE(array[i]); return ret; @@ -228,7 +228,7 @@ testStrndupNegative(const void *opaque ATTRIBUTE_UNUSED) } ret = 0; -cleanup: + cleanup: VIR_FREE(dst); return ret; } diff --git a/tests/viruritest.c b/tests/viruritest.c index 709061a..dbcd877 100644 --- a/tests/viruritest.c +++ b/tests/viruritest.c @@ -130,7 +130,7 @@ static int testURIParse(const void *args) } ret = 0; -cleanup: + cleanup: VIR_FREE(uristr); virURIFree(uri); return ret; diff --git a/tests/virusbmock.c b/tests/virusbmock.c index 1932ec8..8d60664 100644 --- a/tests/virusbmock.c +++ b/tests/virusbmock.c @@ -64,7 +64,7 @@ static char *get_fake_path(const char *real_path) return path; -error: + error: errno = ENOMEM; return NULL; } diff --git a/tests/virusbtest.c b/tests/virusbtest.c index b66c7d0..d08e03b 100644 --- a/tests/virusbtest.c +++ b/tests/virusbtest.c @@ -127,7 +127,7 @@ static int testDeviceFind(const void *opaque) ret = 0; -cleanup: + cleanup: virObjectUnref(devs); virUSBDeviceFree(dev); return ret; @@ -226,7 +226,7 @@ testUSBList(const void *opaque ATTRIBUTE_UNUSED) ret = 0; -cleanup: + cleanup: virObjectUnref(list); virObjectUnref(devlist); virUSBDeviceFree(dev); diff --git a/tests/vmwarevertest.c b/tests/vmwarevertest.c index e981dda..16e48de 100644 --- a/tests/vmwarevertest.c +++ b/tests/vmwarevertest.c @@ -66,7 +66,7 @@ testVerStrParse(const void *data) ret = 0; -cleanup: + cleanup: VIR_FREE(path); VIR_FREE(databuf); return ret; diff --git a/tests/vmx2xmltest.c b/tests/vmx2xmltest.c index 963824a..1d2e012 100644 --- a/tests/vmx2xmltest.c +++ b/tests/vmx2xmltest.c @@ -65,7 +65,7 @@ testCapsInit(void) return; - failure: + failure: virObjectUnref(caps); caps = NULL; } @@ -103,7 +103,7 @@ testCompareFiles(const char *vmx, const char *xml) ret = 0; - cleanup: + cleanup: VIR_FREE(vmxData); VIR_FREE(xmlData); VIR_FREE(formatted); @@ -134,7 +134,7 @@ testCompareHelper(const void *data) ret = testCompareFiles(vmx, xml); - cleanup: + cleanup: VIR_FREE(vmx); VIR_FREE(xml); @@ -178,7 +178,7 @@ testParseVMXFileName(const char *fileName, void *opaque ATTRIBUTE_UNUSED) goto cleanup; } - cleanup: + cleanup: VIR_FREE(copyOfFileName); return src; diff --git a/tests/xmconfigtest.c b/tests/xmconfigtest.c index 5ed16b1..e9fccc3 100644 --- a/tests/xmconfigtest.c +++ b/tests/xmconfigtest.c @@ -188,7 +188,7 @@ testCompareHelper(const void *data) else result = testCompareFormatXML(cfg, xml, info->version); -cleanup: + cleanup: VIR_FREE(xml); VIR_FREE(cfg); diff --git a/tests/xml2sexprtest.c b/tests/xml2sexprtest.c index d6bc51a..1bead7d 100644 --- a/tests/xml2sexprtest.c +++ b/tests/xml2sexprtest.c @@ -88,7 +88,7 @@ testCompareHelper(const void *data) result = testCompareFiles(xml, args, info->version); -cleanup: + cleanup: VIR_FREE(xml); VIR_FREE(args); diff --git a/tests/xml2vmxtest.c b/tests/xml2vmxtest.c index 242b854..058b19f 100644 --- a/tests/xml2vmxtest.c +++ b/tests/xml2vmxtest.c @@ -66,7 +66,7 @@ testCapsInit(void) return; - failure: + failure: virObjectUnref(caps); virObjectUnref(xmlopt); caps = NULL; @@ -115,7 +115,7 @@ testCompareFiles(const char *xml, const char *vmx, int virtualHW_version) result = 0; - failure: + failure: VIR_FREE(xmlData); VIR_FREE(vmxData); VIR_FREE(formatted); @@ -147,7 +147,7 @@ testCompareHelper(const void *data) result = testCompareFiles(xml, vmx, info->virtualHW_version); - cleanup: + cleanup: VIR_FREE(xml); VIR_FREE(vmx); @@ -206,7 +206,7 @@ testFormatVMXFileName(const char *src, void *opaque ATTRIBUTE_UNUSED) success = true; - cleanup: + cleanup: if (! success) { VIR_FREE(absolutePath); } -- 1.8.3.2 -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list