diff options
author | J. Bruce Fields <bfields@redhat.com> | 2012-12-03 22:35:35 +0100 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2012-12-04 13:49:06 +0100 |
commit | 6a72ae2e23922bc96e8f3de24a5203be6edc2539 (patch) | |
tree | fbcaebe3db5d030ddf8c0ad4cadf936281117360 /net/sunrpc/svcsock.c | |
parent | svcrpc: delay minimum-rpc-size check till later (diff) | |
download | linux-6a72ae2e23922bc96e8f3de24a5203be6edc2539.tar.xz linux-6a72ae2e23922bc96e8f3de24a5203be6edc2539.zip |
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/sunrpc/svcsock.c')
-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 15571790dc9e..1db42b1ffe28 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; } |