diff options
| author |   Dan Carpenter <dan.carpenter@oracle.com> | 2016-06-18 11:44:03 +0300 |
|---|---|---|
| committer |   David Howells <dhowells@redhat.com> | 2016-06-22 09:09:58 +0100 |
| commit | 0e4699e4a37be4cfa07f0340cef6a3fa6a46f5f8 (patch) | |
| tree | b30879ac5a1734d71521923d429a407a2bf273dc /net/rxrpc/call_accept.c | |
| parent | Merge branch 'mlxsw-next' (diff) | |
| download | linux-dev-0e4699e4a37be4cfa07f0340cef6a3fa6a46f5f8.tar.xz linux-dev-0e4699e4a37be4cfa07f0340cef6a3fa6a46f5f8.zip | |
rxrpc: checking for IS_ERR() instead of NULL
rxrpc_lookup_peer_rcu() and rxrpc_lookup_peer() return NULL on error, never
error pointers, so IS_ERR() can't be used.
Fix three callers of those functions.
Fixes: be6e6707f6ee ('rxrpc: Rework peer object handling to use hash table and RCU')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David Howells <dhowells@redhat.com>
Diffstat (limited to 'net/rxrpc/call_accept.c')
| -rw-r--r-- | net/rxrpc/call_accept.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/rxrpc/call_accept.c b/net/rxrpc/call_accept.c index 50136c76ebd1..553b67c144e5 100644 --- a/net/rxrpc/call_accept.c +++ b/net/rxrpc/call_accept.c @@ -96,7 +96,7 @@ static int rxrpc_accept_incoming_call(struct rxrpc_local *local, notification->mark = RXRPC_SKB_MARK_NEW_CALL; peer = rxrpc_lookup_peer(local, srx, GFP_NOIO); - if (IS_ERR(peer)) { + if (!peer) { _debug("no peer"); ret = -EBUSY; goto error; |