Chuck Lever e4d0e135ee xdrgen: Initialize data pointer for zero-length items ...

[ Upstream commit 27b0fcae8f ]

The xdrgen decoders for strings and opaque data had an
optimization that skipped calling xdr_inline_decode() when the
item length was zero. This left the data pointer uninitialized,
which could lead to unpredictable behavior when callers access
it.

Remove the zero-length check and always call xdr_inline_decode().
When passed a length of zero, xdr_inline_decode() returns the
current buffer position, which is valid and matches the behavior
of hand-coded XDR decoders throughout the kernel.

Fixes: 4b132aacb0 ("tools: Add xdrgen")
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Reviewed-by: NeilBrown <neil@brown.name>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

2026-03-04 07:20:05 -05:00

..

xdrgen

…

addr.h

…

auth_gss.h

…

auth.h

…

bc_xprt.h

…

cache.h

…

clnt.h

…

debug.h

…

gss_api.h

…

gss_asn1.h

…

gss_err.h

…

gss_krb5.h

…

metrics.h

…

msg_prot.h

…

rdma_rn.h

…

rpc_pipe_fs.h

…

rpc_rdma_cid.h

…

rpc_rdma.h

…

sched.h

…

stats.h

…

svc_rdma_pcl.h

…

svc_rdma.h

…

svc_xprt.h

…

svc.h

…

svcauth_gss.h

…

svcauth.h

…

svcsock.h

…

timer.h

…

types.h

…

xdr.h

…

xprt.h

…

xprtmultipath.h

…

xprtrdma.h

…

xprtsock.h

…