diff options
author | Yu Watanabe <watanabe.yu+github@gmail.com> | 2024-02-02 08:08:23 +0100 |
---|---|---|
committer | Yu Watanabe <watanabe.yu+github@gmail.com> | 2024-02-08 03:30:19 +0100 |
commit | 155d7a2c049cf866a0bfde8de371f09dfb3b6f29 (patch) | |
tree | 32e6da140b19cad5d0ee3b9ed703719996cc3c1f /src/network | |
parent | network/ndisc: fix use-of-uninitialized-value on failure path (diff) | |
download | systemd-155d7a2c049cf866a0bfde8de371f09dfb3b6f29.tar.xz systemd-155d7a2c049cf866a0bfde8de371f09dfb3b6f29.zip |
network/ndisc: drop onlink prefix route when on-link flag is zero
Fixes #28435.
Diffstat (limited to 'src/network')
-rw-r--r-- | src/network/networkd-ndisc.c | 72 |
1 files changed, 68 insertions, 4 deletions
diff --git a/src/network/networkd-ndisc.c b/src/network/networkd-ndisc.c index dfcd400a52..808a9f2bae 100644 --- a/src/network/networkd-ndisc.c +++ b/src/network/networkd-ndisc.c @@ -545,6 +545,69 @@ static int ndisc_router_process_onlink_prefix(Link *link, sd_ndisc_router *rt) { return 0; } +static int ndisc_router_drop_onlink_prefix(Link *link, sd_ndisc_router *rt) { + _cleanup_(route_unrefp) Route *route = NULL; + unsigned prefixlen, preference; + struct in6_addr prefix; + usec_t lifetime_usec; + int r; + + assert(link); + assert(link->network); + assert(rt); + + /* RFC 4861 section 6.3.4. + * Note, however, that a Prefix Information option with the on-link flag set to zero conveys no + * information concerning on-link determination and MUST NOT be interpreted to mean that addresses + * covered by the prefix are off-link. The only way to cancel a previous on-link indication is to + * advertise that prefix with the L-bit set and the Lifetime set to zero. */ + + if (!link->network->ipv6_accept_ra_use_onlink_prefix) + return 0; + + r = sd_ndisc_router_prefix_get_valid_lifetime(rt, &lifetime_usec); + if (r < 0) + return log_link_warning_errno(link, r, "Failed to get prefix lifetime: %m"); + + if (lifetime_usec != 0) + return 0; + + r = sd_ndisc_router_prefix_get_address(rt, &prefix); + if (r < 0) + return log_link_warning_errno(link, r, "Failed to get prefix address: %m"); + + r = sd_ndisc_router_prefix_get_prefixlen(rt, &prefixlen); + if (r < 0) + return log_link_warning_errno(link, r, "Failed to get prefix length: %m"); + + /* Prefix Information option does not have preference, hence we use the 'main' preference here */ + r = sd_ndisc_router_get_preference(rt, &preference); + if (r < 0) + return log_link_warning_errno(link, r, "Failed to get router preference from RA: %m"); + + r = route_new(&route); + if (r < 0) + return log_oom(); + + route->family = AF_INET6; + route->dst.in6 = prefix; + route->dst_prefixlen = prefixlen; + route->table = link_get_ipv6_accept_ra_route_table(link); + route->pref = preference; + ndisc_set_route_priority(link, route); + route->protocol = RTPROT_RA; + + r = route_adjust_nexthops(route, link); + if (r < 0) + return r; + + r = route_remove_and_cancel(route, link->manager); + if (r < 0) + return log_link_warning_errno(link, r, "Could not remove prefix route: %m"); + + return 0; +} + static int ndisc_router_process_prefix(Link *link, sd_ndisc_router *rt) { unsigned prefixlen; struct in6_addr a; @@ -584,11 +647,12 @@ static int ndisc_router_process_prefix(Link *link, sd_ndisc_router *rt) { if (r < 0) return log_link_warning_errno(link, r, "Failed to get RA prefix flags: %m"); - if (FLAGS_SET(flags, ND_OPT_PI_FLAG_ONLINK)) { + if (FLAGS_SET(flags, ND_OPT_PI_FLAG_ONLINK)) r = ndisc_router_process_onlink_prefix(link, rt); - if (r < 0) - return r; - } + else + r = ndisc_router_drop_onlink_prefix(link, rt); + if (r < 0) + return r; if (FLAGS_SET(flags, ND_OPT_PI_FLAG_AUTO)) { r = ndisc_router_process_autonomous_prefix(link, rt); |