summaryrefslogtreecommitdiff
path: root/net/tipc
diff options
context:
space:
mode:
authorEric Dumazet <edumazet@google.com>2019-04-25 03:21:40 +0300
committerDavid S. Miller <davem@davemloft.net>2019-04-26 19:01:11 +0300
commit0e58156d700ac45fd5f0f90698a13233b1fe4c44 (patch)
tree50ffd922fe206d6637b4f1ebb1fdd39b7f5caf69 /net/tipc
parent8c90b795e90f7753d23c18e8b95dd71b4a18c5d9 (diff)
downloadlinux-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.c1
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;
}