NFSD: fix nfsd4_listxattr_validate_cookie
authorJorge Mora <jmora1300@gmail.com>
Thu, 25 Jan 2024 14:46:54 +0000 (07:46 -0700)
committerChuck Lever <chuck.lever@oracle.com>
Fri, 1 Mar 2024 14:12:06 +0000 (09:12 -0500)
commit52a357db8074e18aa51085a8c8d8af7057dffa11
tree86700b6cc0554a894a6bba94a8407782d4250d1e
parent5ff318f645eb6f5d3f93c280bbd5d050b2676cd6
NFSD: fix nfsd4_listxattr_validate_cookie

If LISTXATTRS is sent with a correct cookie but a small maxcount,
this could lead function nfsd4_listxattr_validate_cookie to
return NFS4ERR_BAD_COOKIE. If maxcount = 20, then second check
on function gives RHS = 3 thus any cookie larger than 3 returns
NFS4ERR_BAD_COOKIE.

There is no need to validate the cookie on the return XDR buffer
since attribute referenced by cookie will be the first in the
return buffer.

Fixes: 23e50fe3a5e6 ("nfsd: implement the xattr functions and en/decode logic")
Signed-off-by: Jorge Mora <mora@netapp.com>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/nfs4xdr.c