aboutsummaryrefslogtreecommitdiffstats
path: root/net/rxrpc/conn_event.c
diff options
context:
space:
mode:
authorDavid Howells <dhowells@redhat.com>2016-08-24 13:06:14 +0100
committerDavid Howells <dhowells@redhat.com>2016-08-24 13:06:14 +0100
commit2266ffdef5737fdfa96005204fc5606dbd559956 (patch)
treed56e7e0fecb29ebd45772460e3e248084cb58eab /net/rxrpc/conn_event.c
parentnet: diag: support SOCK_DESTROY for UDP sockets (diff)
downloadlinux-dev-2266ffdef5737fdfa96005204fc5606dbd559956.tar.xz
linux-dev-2266ffdef5737fdfa96005204fc5606dbd559956.zip
rxrpc: Fix conn-based retransmit
If a duplicate packet comes in for a call that has just completed on a connection's channel then there will be an oops in the data_ready handler because it tries to examine the connection struct via a call struct (which we don't have - the pointer is unset). Since the connection struct pointer is available to us, go direct instead. Also, the ACK packet to be retransmitted needs three octets of padding between the soft ack list and the ackinfo. Fixes: 18bfeba50dfd0c8ee420396f2570f16a0bdbd7de ("rxrpc: Perform terminal call ACK/ABORT retransmission from conn processor") Signed-off-by: David Howells <dhowells@redhat.com>
Diffstat (limited to '')
-rw-r--r--net/rxrpc/conn_event.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/net/rxrpc/conn_event.c b/net/rxrpc/conn_event.c
index c1c6b7f305d1..6296374df840 100644
--- a/net/rxrpc/conn_event.c
+++ b/net/rxrpc/conn_event.c
@@ -42,6 +42,7 @@ static void rxrpc_conn_retransmit(struct rxrpc_connection *conn,
} abort;
struct {
struct rxrpc_ackpacket ack;
+ u8 padding[3];
struct rxrpc_ackinfo info;
};
};