summaryrefslogtreecommitdiff
path: root/lib/mpi/mpi-inline.c
diff options
context:
space:
mode:
authorChuck Lever <chuck.lever@oracle.com>2024-12-31 03:29:00 +0300
committerChuck Lever <chuck.lever@oracle.com>2025-01-11 07:43:44 +0300
commit1196bdce3d107194dd15f508602871ffb7ff2d0b (patch)
treef8d48778aa3cfe3269def1dc6682f0e1c59d265f /lib/mpi/mpi-inline.c
parent4163ee711cf141ada9e884a94e6e329431547fd7 (diff)
downloadlinux-1196bdce3d107194dd15f508602871ffb7ff2d0b.tar.xz
SUNRPC: Document validity guarantees of the pointer returned by reserve_space
A subtlety of this API is that if the @nbytes region traverses a page boundary, the next __xdr_commit_encode will shift the data item in the XDR encode buffer. This makes the returned pointer point to something else, leading to unexpected behavior. There are a few cases where the caller saves the returned pointer and then later uses it to insert a computed value into an earlier part of the stream. This can be safe only if either: - the data item is guaranteed to be in the XDR buffer's head, and thus is not ever going to be near a page boundary, or - the data item is no larger than 4 octets, since XDR alignment rules require all data items to start on 4-octet boundaries But that safety is only an artifact of the current implementation. It would be less brittle if these "safe" uses were eventually replaced. Reviewed-by: NeilBrown <neilb@suse.de> Reviewed-by: Jeff Layton <jlayton@kernel.org> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Diffstat (limited to 'lib/mpi/mpi-inline.c')
0 files changed, 0 insertions, 0 deletions