netrom: Fix a data-race around sysctl_netrom_transport_busy_delay
authorJason Xing <kernelxing@tencent.com>
Mon, 4 Mar 2024 08:20:41 +0000 (16:20 +0800)
committerPaolo Abeni <pabeni@redhat.com>
Thu, 7 Mar 2024 09:36:58 +0000 (10:36 +0100)
We need to protect the reader reading the sysctl value because the
value can be changed concurrently.

Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Signed-off-by: Jason Xing <kernelxing@tencent.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
net/netrom/af_netrom.c

index 8ada0da3c0e087c07767600eaed3f7205784ef9a..10eee02ef99edc18f8e133e9d03293282f5cddd4 100644 (file)
@@ -459,7 +459,7 @@ static int nr_create(struct net *net, struct socket *sock, int protocol,
        nr->n2     =
                msecs_to_jiffies(READ_ONCE(sysctl_netrom_transport_maximum_tries));
        nr->t4     =
-               msecs_to_jiffies(sysctl_netrom_transport_busy_delay);
+               msecs_to_jiffies(READ_ONCE(sysctl_netrom_transport_busy_delay));
        nr->idle   =
                msecs_to_jiffies(sysctl_netrom_transport_no_activity_timeout);
        nr->window = sysctl_netrom_transport_requested_window_size;