svcrpc: delay minimum-rpc-size check till later
authorJ. Bruce Fields <bfields@redhat.com>
Mon, 3 Dec 2012 21:30:42 +0000 (16:30 -0500)
committerJ. Bruce Fields <bfields@redhat.com>
Tue, 4 Dec 2012 12:47:58 +0000 (07:47 -0500)
Soon we want to support multiple fragments, in which case it may be
legal for a single fragment to be smaller than 8 bytes, so we'll want to
delay this check till we've reached the last fragment.

Also fix an outdated comment.

Signed-off-by: J. Bruce Fields <bfields@redhat.com>
net/sunrpc/svcsock.c

index d50de2b95036d0ff9238a64484fc5d16ac66d1a2..15571790dc9ebf16237431c1debfb959f6daecc1 100644 (file)
@@ -921,10 +921,8 @@ out:
 }
 
 /*
- * Receive data.
+ * Receive fragment record header.
  * If we haven't gotten the record length yet, get the next four bytes.
- * Otherwise try to gobble up as much as possible up to the complete
- * record length.
  */
 static int svc_tcp_recv_record(struct svc_sock *svsk, struct svc_rqst *rqstp)
 {
@@ -968,9 +966,6 @@ static int svc_tcp_recv_record(struct svc_sock *svsk, struct svc_rqst *rqstp)
                }
        }
 
-       if (svc_sock_reclen(svsk) < 8)
-               goto err_delete; /* client is nuts. */
-
        return svc_sock_reclen(svsk);
 error:
        dprintk("RPC: TCP recv_record got %d\n", len);
@@ -1076,12 +1071,15 @@ static int svc_tcp_recvfrom(struct svc_rqst *rqstp)
        if (len != want) {
                svc_tcp_save_pages(svsk, rqstp);
                if (len < 0 && len != -EAGAIN)
-                       goto err_other;
+                       goto err_delete;
                dprintk("svc: incomplete TCP record (%d of %d)\n",
                        svsk->sk_tcplen, svc_sock_reclen(svsk));
                goto err_noclose;
        }
 
+       if (svc_sock_reclen(svsk) < 8)
+               goto err_delete; /* client is nuts. */
+
        rqstp->rq_arg.len = svc_sock_reclen(svsk);
        rqstp->rq_arg.page_base = 0;
        if (rqstp->rq_arg.len <= rqstp->rq_arg.head[0].iov_len) {
@@ -1117,10 +1115,10 @@ static int svc_tcp_recvfrom(struct svc_rqst *rqstp)
 
 error:
        if (len != -EAGAIN)
-               goto err_other;
+               goto err_delete;
        dprintk("RPC: TCP recvfrom got EAGAIN\n");
        return 0;
-err_other:
+err_delete:
        printk(KERN_NOTICE "%s: recvfrom returned errno %d\n",
               svsk->sk_xprt.xpt_server->sv_name, -len);
        set_bit(XPT_CLOSE, &svsk->sk_xprt.xpt_flags);