NFSD: Clean up nfsd4_encode_rdattr_error()
authorChuck Lever <chuck.lever@oracle.com>
Wed, 4 Oct 2023 13:41:51 +0000 (09:41 -0400)
committerChuck Lever <chuck.lever@oracle.com>
Mon, 16 Oct 2023 16:44:34 +0000 (12:44 -0400)
No need for specialized code here, as this function is invoked only
rarely. Convert it to encode to xdr_stream using conventional XDR
helpers.

Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/nfs4xdr.c

index b6bac09b5a1049a63ddb9ecb23ea12af206383e8..dc98215724cd4ff714421ddf1dc202996cb99866 100644 (file)
@@ -3726,21 +3726,22 @@ out_put:
        return nfserr;
 }
 
-static __be32 *
-nfsd4_encode_rdattr_error(struct xdr_stream *xdr, __be32 nfserr)
+static __be32
+nfsd4_encode_entry4_rdattr_error(struct xdr_stream *xdr, __be32 nfserr)
 {
-       __be32 *p;
-
-       p = xdr_reserve_space(xdr, 20);
-       if (!p)
-               return NULL;
-       *p++ = htonl(2);
-       *p++ = htonl(FATTR4_WORD0_RDATTR_ERROR); /* bmval0 */
-       *p++ = htonl(0);                         /* bmval1 */
+       __be32 status;
 
-       *p++ = htonl(4);     /* attribute length */
-       *p++ = nfserr;       /* no htonl */
-       return p;
+       /* attrmask */
+       status = nfsd4_encode_bitmap4(xdr, FATTR4_WORD0_RDATTR_ERROR, 0, 0);
+       if (status != nfs_ok)
+               return status;
+       /* attr_vals */
+       if (xdr_stream_encode_u32(xdr, XDR_UNIT) != XDR_UNIT)
+               return nfserr_resource;
+       /* rdattr_error */
+       if (xdr_stream_encode_be32(xdr, nfserr) != XDR_UNIT)
+               return nfserr_resource;
+       return nfs_ok;
 }
 
 static int
@@ -3812,8 +3813,7 @@ nfsd4_encode_entry4(void *ccdv, const char *name, int namlen,
                 */
                if (!(cd->rd_bmval[0] & FATTR4_WORD0_RDATTR_ERROR))
                        goto fail;
-               p = nfsd4_encode_rdattr_error(xdr, nfserr);
-               if (p == NULL) {
+               if (nfsd4_encode_entry4_rdattr_error(xdr, nfserr)) {
                        nfserr = nfserr_toosmall;
                        goto fail;
                }