network: drop unnecessary conditions

sd_ipv4acd_stop() and sd_ipv4ll_stop() are idempotent.

diff --git a/src/network/networkd-address.c b/src/network/networkd-address.c
index d006199ab0bf53ba3e60981bab063bae3d5241c8..0b269a1ba1ec6066e2e4759c4221c0da892aa69d 100644 (file)
--- a/src/network/networkd-address.c
+++ b/src/network/networkd-address.c
@@ -1402,11 +1402,9 @@ static int ipv4_dad_update_mac_one(Address *address) {
 
         running = sd_ipv4acd_is_running(address->acd);
 
-        if (running) {
-                r = sd_ipv4acd_stop(address->acd);
-                if (r < 0)
-                        return r;
-        }
+        r = sd_ipv4acd_stop(address->acd);
+        if (r < 0)
+                return r;
 
         r = sd_ipv4acd_set_mac(address->acd, &address->link->mac);
         if (r < 0)

diff --git a/src/network/networkd-ipv4ll.c b/src/network/networkd-ipv4ll.c
index 03f29e3b667fe86cfb01036b2a29ee88bab56b42..3be395e1ada6989ef9de339afc85073240660256 100644 (file)
--- a/src/network/networkd-ipv4ll.c
+++ b/src/network/networkd-ipv4ll.c
@@ -207,11 +207,9 @@ int ipv4ll_update_mac(Link *link) {
 
         restart = sd_ipv4ll_is_running(link->ipv4ll) > 0;
 
-        if (restart) {
-                r = sd_ipv4ll_stop(link->ipv4ll);
-                if (r < 0)
-                        return r;
-        }
+        r = sd_ipv4ll_stop(link->ipv4ll);
+        if (r < 0)
+                return r;
 
         r = sd_ipv4ll_set_mac(link->ipv4ll, &link->mac);
         if (r < 0)

diff --git a/src/network/networkd-radv.c b/src/network/networkd-radv.c
index 1167f2865f61d44202cae9499fa5f00daa2997b1..bb80e91b6c1e289b11b358ff681c2c4f7894041a 100644 (file)
--- a/src/network/networkd-radv.c
+++ b/src/network/networkd-radv.c
@@ -697,11 +697,9 @@ int radv_update_mac(Link *link) {
 
         restart = sd_radv_is_running(link->radv);
 
-        if (restart) {
-                r = sd_radv_stop(link->radv);
-                if (r < 0)
-                        return r;
-        }
+        r = sd_radv_stop(link->radv);
+        if (r < 0)
+                return r;
 
         r = sd_radv_set_mac(link->radv, &link->mac);
         if (r < 0)