Jeff Layton has uploaded this change for review.

View Change

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@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 change 448040. To unsubscribe, or for help writing mail filters, visit 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@redhat.com>
Gerrit-MessageType: newchange