diff options
| author | David Howells <dhowells@redhat.com> | 2016-09-17 10:49:12 +0100 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2016-09-17 10:53:21 +0100 |
| commit | d01dc4c3c1209e865368d5f8d3b5e08f97326ca9 (patch) | |
| tree | 8c32fce41fdcf639de475dffac852577a2bc40ab /net/rxrpc | |
| parent | 78883793f8ac4bb3f97d48db7a8c71d8476bcf98 (diff) | |
| download | op-kernel-dev-d01dc4c3c1209e865368d5f8d3b5e08f97326ca9.zip op-kernel-dev-d01dc4c3c1209e865368d5f8d3b5e08f97326ca9.tar.gz | |
rxrpc: Fix the parsing of soft-ACKs
The soft-ACK parser doesn't increment the pointer into the soft-ACK list,
resulting in the first ACK/NACK value being applied to all the relevant
packets in the Tx queue. This has the potential to miss retransmissions
and cause excessive retransmissions.
Fix this by incrementing the pointer.
Signed-off-by: David Howells <dhowells@redhat.com>
Diffstat (limited to 'net/rxrpc')
| -rw-r--r-- | net/rxrpc/input.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/rxrpc/input.c b/net/rxrpc/input.c index f0d9115..c1f83d2 100644 --- a/net/rxrpc/input.c +++ b/net/rxrpc/input.c @@ -384,7 +384,7 @@ static void rxrpc_input_soft_acks(struct rxrpc_call *call, u8 *acks, for (; nr_acks > 0; nr_acks--, seq++) { ix = seq & RXRPC_RXTX_BUFF_MASK; - switch (*acks) { + switch (*acks++) { case RXRPC_ACK_TYPE_ACK: call->rxtx_annotations[ix] = RXRPC_TX_ANNO_ACK; break; |
