tcp: Fix a data-race around sysctl_tcp_frto.
commit 706c6202a3589f290e1ef9be0584a8f4a3cc0507 upstream.
While reading sysctl_tcp_frto, it can be changed concurrently.
Thus, we need to add READ_ONCE() to its reader.
Fixes: 1da177e4c3
("Linux-2.6.12-rc2")
Signed-off-by: Kuniyuki Iwashima <kuniyu@amazon.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
312ce3901f
commit
81c45f49e6
@@ -2135,7 +2135,7 @@ void tcp_enter_loss(struct sock *sk)
|
|||||||
* loss recovery is underway except recurring timeout(s) on
|
* loss recovery is underway except recurring timeout(s) on
|
||||||
* the same SND.UNA (sec 3.2). Disable F-RTO on path MTU probing
|
* the same SND.UNA (sec 3.2). Disable F-RTO on path MTU probing
|
||||||
*/
|
*/
|
||||||
tp->frto = net->ipv4.sysctl_tcp_frto &&
|
tp->frto = READ_ONCE(net->ipv4.sysctl_tcp_frto) &&
|
||||||
(new_recovery || icsk->icsk_retransmits) &&
|
(new_recovery || icsk->icsk_retransmits) &&
|
||||||
!inet_csk(sk)->icsk_mtup.probe_size;
|
!inet_csk(sk)->icsk_mtup.probe_size;
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user