diff options
| author |   Jason A. Donenfeld <Jason@zx2c4.com> | 2018-07-31 07:03:07 +0200 |
|---|---|---|
| committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-07-31 07:19:52 +0200 |
| commit | b713ab0e605b4f9291f4fde38103bf689c2cdb76 (patch) | |
| tree | e18f74fc2795b8c8b80141e8bf411a8a22959dc2 /src/timers.c | |
| parent | ratelimiter: prevent init/uninit race (diff) | |
| download | wireguard-monolithic-historical-b713ab0e605b4f9291f4fde38103bf689c2cdb76.tar.xz wireguard-monolithic-historical-b713ab0e605b4f9291f4fde38103bf689c2cdb76.zip | |
peer: simplify rcu reference counts
Use RCU reference counts only when we must, and otherwise use a more
reasonably named function.
Reported-by: Jann Horn <jann@thejh.net>
Diffstat (limited to 'src/timers.c')
| -rw-r--r-- | src/timers.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/timers.c b/src/timers.c index f1549a7..e8bb101 100644 --- a/src/timers.c +++ b/src/timers.c @@ -20,7 +20,10 @@ */ #define peer_get_from_timer(timer_name) \ - struct wireguard_peer *peer = peer_rcu_get(from_timer(peer, timer, timer_name)); \ + struct wireguard_peer *peer; \ + rcu_read_lock_bh(); \ + peer = peer_get_maybe_zero(from_timer(peer, timer, timer_name)); \ + rcu_read_unlock_bh(); \ if (unlikely(!peer)) \ return; |