rtnl: make checks for default routes more strict

Also check that the source netmask is 0, not only the destination
netmask.

diff --git a/src/libsystemd/sd-rtnl/local-addresses.c b/src/libsystemd/sd-rtnl/local-addresses.c
index 3ab99420a375aec78823986362849e7a364ca30a..b669f9025ba428d9d5861712872cb8ae071569ad 100644 (file)
--- a/src/libsystemd/sd-rtnl/local-addresses.c
+++ b/src/libsystemd/sd-rtnl/local-addresses.c
@@ -197,7 +197,7 @@ int local_gateways(sd_rtnl *context, int ifindex, struct local_address **ret) {
         for (m = reply; m; m = sd_rtnl_message_next(m)) {
                 struct local_address *a;
                 uint16_t type;
-                unsigned char dst_len;
+                unsigned char dst_len, src_len;
                 uint32_t ifi;
 
                 r = sd_rtnl_message_get_errno(m);
@@ -211,14 +211,19 @@ int local_gateways(sd_rtnl *context, int ifindex, struct local_address **ret) {
                 if (type != RTM_NEWROUTE)
                         continue;
 
+                /* We only care for default routes */
                 r = sd_rtnl_message_route_get_dst_len(m, &dst_len);
                 if (r < 0)
                         return r;
-
-                /* We only care for default routes */
                 if (dst_len != 0)
                         continue;
 
+                r = sd_rtnl_message_route_get_src_len(m, &src_len);
+                if (r < 0)
+                        return r;
+                if (src_len != 0)
+                        continue;
+
                 r = sd_rtnl_message_read_u32(m, RTA_OIF, &ifi);
                 if (r < 0)
                         return r;

diff --git a/src/libsystemd/sd-rtnl/rtnl-message.c b/src/libsystemd/sd-rtnl/rtnl-message.c
index 076c822e6fcb135724d6ceda0c6e850951552d0f..a2f75471b4cff69e2aaa4da844ed3346f5038751 100644 (file)
--- a/src/libsystemd/sd-rtnl/rtnl-message.c
+++ b/src/libsystemd/sd-rtnl/rtnl-message.c
@@ -158,6 +158,21 @@ int sd_rtnl_message_route_get_dst_len(sd_rtnl_message *m, unsigned char *dst_len
         return 0;
 }
 
+int sd_rtnl_message_route_get_src_len(sd_rtnl_message *m, unsigned char *src_len) {
+        struct rtmsg *rtm;
+
+        assert_return(m, -EINVAL);
+        assert_return(m->hdr, -EINVAL);
+        assert_return(rtnl_message_type_is_route(m->hdr->nlmsg_type), -EINVAL);
+        assert_return(src_len, -EINVAL);
+
+        rtm = NLMSG_DATA(m->hdr);
+
+        *src_len = rtm->rtm_src_len;
+
+        return 0;
+}
+
 int sd_rtnl_message_new_route(sd_rtnl *rtnl, sd_rtnl_message **ret,
                               uint16_t nlmsg_type, int rtm_family,
                               unsigned char rtm_protocol) {

diff --git a/src/systemd/sd-rtnl.h b/src/systemd/sd-rtnl.h
index 554fc8bbfe3634014a2788254415cb51f0dfdf09..bf1dde4008058b921e28dbcf10af47e2e04bf4b7 100644 (file)
--- a/src/systemd/sd-rtnl.h
+++ b/src/systemd/sd-rtnl.h
@@ -105,6 +105,7 @@ int sd_rtnl_message_route_set_dst_prefixlen(sd_rtnl_message *m, unsigned char pr
 int sd_rtnl_message_route_set_scope(sd_rtnl_message *m, unsigned char scope);
 int sd_rtnl_message_route_get_family(sd_rtnl_message *m, int *family);
 int sd_rtnl_message_route_get_dst_len(sd_rtnl_message *m, unsigned char *dst_len);
+int sd_rtnl_message_route_get_src_len(sd_rtnl_message *m, unsigned char *src_len);
 
 int sd_rtnl_message_neigh_get_family(sd_rtnl_message *m, int *family);
 int sd_rtnl_message_neigh_get_ifindex(sd_rtnl_message *m, int *family);