diff options
| author | David Howells <dhowells@redhat.com> | 2017-11-24 10:18:40 +0000 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2017-11-24 10:18:40 +0000 |
| commit | 03a6c82218b9a87014b2c6c4e178294fdc8ebd8a (patch) | |
| tree | 79ec7d435214608d096202d0513fa5209b8925d2 /net/rxrpc/sendmsg.c | |
| parent | 1d3b78bbc6e983fabb3fbf91b76339bf66e4a12c (diff) | |
| download | linux-03a6c82218b9a87014b2c6c4e178294fdc8ebd8a.tar.bz2 | |
rxrpc: The mutex lock returned by rxrpc_accept_call() needs releasing
The caller of rxrpc_accept_call() must release the lock on call->user_mutex
returned by that function.
Signed-off-by: David Howells <dhowells@redhat.com>
Diffstat (limited to 'net/rxrpc/sendmsg.c')
| -rw-r--r-- | net/rxrpc/sendmsg.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/net/rxrpc/sendmsg.c b/net/rxrpc/sendmsg.c index 7d2595582c09..3a99b1a908df 100644 --- a/net/rxrpc/sendmsg.c +++ b/net/rxrpc/sendmsg.c @@ -619,8 +619,8 @@ int rxrpc_do_sendmsg(struct rxrpc_sock *rx, struct msghdr *msg, size_t len) /* The socket is now unlocked. */ if (IS_ERR(call)) return PTR_ERR(call); - rxrpc_put_call(call, rxrpc_call_put); - return 0; + ret = 0; + goto out_put_unlock; } call = rxrpc_find_call_by_user_ID(rx, p.user_call_ID); @@ -689,6 +689,7 @@ int rxrpc_do_sendmsg(struct rxrpc_sock *rx, struct msghdr *msg, size_t len) ret = rxrpc_send_data(rx, call, msg, len, NULL); } +out_put_unlock: mutex_unlock(&call->user_mutex); error_put: rxrpc_put_call(call, rxrpc_call_put); |