diff options
| author | Eric Dumazet <edumazet@google.com> | 2025-02-05 15:51:12 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-02-21 14:11:14 +0100 |
| commit | 990d95dcdbf7a97c6a582dcddd8c31a2bf394a7a (patch) | |
| tree | ee1eb43403232ce2a155ffd9b62ed01b99bf5a9d /net/ipv4 | |
| parent | 7c02fcb229a21c9f020ca8b915d34027c4b0e2bc (diff) | |
| download | linux-990d95dcdbf7a97c6a582dcddd8c31a2bf394a7a.tar.gz linux-990d95dcdbf7a97c6a582dcddd8c31a2bf394a7a.tar.bz2 linux-990d95dcdbf7a97c6a582dcddd8c31a2bf394a7a.zip | |
ipv4: use RCU protection in ipv4_default_advmss()
[ Upstream commit 71b8471c93fa0bcab911fcb65da1eb6c4f5f735f ]
ipv4_default_advmss() must use RCU protection to make
sure the net structure it reads does not disappear.
Fixes: 2e9589ff809e ("ipv4: Namespaceify min_adv_mss sysctl knob")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Reviewed-by: Kuniyuki Iwashima <kuniyu@amazon.com>
Link: https://patch.msgid.link/20250205155120.1676781-5-edumazet@google.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'net/ipv4')
| -rw-r--r-- | net/ipv4/route.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/net/ipv4/route.c b/net/ipv4/route.c index 3a1467f2d553..829c8d41aaae 100644 --- a/net/ipv4/route.c +++ b/net/ipv4/route.c @@ -1307,10 +1307,15 @@ static void set_class_tag(struct rtable *rt, u32 tag) static unsigned int ipv4_default_advmss(const struct dst_entry *dst) { - struct net *net = dev_net(dst->dev); unsigned int header_size = sizeof(struct tcphdr) + sizeof(struct iphdr); - unsigned int advmss = max_t(unsigned int, ipv4_mtu(dst) - header_size, - net->ipv4.ip_rt_min_advmss); + unsigned int advmss; + struct net *net; + + rcu_read_lock(); + net = dev_net_rcu(dst->dev); + advmss = max_t(unsigned int, ipv4_mtu(dst) - header_size, + net->ipv4.ip_rt_min_advmss); + rcu_read_unlock(); return min(advmss, IPV4_MAX_PMTU - header_size); } |