From: Kuniyuki Iwashima Date: Mon, 18 Jul 2022 17:26:50 +0000 (-0700) Subject: tcp: Fix a data-race around sysctl_tcp_stdurg. X-Git-Tag: v5.15.73~1981 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d46d55e3f9f8051585fada583fc92d7826d0731a;p=platform%2Fkernel%2Flinux-rpi.git tcp: Fix a data-race around sysctl_tcp_stdurg. [ Upstream commit 4e08ed41cb1194009fc1a916a59ce3ed4afd77cd ] While reading sysctl_tcp_stdurg, it can be changed concurrently. Thus, we need to add READ_ONCE() to its reader. Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2") Signed-off-by: Kuniyuki Iwashima Signed-off-by: David S. Miller Signed-off-by: Sasha Levin --- diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c index 1cc0aca..6309a4e 100644 --- a/net/ipv4/tcp_input.c +++ b/net/ipv4/tcp_input.c @@ -5542,7 +5542,7 @@ static void tcp_check_urg(struct sock *sk, const struct tcphdr *th) struct tcp_sock *tp = tcp_sk(sk); u32 ptr = ntohs(th->urg_ptr); - if (ptr && !sock_net(sk)->ipv4.sysctl_tcp_stdurg) + if (ptr && !READ_ONCE(sock_net(sk)->ipv4.sysctl_tcp_stdurg)) ptr--; ptr += ntohl(th->seq);