diff options
| author | David Howells <dhowells@redhat.com> | 2016-09-17 11:49:11 +0200 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2016-09-17 11:50:49 +0200 |
| commit | 2e2ea51dec2ab6a81950d4b436eb66ebf45dd507 (patch) | |
| tree | aab85ecd55cf497bdbe67dddb8eea9d0cabb5e5b | |
| parent | rxrpc: Move the check of rx_pkt_offset from rxrpc_locate_data() to caller (diff) | |
| download | linux-2e2ea51dec2ab6a81950d4b436eb66ebf45dd507.tar.xz linux-2e2ea51dec2ab6a81950d4b436eb66ebf45dd507.zip | |
rxrpc: Check the return value of rxrpc_locate_data()
Check the return value of rxrpc_locate_data() in rxrpc_recvmsg_data().
Signed-off-by: David Howells <dhowells@redhat.com>
| -rw-r--r-- | net/rxrpc/recvmsg.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/net/rxrpc/recvmsg.c b/net/rxrpc/recvmsg.c index 0d085f5cf1bf..1edf2cf62cc5 100644 --- a/net/rxrpc/recvmsg.c +++ b/net/rxrpc/recvmsg.c @@ -300,10 +300,13 @@ static int rxrpc_recvmsg_data(struct socket *sock, struct rxrpc_call *call, if (msg) sock_recv_timestamp(msg, sock->sk, skb); - if (rx_pkt_offset == 0) + if (rx_pkt_offset == 0) { ret = rxrpc_locate_data(call, skb, &call->rxtx_annotations[ix], &rx_pkt_offset, &rx_pkt_len); + if (ret < 0) + goto out; + } _debug("recvmsg %x DATA #%u { %d, %d }", sp->hdr.callNumber, seq, rx_pkt_offset, rx_pkt_len); |