* [PATCH] NFSD: change LISTXATTRS cookie encoding to big-endian
@ 2024-01-25 14:46 Jorge Mora
2024-01-25 17:08 ` Jeff Layton
0 siblings, 1 reply; 2+ messages in thread
From: Jorge Mora @ 2024-01-25 14:46 UTC (permalink / raw)
To: linux-nfs; +Cc: chuck.lever, jlayton
Function nfsd4_listxattr_validate_cookie() expects the cookie
as an offset to the list thus it needs to be encoded in big-endian.
Fixes: 23e50fe3a5e6 ("nfsd: implement the xattr functions and en/decode logic")
Signed-off-by: Jorge Mora <mora@netapp.com>
---
fs/nfsd/nfs4xdr.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/fs/nfsd/nfs4xdr.c b/fs/nfsd/nfs4xdr.c
index 26993bf368fc..913d566519bf 100644
--- a/fs/nfsd/nfs4xdr.c
+++ b/fs/nfsd/nfs4xdr.c
@@ -5142,6 +5142,7 @@ nfsd4_encode_listxattrs(struct nfsd4_compoundres *resp, __be32 nfserr,
u64 cookie;
char *sp;
__be32 status, tmp;
+ __be64 wire_cookie;
__be32 *p;
u32 nuser;
@@ -5235,7 +5236,8 @@ nfsd4_encode_listxattrs(struct nfsd4_compoundres *resp, __be32 nfserr,
cookie = offset + count;
- write_bytes_to_xdr_buf(xdr->buf, cookie_offset, &cookie, 8);
+ wire_cookie = cpu_to_be64(cookie);
+ write_bytes_to_xdr_buf(xdr->buf, cookie_offset, &wire_cookie, 8);
tmp = cpu_to_be32(count);
write_bytes_to_xdr_buf(xdr->buf, count_offset, &tmp, 4);
out:
--
2.43.0
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] NFSD: change LISTXATTRS cookie encoding to big-endian
2024-01-25 14:46 [PATCH] NFSD: change LISTXATTRS cookie encoding to big-endian Jorge Mora
@ 2024-01-25 17:08 ` Jeff Layton
0 siblings, 0 replies; 2+ messages in thread
From: Jeff Layton @ 2024-01-25 17:08 UTC (permalink / raw)
To: Jorge Mora, linux-nfs; +Cc: chuck.lever
On Thu, 2024-01-25 at 07:46 -0700, Jorge Mora wrote:
> Function nfsd4_listxattr_validate_cookie() expects the cookie
> as an offset to the list thus it needs to be encoded in big-endian.
>
> Fixes: 23e50fe3a5e6 ("nfsd: implement the xattr functions and en/decode logic")
> Signed-off-by: Jorge Mora <mora@netapp.com>
> ---
> fs/nfsd/nfs4xdr.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/fs/nfsd/nfs4xdr.c b/fs/nfsd/nfs4xdr.c
> index 26993bf368fc..913d566519bf 100644
> --- a/fs/nfsd/nfs4xdr.c
> +++ b/fs/nfsd/nfs4xdr.c
> @@ -5142,6 +5142,7 @@ nfsd4_encode_listxattrs(struct nfsd4_compoundres *resp, __be32 nfserr,
> u64 cookie;
> char *sp;
> __be32 status, tmp;
> + __be64 wire_cookie;
> __be32 *p;
> u32 nuser;
>
> @@ -5235,7 +5236,8 @@ nfsd4_encode_listxattrs(struct nfsd4_compoundres *resp, __be32 nfserr,
>
> cookie = offset + count;
>
> - write_bytes_to_xdr_buf(xdr->buf, cookie_offset, &cookie, 8);
> + wire_cookie = cpu_to_be64(cookie);
> + write_bytes_to_xdr_buf(xdr->buf, cookie_offset, &wire_cookie, 8);
> tmp = cpu_to_be32(count);
> write_bytes_to_xdr_buf(xdr->buf, count_offset, &tmp, 4);
> out:
Reviewed-by: Jeff Layton <jlayton@kernel.org>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-01-25 17:08 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-25 14:46 [PATCH] NFSD: change LISTXATTRS cookie encoding to big-endian Jorge Mora
2024-01-25 17:08 ` Jeff Layton
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox