diff options
| author | Eric Dumazet <edumazet@google.com> | 2019-04-25 03:21:40 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2019-04-26 19:01:11 +0300 |
| commit | 0e58156d700ac45fd5f0f90698a13233b1fe4c44 (patch) | |
| tree | 50ffd922fe206d6637b4f1ebb1fdd39b7f5caf69 /net/tipc | |
| parent | 8c90b795e90f7753d23c18e8b95dd71b4a18c5d9 (diff) | |
| download | linux-0e58156d700ac45fd5f0f90698a13233b1fe4c44.tar.xz | |
tipc: remove rcu_read_unlock() left in tipc_udp_recv()
I forgot to remove one rcu_read_unlock() before a return statement.
Joy of mixing goto and return styles in a function :)
Fixes: 4109a2c3b91e ("tipc: tipc_udp_recv() cleanup vs rcu verbs")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc')
| -rw-r--r-- | net/tipc/udp_media.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/net/tipc/udp_media.c b/net/tipc/udp_media.c index 7413cbc9b638..0884a1b8ad12 100644 --- a/net/tipc/udp_media.c +++ b/net/tipc/udp_media.c @@ -360,7 +360,6 @@ static int tipc_udp_recv(struct sock *sk, struct sk_buff *skb) if (b && test_bit(0, &b->up)) { tipc_rcv(sock_net(sk), skb, b); - rcu_read_unlock(); return 0; } |