]> xenbits.xensource.com Git - libvirt.git/commitdiff
networkStartNetworkVirtual: Dissolve 'err1' label in 'error'
authorMichal Privoznik <mprivozn@redhat.com>
Wed, 24 Apr 2019 07:26:15 +0000 (09:26 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Tue, 21 May 2019 11:52:19 +0000 (13:52 +0200)
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Erik Skultety <eskultet@redhat.com>
src/network/bridge_driver.c

index 4c920d1fdd5eb0a85f05597c9b3e1f5738098102..59c9780a7e83cae7a47695e363f056a4a9a44dc4 100644 (file)
@@ -2387,7 +2387,7 @@ networkStartNetworkVirtual(virNetworkDriverStatePtr driver,
     if (!(macMapFile = virMacMapFileName(driver->dnsmasqStateDir,
                                          def->bridge)) ||
         !(macmap = virMacMapNew(macMapFile)))
-        goto err1;
+        goto error;
 
     virNetworkObjSetMacMap(obj, macmap);
     macmap = NULL;
@@ -2398,21 +2398,21 @@ networkStartNetworkVirtual(virNetworkDriverStatePtr driver,
      * expects milliseconds
      */
     if (virNetDevBridgeSetSTPDelay(def->bridge, def->delay * 1000) < 0)
-        goto err1;
+        goto error;
 
     if (virNetDevBridgeSetSTP(def->bridge, def->stp ? true : false) < 0)
-        goto err1;
+        goto error;
 
     /* Disable IPv6 on the bridge if there are no IPv6 addresses
      * defined, and set other IPv6 sysctl tunables appropriately.
      */
     if (networkSetIPv6Sysctls(obj) < 0)
-        goto err1;
+        goto error;
 
     /* Add "once per network" rules */
     if (def->forward.type != VIR_NETWORK_FORWARD_OPEN &&
         networkAddFirewallRules(def) < 0)
-        goto err1;
+        goto error;
 
     firewalRulesAdded = true;
 
@@ -2518,10 +2518,6 @@ networkStartNetworkVirtual(virNetworkDriverStatePtr driver,
         def->forward.type != VIR_NETWORK_FORWARD_OPEN)
         networkRemoveFirewallRules(def);
 
- err1:
-    if (!save_err)
-        virErrorPreserveLast(&save_err);
-
     if (macTapIfName) {
         VIR_FORCE_CLOSE(tapfd);
         ignore_value(virNetDevTapDelete(macTapIfName, NULL));