Remove redundant labels

Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Martin Kletzander <mkletzan@redhat.com>
Reviewed-by: Pavel Hrdina <phrdina@redhat.com>

diff --git a/src/conf/cpu_conf.c b/src/conf/cpu_conf.c
index 44341205575cf5277bedc9a4bf497d34633cc0ce..1674cd695781214b8b47f399c1b476b72f610eb4 100644 (file)
--- a/src/conf/cpu_conf.c
+++ b/src/conf/cpu_conf.c
@@ -283,23 +283,19 @@ virCPUDefParseXMLString(const char *xml,
 {
     g_autoptr(xmlDoc) doc = NULL;
     g_autoptr(xmlXPathContext) ctxt = NULL;
-    int ret = -1;
 
     if (!xml) {
         virReportError(VIR_ERR_INVALID_ARG, "%s", _("missing CPU definition"));
-        goto cleanup;
+        return -1;
     }
 
     if (!(doc = virXMLParseStringCtxt(xml, _("(CPU_definition)"), &ctxt)))
-        goto cleanup;
+        return -1;
 
     if (virCPUDefParseXML(ctxt, NULL, type, cpu, validateXML) < 0)
-        goto cleanup;
-
-    ret = 0;
+        return -1;
 
- cleanup:
-    return ret;
+    return 0;
 }
 
 

diff --git a/src/conf/virnetworkportdef.c b/src/conf/virnetworkportdef.c
index 5e916ab2b8a151635ef48885b57c2c7d72b793a6..c1749eebe022672d00c7261198e5bfbb29386270 100644 (file)
--- a/src/conf/virnetworkportdef.c
+++ b/src/conf/virnetworkportdef.c
@@ -269,23 +269,19 @@ virNetworkPortDefParseNode(xmlDocPtr xml,
                            xmlNodePtr root)
 {
     g_autoptr(xmlXPathContext) ctxt = NULL;
-    virNetworkPortDef *def = NULL;
 
     if (STRNEQ((const char *)root->name, "networkport")) {
         virReportError(VIR_ERR_XML_ERROR,
                        "%s",
                        _("unknown root element for network port"));
-        goto cleanup;
+        return NULL;
     }
 
     if (!(ctxt = virXMLXPathContextNew(xml)))
-        goto cleanup;
+        return NULL;
 
     ctxt->node = root;
-    def = virNetworkPortDefParseXML(ctxt);
-
- cleanup:
-    return def;
+    return virNetworkPortDefParseXML(ctxt);
 }
 
 

diff --git a/src/conf/virnwfilterbindingdef.c b/src/conf/virnwfilterbindingdef.c
index 3428b058419ccb11a7e58745d272217b065e6b9f..5f671030bb657666b1d84448282ab78d4def0274 100644 (file)
--- a/src/conf/virnwfilterbindingdef.c
+++ b/src/conf/virnwfilterbindingdef.c
@@ -160,23 +160,19 @@ virNWFilterBindingDefParseNode(xmlDocPtr xml,
                                xmlNodePtr root)
 {
     g_autoptr(xmlXPathContext) ctxt = NULL;
-    virNWFilterBindingDef *def = NULL;
 
     if (STRNEQ((const char *)root->name, "filterbinding")) {
         virReportError(VIR_ERR_XML_ERROR,
                        "%s",
                        _("unknown root element for nwfilter binding"));
-        goto cleanup;
+        return NULL;
     }
 
     if (!(ctxt = virXMLXPathContextNew(xml)))
-        goto cleanup;
+        return NULL;
 
     ctxt->node = root;
-    def = virNWFilterBindingDefParseXML(ctxt);
-
- cleanup:
-    return def;
+    return virNWFilterBindingDefParseXML(ctxt);
 }
 
 

diff --git a/src/conf/virnwfilterbindingobj.c b/src/conf/virnwfilterbindingobj.c
index 279e334d3f2d8e607b90116f2357a41d03073d4e..29fbb63b5f977b245ea6f4f43dd47414a1677658 100644 (file)
--- a/src/conf/virnwfilterbindingobj.c
+++ b/src/conf/virnwfilterbindingobj.c
@@ -237,23 +237,19 @@ virNWFilterBindingObjParseNode(xmlDocPtr doc,
                                xmlNodePtr root)
 {
     g_autoptr(xmlXPathContext) ctxt = NULL;
-    virNWFilterBindingObj *obj = NULL;
 
     if (STRNEQ((const char *)root->name, "filterbindingstatus")) {
         virReportError(VIR_ERR_XML_ERROR,
                        _("unknown root element '%s' for filter binding"),
                        root->name);
-        goto cleanup;
+        return NULL;
     }
 
     if (!(ctxt = virXMLXPathContextNew(doc)))
-        goto cleanup;
+        return NULL;
 
     ctxt->node = root;
-    obj = virNWFilterBindingObjParseXML(doc, ctxt);
-
- cleanup:
-    return obj;
+    return virNWFilterBindingObjParseXML(doc, ctxt);
 }
 
 

diff --git a/src/conf/virsavecookie.c b/src/conf/virsavecookie.c
index 3bb97246af265beef3f321df604d948140a86c10..6cb7fafb1f15acd958e7e0f9fe3060bc728c8720 100644 (file)
--- a/src/conf/virsavecookie.c
+++ b/src/conf/virsavecookie.c
@@ -81,22 +81,16 @@ virSaveCookieParseString(const char *xml,
 {
     g_autoptr(xmlDoc) doc = NULL;
     g_autoptr(xmlXPathContext) ctxt = NULL;
-    int ret = -1;
 
     *obj = NULL;
 
-    if (!xml) {
-        ret = 0;
-        goto cleanup;
-    }
+    if (!xml)
+        return 0;
 
     if (!(doc = virXMLParseStringCtxt(xml, _("(save cookie)"), &ctxt)))
-        goto cleanup;
-
-    ret = virSaveCookieParseNode(ctxt, obj, saveCookie);
+        return -1;
 
- cleanup:
-    return ret;
+    return virSaveCookieParseNode(ctxt, obj, saveCookie);
 }
 
 

diff --git a/src/conf/virstorageobj.c b/src/conf/virstorageobj.c
index 1f6fedba13b9af0f237667709c86ca38c2242a8c..f906c5b141cc617996ea0b485d576908d914b951 100644 (file)
--- a/src/conf/virstorageobj.c
+++ b/src/conf/virstorageobj.c
@@ -1652,30 +1652,30 @@ virStoragePoolObjLoadState(virStoragePoolObjList *pools,
         return NULL;
 
     if (!(xml = virXMLParseCtxt(stateFile, NULL, _("(pool state)"), &ctxt)))
-        goto cleanup;
+        return NULL;
 
     if (!(node = virXPathNode("//pool", ctxt))) {
         virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                        _("Could not find any 'pool' element in state file"));
-        goto cleanup;
+        return NULL;
     }
 
     ctxt->node = node;
     if (!(def = virStoragePoolDefParseXML(ctxt)))
-        goto cleanup;
+        return NULL;
 
     if (STRNEQ(name, def->name)) {
         virReportError(VIR_ERR_INTERNAL_ERROR,
                        _("Storage pool state file '%s' does not match "
                          "pool name '%s'"),
                        stateFile, def->name);
-        goto cleanup;
+        return NULL;
     }
 
     /* create the object */
     if (!(obj = virStoragePoolObjListAdd(pools, def,
                                          VIR_STORAGE_POOL_OBJ_LIST_ADD_CHECK_LIVE)))
-        goto cleanup;
+        return NULL;
     def = NULL;
 
     /* XXX: future handling of some additional useful status data,
@@ -1684,8 +1684,6 @@ virStoragePoolObjLoadState(virStoragePoolObjList *pools,
      */
 
     obj->active = true;
-
- cleanup:
     return obj;
 }
 

diff --git a/src/libxl/libxl_migration.c b/src/libxl/libxl_migration.c
index a2c2b6dbdea439968dedb343e7d8e8cbd320941d..aa719a19d2c114931b80e179ce1337867a640b98 100644 (file)
--- a/src/libxl/libxl_migration.c
+++ b/src/libxl/libxl_migration.c
@@ -151,7 +151,6 @@ libxlMigrationEatCookie(const char *cookiein,
     g_autoptr(xmlDoc) doc = NULL;
     g_autoptr(xmlXPathContext) ctxt = NULL;
     g_autofree char *uuidstr = NULL;
-    int ret = -1;
 
     /*
      * Assume a legacy (V1) migration stream if request came from a
@@ -209,14 +208,11 @@ libxlMigrationEatCookie(const char *cookiein,
     }
 
     *migout = mig;
-    ret = 0;
-    goto cleanup;
+    return 0;
 
  error:
     libxlMigrationCookieFree(mig);
-
- cleanup:
-    return ret;
+    return -1;
 }
 
 static void

diff --git a/src/qemu/qemu_migration_cookie.c b/src/qemu/qemu_migration_cookie.c
index 106897cacd43295113c805ddd0f86cefcfa899b8..b67728f9c0fb4dd4e2341841ca3dcded30974d9a 100644 (file)
--- a/src/qemu/qemu_migration_cookie.c
+++ b/src/qemu/qemu_migration_cookie.c
@@ -1433,18 +1433,13 @@ qemuMigrationCookieXMLParseStr(qemuMigrationCookie *mig,
 {
     g_autoptr(xmlDoc) doc = NULL;
     g_autoptr(xmlXPathContext) ctxt = NULL;
-    int ret = -1;
 
     VIR_DEBUG("xml=%s", NULLSTR(xml));
 
     if (!(doc = virXMLParseStringCtxt(xml, _("(qemu_migration_cookie)"), &ctxt)))
-        goto cleanup;
-
-    ret = qemuMigrationCookieXMLParse(mig, driver, qemuCaps, doc, ctxt, flags);
-
- cleanup:
+        return -1;
 
-    return ret;
+    return qemuMigrationCookieXMLParse(mig, driver, qemuCaps, doc, ctxt, flags);
 }
 
 

diff --git a/src/security/virt-aa-helper.c b/src/security/virt-aa-helper.c
index c0bbd7414ae289dd53922655c4984df432c87813..77b2307594f98e5c7f37de45a8ea7cb9b4baf268 100644 (file)
--- a/src/security/virt-aa-helper.c
+++ b/src/security/virt-aa-helper.c
@@ -567,34 +567,33 @@ verify_xpath_context(xmlXPathContextPtr ctxt)
 static int
 caps_mockup(vahControl * ctl, const char *xmlStr)
 {
-    int rc = -1;
     g_autoptr(xmlDoc) xml = NULL;
     g_autoptr(xmlXPathContext) ctxt = NULL;
     char *arch;
 
     if (!(xml = virXMLParseStringCtxt(xmlStr, _("(domain_definition)"),
                                       &ctxt))) {
-        goto cleanup;
+        return -1;
     }
 
     if (!virXMLNodeNameEqual(ctxt->node, "domain")) {
         vah_error(NULL, 0, _("unexpected root element, expecting <domain>"));
-        goto cleanup;
+        return -1;
     }
 
     /* Quick sanity check for some required elements */
     if (verify_xpath_context(ctxt) != 0)
-        goto cleanup;
+        return -1;
 
     ctl->virtType = virXPathString("string(./@type)", ctxt);
     if (!ctl->virtType) {
         vah_error(ctl, 0, _("domain type is not defined"));
-        goto cleanup;
+        return -1;
     }
     ctl->os = virXPathString("string(./os/type[1])", ctxt);
     if (!ctl->os) {
         vah_error(ctl, 0, _("os.type is not defined"));
-        goto cleanup;
+        return -1;
     }
     arch = virXPathString("string(./os/type[1]/@arch)", ctxt);
     if (!arch) {
@@ -604,11 +603,7 @@ caps_mockup(vahControl * ctl, const char *xmlStr)
         VIR_FREE(arch);
     }
 
-    rc = 0;
-
- cleanup:
-
-    return rc;
+    return 0;
 }
 
 virDomainDefParserConfig virAAHelperDomainDefParserConfig = {

diff --git a/src/storage/storage_util.c b/src/storage/storage_util.c
index 7ee736535ce22c51a1e890f467e2a77276e87966..53792765ff707ef61e8bc4de3c6e1cab800a159e 100644 (file)
--- a/src/storage/storage_util.c
+++ b/src/storage/storage_util.c
@@ -2897,26 +2897,25 @@ virStorageUtilGlusterExtractPoolSources(const char *host,
     virStoragePoolSource *src = NULL;
     size_t i;
     int nnodes;
-    int ret = -1;
     g_autofree xmlNodePtr *nodes = NULL;
     g_autofree char *volname = NULL;
 
     if (!(doc = virXMLParseStringCtxt(xml, _("(gluster_cli_output)"), &ctxt)))
-        goto cleanup;
+        return -1;
 
     if ((nnodes = virXPathNodeSet("//volumes/volume", ctxt, &nodes)) < 0)
-        goto cleanup;
+        return -1;
 
     for (i = 0; i < nnodes; i++) {
         ctxt->node = nodes[i];
 
         if (!(src = virStoragePoolSourceListNewSource(list)))
-            goto cleanup;
+            return -1;
 
         if (!(volname = virXPathString("string(./name)", ctxt))) {
             virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                            _("failed to extract gluster volume name"));
-            goto cleanup;
+            return -1;
         }
 
         if (pooltype == VIR_STORAGE_POOL_NETFS) {
@@ -2928,7 +2927,7 @@ virStorageUtilGlusterExtractPoolSources(const char *host,
         } else {
             virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                            _("unsupported gluster lookup"));
-            goto cleanup;
+            return -1;
         }
 
         src->hosts = g_new0(virStoragePoolSourceHost, 1);
@@ -2937,11 +2936,7 @@ virStorageUtilGlusterExtractPoolSources(const char *host,
         src->hosts[0].name = g_strdup(host);
     }
 
-    ret = nnodes;
-
- cleanup:
-
-    return ret;
+    return nnodes;
 }
 
 

diff --git a/src/vz/vz_driver.c b/src/vz/vz_driver.c
index 9eba9d47fb0a040d26245cc317b686beab6d79e2..a1d05f834b42989b2735117e67739e1dfea58240 100644 (file)
--- a/src/vz/vz_driver.c
+++ b/src/vz/vz_driver.c
@@ -2837,13 +2837,11 @@ vzEatCookie(const char *cookiein, int cookieinlen, unsigned int flags)
         goto error;
     }
 
- cleanup:
     return mig;
 
  error:
     vzMigrationCookieFree(mig);
-    mig = NULL;
-    goto cleanup;
+    return NULL;
 }
 
 #define VZ_MIGRATION_FLAGS         (VIR_MIGRATE_PAUSED | \

diff --git a/tests/cputest.c b/tests/cputest.c
index 14606f586404e490de66f9d31c665854a28bb25c..7816de87f79bc7557e3ebbbe9cdc2e97fba507ff 100644 (file)
--- a/tests/cputest.c
+++ b/tests/cputest.c
@@ -79,11 +79,10 @@ cpuTestLoadXML(virArch arch, const char *name)
                           virArchToString(arch), name);
 
     if (!(doc = virXMLParseFileCtxt(xml, &ctxt)))
-        goto cleanup;
+        return NULL;
 
     virCPUDefParseXML(ctxt, NULL, VIR_CPU_TYPE_AUTO, &cpu, false);
 
- cleanup:
     return cpu;
 }
 
@@ -105,12 +104,12 @@ cpuTestLoadMultiXML(virArch arch,
                           virArchToString(arch), name);
 
     if (!(doc = virXMLParseFileCtxt(xml, &ctxt)))
-        goto cleanup;
+        return NULL;
 
     n = virXPathNodeSet("/cpuTest/cpu", ctxt, &nodes);
     if (n <= 0) {
         fprintf(stderr, "\nNo /cpuTest/cpu elements found in %s\n", xml);
-        goto cleanup;
+        return NULL;
     }
 
     cpus = g_new0(virCPUDef *, n);
@@ -119,19 +118,18 @@ cpuTestLoadMultiXML(virArch arch,
         ctxt->node = nodes[i];
         if (virCPUDefParseXML(ctxt, NULL, VIR_CPU_TYPE_HOST, &cpus[i],
                               false) < 0)
-            goto cleanup_cpus;
+            goto error;
     }
 
     *count = n;
 
- cleanup:
     return cpus;
 
- cleanup_cpus:
+ error:
     for (i = 0; i < n; i++)
         virCPUDefFree(cpus[i]);
     VIR_FREE(cpus);
-    goto cleanup;
+    return NULL;
 }
 
 

diff --git a/tests/metadatatest.c b/tests/metadatatest.c
index 2c560c27c1d33bbe913effef343149ab28821909..e565383ce2ab74f3eaa3ffc9832c72e1ed8d5cff 100644 (file)
--- a/tests/metadatatest.c
+++ b/tests/metadatatest.c
@@ -61,22 +61,17 @@ getMetadataFromXML(virDomainPtr dom)
     xmlNodePtr node;
 
     g_autofree char *xml = NULL;
-    char *ret = NULL;
 
     if (!(xml = virDomainGetXMLDesc(dom, 0)))
-        goto cleanup;
+        return NULL;
 
     if (!(doc = virXMLParseStringCtxt(xml, "(domain_definition)", &ctxt)))
-        goto cleanup;
+        return NULL;
 
     if (!(node = virXPathNode("//metadata/*", ctxt)))
-        goto cleanup;
-
-    ret = virXMLNodeToString(node->doc, node);
+        return NULL;
 
- cleanup:
-
-    return ret;
+    return virXMLNodeToString(node->doc, node);
 }