diff options
| author | Lennart Poettering <lennart@poettering.net> | 2014-12-03 22:23:06 +0100 |
|---|---|---|
| committer | Lennart Poettering <lennart@poettering.net> | 2014-12-03 22:23:21 +0100 |
| commit | a98433c05c5cce62a6a17db74e49363a88156486 (patch) | |
| tree | baa8b92a0de5f70292bd5f71d054ee62fd075557 /src/libsystemd | |
| parent | e9140aff750e4f251f5f09b67412fed995fe9c47 (diff) | |
rtnl: make checks for default routes more strict
Also check that the source netmask is 0, not only the destination
netmask.
Diffstat (limited to 'src/libsystemd')
| -rw-r--r-- | src/libsystemd/sd-rtnl/local-addresses.c | 11 | ||||
| -rw-r--r-- | src/libsystemd/sd-rtnl/rtnl-message.c | 15 |
2 files changed, 23 insertions, 3 deletions
diff --git a/src/libsystemd/sd-rtnl/local-addresses.c b/src/libsystemd/sd-rtnl/local-addresses.c index 3ab99420a3..b669f9025b 100644 --- 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 076c822e6f..a2f75471b4 100644 --- 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) { |