Explorar el Código

libnetwork/iptables: checkRunning(): use early return

Remove redundant variable declarations, and use an early return instead.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Sebastiaan van Stijn hace 2 años
padre
commit
0921360133
Se han modificado 1 ficheros con 5 adiciones y 7 borrados
  1. 5 7
      libnetwork/iptables/firewalld.go

+ 5 - 7
libnetwork/iptables/firewalld.go

@@ -177,14 +177,12 @@ func OnReloaded(callback func()) {
 
 // Call some remote method to see whether the service is actually running.
 func checkRunning() bool {
-	var zone string
-	var err error
-
-	if connection != nil {
-		err = connection.sysObj.Call(dbusInterface+".getDefaultZone", 0).Store(&zone)
-		return err == nil
+	if connection == nil {
+		return false
 	}
-	return false
+	var zone string
+	err := connection.sysObj.Call(dbusInterface+".getDefaultZone", 0).Store(&zone)
+	return err == nil
 }
 
 // Passthrough method simply passes args through to iptables/ip6tables