]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
ndisc: ndisc_send_redirect() must use dev_get_by_index_rcu()
authorEric Dumazet <edumazet@google.com>
Fri, 7 Feb 2025 13:58:33 +0000 (13:58 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 13 Mar 2025 11:47:14 +0000 (12:47 +0100)
[ Upstream commit 48145a57d4bbe3496e8e4880b23ea6b511e6e519 ]

ndisc_send_redirect() is called under RCU protection, not RTNL.

It must use dev_get_by_index_rcu() instead of __dev_get_by_index()

Fixes: 2f17becfbea5 ("vrf: check the original netdevice for generating redirect")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Stephen Suryaputra <ssuryaextr@gmail.com>
Reviewed-by: David Ahern <dsahern@kernel.org>
Reviewed-by: Kuniyuki Iwashima <kuniyu@amazon.com>
Link: https://patch.msgid.link/20250207135841.1948589-2-edumazet@google.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
net/ipv6/ndisc.c

index 4f46b0a2e5680a5339360ca5cb071eca93e04a5a..c0a5552733177c02a1d94466106cca4e718fab60 100644 (file)
@@ -1607,7 +1607,7 @@ void ndisc_send_redirect(struct sk_buff *skb, const struct in6_addr *target)
        bool ret;
 
        if (netif_is_l3_master(skb->dev)) {
-               dev = __dev_get_by_index(dev_net(skb->dev), IPCB(skb)->iif);
+               dev = dev_get_by_index_rcu(dev_net(skb->dev), IPCB(skb)->iif);
                if (!dev)
                        return;
        }