rxrpc: Fix call leak
authorDavid Howells <dhowells@redhat.com>
Mon, 28 Nov 2022 09:24:06 +0000 (09:24 +0000)
committerDavid Howells <dhowells@redhat.com>
Thu, 1 Dec 2022 13:36:37 +0000 (13:36 +0000)
When retransmitting a packet, rxrpc_resend() shouldn't be attaching a ref
to the call to the txbuf as that pins the call and prevents the call from
clearing the packet buffer.

Signed-off-by: David Howells <dhowells@redhat.com>
Fixes: d57a3a151660 ("rxrpc: Save last ACK's SACK table rather than marking txbufs")
cc: Marc Dionne <marc.dionne@auristor.com>
cc: linux-afs@lists.infradead.org

net/rxrpc/call_event.c

index 1e21a708390e74c3e782c0b4d80bad262207f67f..349f3df569bafb1caba4b0bdcc3a625c7b654faa 100644 (file)
@@ -198,7 +198,6 @@ static void rxrpc_resend(struct rxrpc_call *call, unsigned long now_j)
 
                        if (list_empty(&txb->tx_link)) {
                                rxrpc_get_txbuf(txb, rxrpc_txbuf_get_retrans);
-                               rxrpc_get_call(call, rxrpc_call_got_tx);
                                list_add_tail(&txb->tx_link, &retrans_queue);
                                set_bit(RXRPC_TXBUF_RESENT, &txb->flags);
                        }