NFSv4.2: Always decode the security label

If the server returns a reply that includes a security label, then we
must decode it whether or not we can store the results.

Fixes: 1e2f67da89 ("NFS: Remove the nfs4_label argument from decode_getattr_*() functions")
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
This commit is contained in:
Trond Myklebust 2022-10-18 17:41:30 -04:00
parent eef7314caf
commit c8a62f4402

View File

@ -4755,12 +4755,10 @@ static int decode_getfattr_attrs(struct xdr_stream *xdr, uint32_t *bitmap,
if (status < 0) if (status < 0)
goto xdr_error; goto xdr_error;
if (fattr->label) { status = decode_attr_security_label(xdr, bitmap, fattr->label);
status = decode_attr_security_label(xdr, bitmap, fattr->label); if (status < 0)
if (status < 0) goto xdr_error;
goto xdr_error; fattr->valid |= status;
fattr->valid |= status;
}
xdr_error: xdr_error:
dprintk("%s: xdr returned %d\n", __func__, -status); dprintk("%s: xdr returned %d\n", __func__, -status);