diff options
author | J. Bruce Fields <bfields@redhat.com> | 2012-12-03 16:35:35 -0500 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2012-12-04 07:49:06 -0500 |
commit | 6a72ae2e23922bc96e8f3de24a5203be6edc2539 (patch) | |
tree | fbcaebe3db5d030ddf8c0ad4cadf936281117360 /net | |
parent | ad46ccf09440975618e8fc1ead53d0a27b9bcf4c (diff) | |
download | op-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.c | 3 |
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; } |