nfsd4: tweak rd_dircount accounting
authorJ. Bruce Fields <bfields@redhat.com>
Fri, 19 Dec 2014 23:01:35 +0000 (18:01 -0500)
committerJ. Bruce Fields <bfields@redhat.com>
Wed, 7 Jan 2015 19:48:10 +0000 (14:48 -0500)
RFC 3530 14.2.24 says

This value represents the length of the names of the directory
entries and the cookie value for these entries.  This length
represents the XDR encoding of the data (names and cookies)...

The "xdr encoding" of the name should probably include the 4 bytes for
the length.

But this is all just a hint so not worth e.g. backporting to stable.

Also reshuffle some lines to more clearly group together the
dircount-related code.

Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/nfs4xdr.c

index 15f7b73e0c0fd8e49012134ca2d4afe83671012c..91f7a3644ffbc7ab00525cc6f53f74ccf916f777 100644 (file)
@@ -2768,16 +2768,17 @@ nfsd4_encode_dirent(void *ccdv, const char *name, int namlen,
        if (entry_bytes > cd->rd_maxcount)
                goto fail;
        cd->rd_maxcount -= entry_bytes;
-       if (!cd->rd_dircount)
-               goto fail;
        /*
         * RFC 3530 14.2.24 describes rd_dircount as only a "hint", so
         * let's always let through the first entry, at least:
         */
-       name_and_cookie = 4 * XDR_QUADLEN(namlen) + 8;
+       if (!cd->rd_dircount)
+               goto fail;
+       name_and_cookie = 4 + 4 * XDR_QUADLEN(namlen) + 8;
        if (name_and_cookie > cd->rd_dircount && cd->cookie_offset)
                goto fail;
        cd->rd_dircount -= min(cd->rd_dircount, name_and_cookie);
+
        cd->cookie_offset = cookie_offset;
 skip_entry:
        cd->common.err = nfs_ok;