diff options
author | Jason Xing <kernelxing@tencent.com> | 2024-03-04 11:20:45 +0300 |
---|---|---|
committer | Sasha Levin <sashal@kernel.org> | 2024-03-15 17:48:16 +0300 |
commit | 97a4d8b9f67cc7efe9a0c137e12f6d9e40795bf1 (patch) | |
tree | fcfcc6841c957c0350234c201d57c41dd2c5999e | |
parent | c13fbb5902bce848759385986d4833f5b90782c1 (diff) | |
download | linux-97a4d8b9f67cc7efe9a0c137e12f6d9e40795bf1.tar.xz |
netrom: Fix a data-race around sysctl_netrom_link_fails_count
[ Upstream commit bc76645ebdd01be9b9994dac39685a3d0f6f7985 ]
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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
-rw-r--r-- | net/netrom/nr_route.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/netrom/nr_route.c b/net/netrom/nr_route.c index eb285b12977b..744c19a7a469 100644 --- a/net/netrom/nr_route.c +++ b/net/netrom/nr_route.c @@ -728,7 +728,7 @@ void nr_link_failed(ax25_cb *ax25, int reason) nr_neigh->ax25 = NULL; ax25_cb_put(ax25); - if (++nr_neigh->failed < sysctl_netrom_link_fails_count) { + if (++nr_neigh->failed < READ_ONCE(sysctl_netrom_link_fails_count)) { nr_neigh_put(nr_neigh); return; } |