diff options
| author | Kuniyuki Iwashima <kuniyu@amazon.com> | 2022-07-06 16:39:59 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2022-07-08 12:10:33 +0100 |
| commit | 3d32edf1f3c38d3301f6434e56316f293466d7fb (patch) | |
| tree | 5666bc6903b3208d7af103c64fb4dcfd33d1e548 | |
| parent | 47e6ab24e8c6e3ca10ceb5835413f401f90de4bf (diff) | |
| download | linux-3d32edf1f3c38d3301f6434e56316f293466d7fb.tar.bz2 | |
inetpeer: Fix data-races around sysctl.
While reading inetpeer sysctl variables, they can be changed
concurrently. So, we need to add READ_ONCE() to avoid data-races.
Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Signed-off-by: Kuniyuki Iwashima <kuniyu@amazon.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/ipv4/inetpeer.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/net/ipv4/inetpeer.c b/net/ipv4/inetpeer.c index da21dfce24d7..e9fed83e9b3c 100644 --- a/net/ipv4/inetpeer.c +++ b/net/ipv4/inetpeer.c @@ -141,16 +141,20 @@ static void inet_peer_gc(struct inet_peer_base *base, struct inet_peer *gc_stack[], unsigned int gc_cnt) { + int peer_threshold, peer_maxttl, peer_minttl; struct inet_peer *p; __u32 delta, ttl; int i; - if (base->total >= inet_peer_threshold) + peer_threshold = READ_ONCE(inet_peer_threshold); + peer_maxttl = READ_ONCE(inet_peer_maxttl); + peer_minttl = READ_ONCE(inet_peer_minttl); + + if (base->total >= peer_threshold) ttl = 0; /* be aggressive */ else - ttl = inet_peer_maxttl - - (inet_peer_maxttl - inet_peer_minttl) / HZ * - base->total / inet_peer_threshold * HZ; + ttl = peer_maxttl - (peer_maxttl - peer_minttl) / HZ * + base->total / peer_threshold * HZ; for (i = 0; i < gc_cnt; i++) { p = gc_stack[i]; |