Parcourir la source

libnetwork/drivers/bridge: link.Enable: don't register reload on error

Only register a reload function if we actually managed to enable the link.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Sebastiaan van Stijn il y a 1 an
Parent
commit
8b6203b613
1 fichiers modifiés avec 5 ajouts et 2 suppressions
  1. 5 2
      libnetwork/drivers/bridge/link.go

+ 5 - 2
libnetwork/drivers/bridge/link.go

@@ -36,9 +36,12 @@ func (l *link) Enable() error {
 	linkFunction := func() error {
 		return linkContainers(iptables.Append, l.parentIP, l.childIP, l.ports, l.bridge, false)
 	}
+	if err := linkFunction(); err != nil {
+		return err
+	}
 
-	iptables.OnReloaded(func() { linkFunction() })
-	return linkFunction()
+	iptables.OnReloaded(func() { _ = linkFunction() })
+	return nil
 }
 
 func (l *link) Disable() {