Jeff Layton has uploaded this change for review. (
https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/448040
Change subject: rados_recov: fix buffer handling of clid entries from rados omap
......................................................................
rados_recov: fix buffer handling of clid entries from rados omap
The RADOS recovery backends store the client recovery blobs as
unterminated strings. When they are read in at boot time however, we
expect them to be terminated. This means that we can end up reading past
the end of the buffer.
Fix this by passing the value length to the callback, and have the
callback use that to properly terminate the string in memory.
Change-Id: I0a07672cd5855f329540bed235d260a43f248605
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
M src/SAL/recovery/recovery_rados.h
M src/SAL/recovery/recovery_rados_kv.c
M src/SAL/recovery/recovery_rados_ng.c
3 files changed, 16 insertions(+), 9 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/40/448040/1
--
To view, visit
https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/448040
To unsubscribe, or for help writing mail filters, visit
https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I0a07672cd5855f329540bed235d260a43f248605
Gerrit-Change-Number: 448040
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>
Gerrit-MessageType: newchange