summaryrefslogtreecommitdiffstats
path: root/src/network
diff options
context:
space:
mode:
authorYu Watanabe <watanabe.yu+github@gmail.com>2024-02-02 08:08:23 +0100
committerYu Watanabe <watanabe.yu+github@gmail.com>2024-02-08 03:30:19 +0100
commit155d7a2c049cf866a0bfde8de371f09dfb3b6f29 (patch)
tree32e6da140b19cad5d0ee3b9ed703719996cc3c1f /src/network
parentnetwork/ndisc: fix use-of-uninitialized-value on failure path (diff)
downloadsystemd-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.c72
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);