nfsd4: Remove check for a 32-bit cookie in nfsd4_readdir()
authorBernd Schubert <bernd.schubert@itwm.fraunhofer.de>
Mon, 8 Aug 2011 15:38:08 +0000 (17:38 +0200)
committerJ. Bruce Fields <bfields@redhat.com>
Tue, 16 Aug 2011 19:19:28 +0000 (15:19 -0400)
Fan Yong <yong.fan@whamcloud.com> noticed setting
FMODE_32bithash wouldn't work with nfsd v4, as
nfsd4_readdir() checks for 32 bit cookies. However, according to RFC 3530
cookies have a 64 bit type and cookies are also defined as u64 in
'struct nfsd4_readdir'. So remove the test for >32-bit values.

Cc: stable@kernel.org
Signed-off-by: Bernd Schubert <bernd.schubert@itwm.fraunhofer.de>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/nfs4proc.c

index e80777666618d40d9c3f096d4c452c35e3f93cdf..9bf0a66251877e8cdd8cf3d0d556fbcf0630598a 100644 (file)
@@ -691,7 +691,7 @@ nfsd4_readdir(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
        readdir->rd_bmval[1] &= nfsd_suppattrs1(cstate->minorversion);
        readdir->rd_bmval[2] &= nfsd_suppattrs2(cstate->minorversion);
 
-       if ((cookie > ~(u32)0) || (cookie == 1) || (cookie == 2) ||
+       if ((cookie == 1) || (cookie == 2) ||
            (cookie == 0 && memcmp(readdir->rd_verf.data, zeroverf.data, NFS4_VERIFIER_SIZE)))
                return nfserr_bad_cookie;