summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorJ. Bruce Fields <bfields@redhat.com>2012-12-03 16:35:35 -0500
committerJ. Bruce Fields <bfields@redhat.com>2012-12-04 07:49:06 -0500
commit6a72ae2e23922bc96e8f3de24a5203be6edc2539 (patch)
treefbcaebe3db5d030ddf8c0ad4cadf936281117360 /net
parentad46ccf09440975618e8fc1ead53d0a27b9bcf4c (diff)
downloadop-kernel-dev-6a72ae2e23922bc96e8f3de24a5203be6edc2539.zip
op-kernel-dev-6a72ae2e23922bc96e8f3de24a5203be6edc2539.tar.gz
svcrpc: fix off-by-4 error in "incomplete TCP record" dprintk
The full reclen doesn't include the fragment header, but sk_tcplen does. Fix this to make it an apples-to-apples comparison. Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'net')
-rw-r--r--net/sunrpc/svcsock.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/net/sunrpc/svcsock.c b/net/sunrpc/svcsock.c
index 1557179..1db42b1 100644
--- a/net/sunrpc/svcsock.c
+++ b/net/sunrpc/svcsock.c
@@ -1073,7 +1073,8 @@ static int svc_tcp_recvfrom(struct svc_rqst *rqstp)
if (len < 0 && len != -EAGAIN)
goto err_delete;
dprintk("svc: incomplete TCP record (%d of %d)\n",
- svsk->sk_tcplen, svc_sock_reclen(svsk));
+ svsk->sk_tcplen - sizeof(rpc_fraghdr),
+ svc_sock_reclen(svsk));
goto err_noclose;
}
OpenPOWER on IntegriCloud