diff options
author | Eric Dumazet <edumazet@google.com> | 2024-02-28 14:54:29 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2024-03-01 09:42:31 +0100 |
commit | e0bb2675fea2783c45bb95d74f00c55156720863 (patch) | |
tree | 0f7a1c6c8a215727d5dfd5ccc5c86fd9ca717b31 /net/ipv6/ndisc.c | |
parent | ipv6: annotate data-races around cnf.mtu6 (diff) | |
download | linux-e0bb2675fea2783c45bb95d74f00c55156720863.tar.xz linux-e0bb2675fea2783c45bb95d74f00c55156720863.zip |
ipv6: annotate data-races around cnf.hop_limit
idev->cnf.hop_limit and net->ipv6.devconf_all->hop_limit
might be read locklessly, add appropriate READ_ONCE()
and WRITE_ONCE() annotations.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Acked-by: Florian Westphal <fw@strlen.de> # for netfilter parts
Reviewed-by: Jiri Pirko <jiri@nvidia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/ndisc.c')
-rw-r--r-- | net/ipv6/ndisc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv6/ndisc.c b/net/ipv6/ndisc.c index e96d79cd34d2..9c9c31268432 100644 --- a/net/ipv6/ndisc.c +++ b/net/ipv6/ndisc.c @@ -1423,7 +1423,7 @@ static enum skb_drop_reason ndisc_router_discovery(struct sk_buff *skb) if (in6_dev->cnf.accept_ra_min_hop_limit < 256 && ra_msg->icmph.icmp6_hop_limit) { if (in6_dev->cnf.accept_ra_min_hop_limit <= ra_msg->icmph.icmp6_hop_limit) { - in6_dev->cnf.hop_limit = ra_msg->icmph.icmp6_hop_limit; + WRITE_ONCE(in6_dev->cnf.hop_limit, ra_msg->icmph.icmp6_hop_limit); fib6_metric_set(rt, RTAX_HOPLIMIT, ra_msg->icmph.icmp6_hop_limit); } else { |