vbox: remove unneeded labels

Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
Reviewed-by: Erik Skultety <eskultet@redhat.com>

diff --git a/src/vbox/vbox_MSCOMGlue.c b/src/vbox/vbox_MSCOMGlue.c
index 18a291a0d8df2392f9ac9a075b3110890ba6bcbd..5cc3f5c7c89e6462b398d0cb52497c120e7363be 100644 (file)
--- a/src/vbox/vbox_MSCOMGlue.c
+++ b/src/vbox/vbox_MSCOMGlue.c
@@ -357,24 +357,24 @@ vboxLookupRegistryValue(HKEY key, const char *keyName, const char *valueName)
     if (status != ERROR_SUCCESS) {
         VIR_ERROR(_("Could not query registry value '%s\\%s'"),
                   keyName, valueName);
-        goto cleanup;
+        return NULL;
     }
 
     if (type != REG_SZ) {
         VIR_ERROR(_("Registry value '%s\\%s' has unexpected type"),
                   keyName, valueName);
-        goto cleanup;
+        return NULL;
     }
 
     if (length < 2) {
         VIR_ERROR(_("Registry value '%s\\%s' is too short"),
                   keyName, valueName);
-        goto cleanup;
+        return NULL;
     }
 
     /* +1 for the null-terminator if it's missing */
     if (VIR_ALLOC_N(value, length + 1) < 0)
-        goto cleanup;
+        return NULL;
 
     status = RegQueryValueEx(key, valueName, NULL, NULL, (LPBYTE)value, &length);
 
@@ -382,13 +382,12 @@ vboxLookupRegistryValue(HKEY key, const char *keyName, const char *valueName)
         VIR_FREE(value);
         VIR_ERROR(_("Could not query registry value '%s\\%s'"),
                   keyName, valueName);
-        goto cleanup;
+        return NULL;
     }
 
     if (value[length - 1] != '\0')
         value[length] = '\0';
 
- cleanup:
     return value;
 }
 

diff --git a/src/vbox/vbox_snapshot_conf.c b/src/vbox/vbox_snapshot_conf.c
index db6c389a64e82dde9667fbc2e8db8cf6109a72dd..4cd4e3a983a556003eb0643aea3ce164f3103ef5 100644 (file)
--- a/src/vbox/vbox_snapshot_conf.c
+++ b/src/vbox/vbox_snapshot_conf.c
@@ -1228,23 +1228,20 @@ virVBoxSnapshotConfIsCurrentSnapshot(virVBoxSnapshotConfMachinePtr machine,
     if (machine == NULL) {
         virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                        _("Machine is null"));
-        goto cleanup;
+        return 0;
     }
     if (snapshotName == NULL) {
         virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
                        _("snapshotName is null"));
-        goto cleanup;
+        return 0;
     }
     snapshot = virVBoxSnapshotConfSnapshotByName(machine->snapshot, snapshotName);
     if (snapshot == NULL) {
         virReportError(VIR_ERR_NO_DOMAIN_SNAPSHOT,
                        _("Unable to find the snapshot %s"), snapshotName);
-        goto cleanup;
+        return 0;
     }
     return STREQ(snapshot->uuid, machine->currentSnapshot);
-
- cleanup:
-    return 0;
 }
 
 /*