diff options
author | David Howells <dhowells@redhat.com> | 2018-10-15 17:37:21 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-10-16 02:13:42 -0400 |
commit | 1890fea7936ad9be0b7caf6a94146b0d905c4b60 (patch) | |
tree | f97ce997c4250d28edb65e9e77397ec00a161de1 /net/rxrpc | |
parent | d805397c3822d57ca3884d4bea37b2291fc40992 (diff) |
rxrpc: Fix a missing rxrpc_put_peer() in the error_report handler
Fix a missing call to rxrpc_put_peer() on the main path through the
rxrpc_error_report() function. This manifests itself as a ref leak
whenever an ICMP packet or other error comes in.
In commit f334430316e7, the hand-off of the ref to a work item was removed
and was not replaced with a put.
Fixes: f334430316e7 ("rxrpc: Fix error distribution")
Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/rxrpc')
-rw-r--r-- | net/rxrpc/peer_event.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/rxrpc/peer_event.c b/net/rxrpc/peer_event.c index 05b51bdbdd41..bd2fa3b7caa7 100644 --- a/net/rxrpc/peer_event.c +++ b/net/rxrpc/peer_event.c | |||
@@ -195,6 +195,7 @@ void rxrpc_error_report(struct sock *sk) | |||
195 | rxrpc_store_error(peer, serr); | 195 | rxrpc_store_error(peer, serr); |
196 | rcu_read_unlock(); | 196 | rcu_read_unlock(); |
197 | rxrpc_free_skb(skb, rxrpc_skb_rx_freed); | 197 | rxrpc_free_skb(skb, rxrpc_skb_rx_freed); |
198 | rxrpc_put_peer(peer); | ||
198 | 199 | ||
199 | _leave(""); | 200 | _leave(""); |
200 | } | 201 | } |