networkd: address - add support for broadcast

diff --git a/src/network/networkd-address.c b/src/network/networkd-address.c
index de5566dfdb44b96b50bf749fc2a4667a4f68de07..aa7927ff78b85e10092135fb9a0d4c08d35232fd 100644 (file)
--- a/src/network/networkd-address.c
+++ b/src/network/networkd-address.c
@@ -186,11 +186,7 @@ int address_configure(Address *address, Link *link,
         }
 
         if (address->family == AF_INET) {
         }
 
         if (address->family == AF_INET) {

-                struct in_addr broadcast;
-
-                broadcast.s_addr = address->in_addr.in.s_addr | address->netmask.s_addr;
-
-                r = sd_rtnl_message_append_in_addr(req, IFA_BROADCAST, &broadcast);
+                r = sd_rtnl_message_append_in_addr(req, IFA_BROADCAST, &address->broadcast);

                 if (r < 0) {
                         log_error("Could not append IFA_BROADCAST attribute: %s",
                                   strerror(-r));
                 if (r < 0) {
                         log_error("Could not append IFA_BROADCAST attribute: %s",
                                   strerror(-r));


@@ -253,6 +249,43 @@ int config_parse_dns(const char *unit,
         return 0;
 }
 
         return 0;
 }
 

+int config_parse_broadcast(const char *unit,
+                const char *filename,
+                unsigned line,
+                const char *section,
+                unsigned section_line,
+                const char *lvalue,
+                int ltype,
+                const char *rvalue,
+                void *data,
+                void *userdata) {
+        Network *network = userdata;
+        _cleanup_address_free_ Address *n = NULL;
+        _cleanup_free_ char *address = NULL;
+        int r;
+
+        assert(filename);
+        assert(section);
+        assert(lvalue);
+        assert(rvalue);
+        assert(data);
+
+        r = address_new_static(network, section_line, &n);
+        if (r < 0)
+                return r;
+
+        r = net_parse_inaddr(address, &n->family, &n->broadcast);
+        if (r < 0) {
+                log_syntax(unit, LOG_ERR, filename, line, EINVAL,
+                           "Broadcast is invalid, ignoring assignment: %s", address);
+                return 0;
+        }
+
+        n = NULL;
+
+        return 0;
+}
+

 int config_parse_address(const char *unit,
                 const char *filename,
                 unsigned line,
 int config_parse_address(const char *unit,
                 const char *filename,
                 unsigned line,


@@ -300,7 +333,6 @@ int config_parse_address(const char *unit,
                 }
 
                 n->prefixlen = (unsigned char) i;
                 }
 
                 n->prefixlen = (unsigned char) i;

-                n->netmask.s_addr = htonl(0xfffffffflu >> n->prefixlen);

 
                 address = strndup(rvalue, e - rvalue);
                 if (!address)
 
                 address = strndup(rvalue, e - rvalue);
                 if (!address)


@@ -318,6 +350,10 @@ int config_parse_address(const char *unit,
                 return 0;
         }
 
                 return 0;
         }
 

+        if (n->family == AF_INET && !n->broadcast.s_addr)
+                n->broadcast.s_addr = n->in_addr.in.s_addr |
+                                      htonl(0xfffffffflu >> n->prefixlen);
+

         n = NULL;
 
         return 0;
         n = NULL;
 
         return 0;

diff --git a/src/network/networkd-gperf.gperf b/src/network/networkd-gperf.gperf
index de7280562556b994c983757be5f438a4541e434b..64dab7cf0cf0117179781b0f8bfbfb799359bed9 100644 (file)
--- a/src/network/networkd-gperf.gperf
+++ b/src/network/networkd-gperf.gperf
@@ -29,6 +29,7 @@ Network.Address,           config_parse_address,          0,       0
 Network.Gateway,           config_parse_gateway,          0,       0
 Network.DNS,               config_parse_dns,              0,       offsetof(Network, dns)
 Address.Address,           config_parse_address,          0,       0
 Network.Gateway,           config_parse_gateway,          0,       0
 Network.DNS,               config_parse_dns,              0,       offsetof(Network, dns)
 Address.Address,           config_parse_address,          0,       0

+Address.Broadcast,         config_parse_broadcast,        0,       0

 Address.Label,             config_parse_label,            0,       0
 Route.Gateway,             config_parse_gateway,          0,       0
 Route.Destination,         config_parse_destination,      0,       0
 Address.Label,             config_parse_label,            0,       0
 Route.Gateway,             config_parse_gateway,          0,       0
 Route.Destination,         config_parse_destination,      0,       0

diff --git a/src/network/networkd-link.c b/src/network/networkd-link.c
index 9b5ebeda2072c960d125c5a202d864a519b1b716..dec33e88eec71832bd0f8c899d43fb2087efc7d3 100644 (file)
--- a/src/network/networkd-link.c
+++ b/src/network/networkd-link.c
@@ -540,7 +540,7 @@ static void dhcp_handler(sd_dhcp_client *client, int event, void *userdata) {
                 addr->family = AF_INET;
                 addr->in_addr.in = address;
                 addr->prefixlen = prefixlen;
                 addr->family = AF_INET;
                 addr->in_addr.in = address;
                 addr->prefixlen = prefixlen;

-                addr->netmask = netmask;
+                addr->broadcast.s_addr = address.s_addr | ~netmask.s_addr;

 
                 r = route_new_dynamic(&rt);
                 if (r < 0) {
 
                 r = route_new_dynamic(&rt);
                 if (r < 0) {

diff --git a/src/network/networkd.h b/src/network/networkd.h
index f174abbf5937ba85cb546818c7951068cb5ac39e..01a8a7f681a448a261def0db5483b92f1c4f8b49 100644 (file)
--- a/src/network/networkd.h
+++ b/src/network/networkd.h
@@ -123,7 +123,7 @@ struct Address {
         unsigned char prefixlen;
         char *label;
 
         unsigned char prefixlen;
         char *label;
 

-        struct in_addr netmask;
+        struct in_addr broadcast;

 
         union {
                 struct in_addr in;
 
         union {
                 struct in_addr in;


@@ -305,6 +305,10 @@ int config_parse_address(const char *unit, const char *filename, unsigned line,
                          const char *section, unsigned section_line, const char *lvalue,
                          int ltype, const char *rvalue, void *data, void *userdata);
 
                          const char *section, unsigned section_line, const char *lvalue,
                          int ltype, const char *rvalue, void *data, void *userdata);
 

+int config_parse_broadcast(const char *unit, const char *filename, unsigned line,
+                           const char *section, unsigned section_line, const char *lvalue,
+                           int ltype, const char *rvalue, void *data, void *userdata);
+

 int config_parse_label(const char *unit, const char *filename, unsigned line,
                        const char *section, unsigned section_line, const char *lvalue,
                        int ltype, const char *rvalue, void *data, void *userdata);
 int config_parse_label(const char *unit, const char *filename, unsigned line,
                        const char *section, unsigned section_line, const char *lvalue,
                        int ltype, const char *rvalue, void *data, void *userdata);