From: Yu Watanabe <watanabe.yu+github@gmail.com>
Date: Thu, 7 Sep 2023 19:16:31 +0000 (+0900)
Subject: network: drop duplicated address_set_broadcast()
X-Git-Tag: v255-rc1~549^2~1
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=0ae8ff50ac9239570f255eaa34ae972b62dc5183;p=thirdparty%2Fsystemd.git

network: drop duplicated address_set_broadcast()

It will be called later in link_request_address().
---

diff --git a/src/network/networkd-address.c b/src/network/networkd-address.c
index 58eebc63c6e..048f288df88 100644
--- a/src/network/networkd-address.c
+++ b/src/network/networkd-address.c
@@ -240,7 +240,7 @@ void link_mark_addresses(Link *link, NetworkConfigSource source) {
         }
 }
 
-int address_get_broadcast(const Address *a, Link *link, struct in_addr *ret) {
+static int address_get_broadcast(const Address *a, Link *link, struct in_addr *ret) {
         struct in_addr b_addr = {};
 
         assert(a);
@@ -291,6 +291,11 @@ finalize:
         return in4_addr_is_set(&b_addr);
 }
 
+static void address_set_broadcast(Address *a, Link *link) {
+        assert(a);
+        assert_se(address_get_broadcast(a, link, &a->broadcast) >= 0);
+}
+
 static void address_set_cinfo(Manager *m, const Address *a, struct ifa_cacheinfo *cinfo) {
         usec_t now_usec;
 
diff --git a/src/network/networkd-address.h b/src/network/networkd-address.h
index 4be8f92ef08..87c7d364868 100644
--- a/src/network/networkd-address.h
+++ b/src/network/networkd-address.h
@@ -83,11 +83,6 @@ int address_remove_and_drop(Address *address);
 int address_dup(const Address *src, Address **ret);
 bool address_is_ready(const Address *a);
 bool link_check_addresses_ready(Link *link, NetworkConfigSource source);
-int address_get_broadcast(const Address *a, Link *link, struct in_addr *ret);
-static inline void address_set_broadcast(Address *a, Link *link) {
-        assert(a);
-        assert_se(address_get_broadcast(a, link, &a->broadcast) >= 0);
-}
 
 DEFINE_SECTION_CLEANUP_FUNCTIONS(Address, address_free);
 
diff --git a/src/network/networkd-dhcp-server.c b/src/network/networkd-dhcp-server.c
index fedc2e4f04d..c1d9fbc17cf 100644
--- a/src/network/networkd-dhcp-server.c
+++ b/src/network/networkd-dhcp-server.c
@@ -107,7 +107,6 @@ int link_request_dhcp_server_address(Link *link) {
         address->family = AF_INET;
         address->in_addr.in = link->network->dhcp_server_address;
         address->prefixlen = link->network->dhcp_server_address_prefixlen;
-        address_set_broadcast(address, link);
 
         if (address_get_harder(link, address, &existing) >= 0 &&
             (address_exists(existing) || address_is_requesting(existing)) &&
diff --git a/src/network/networkd-dhcp4.c b/src/network/networkd-dhcp4.c
index a3f28035d76..f14e62588fe 100644
--- a/src/network/networkd-dhcp4.c
+++ b/src/network/networkd-dhcp4.c
@@ -959,7 +959,6 @@ static int dhcp4_request_address(Link *link, bool announce) {
         r = sd_dhcp_lease_get_broadcast(link->dhcp_lease, &addr->broadcast);
         if (r < 0 && r != -ENODATA)
                 return log_link_warning_errno(link, r, "DHCP: failed to get broadcast address: %m");
-        address_set_broadcast(addr, link);
         SET_FLAG(addr->flags, IFA_F_NOPREFIXROUTE, !link_prefixroute(link));
         addr->route_metric = link->network->dhcp_route_metric;
         addr->duplicate_address_detection = link->network->dhcp_send_decline ? ADDRESS_FAMILY_IPV4 : ADDRESS_FAMILY_NO;
diff --git a/src/network/networkd-ipv4ll.c b/src/network/networkd-ipv4ll.c
index ee78fffdfb3..c35738252bb 100644
--- a/src/network/networkd-ipv4ll.c
+++ b/src/network/networkd-ipv4ll.c
@@ -50,7 +50,6 @@ static int address_new_from_ipv4ll(Link *link, Address **ret) {
         address->prefixlen = 16;
         address->scope = RT_SCOPE_LINK;
         address->route_metric = IPV4LL_ROUTE_METRIC;
-        address_set_broadcast(address, link);
 
         *ret = TAKE_PTR(address);
         return 0;