Jim Chen has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255635?usp=email ) Change subject: recovery_fs_ng: Fix NUL termination and cid_len check in clid parser ...................................................................... recovery_fs_ng: Fix NUL termination and cid_len check in clid parser In fs_ng_read_recov_clids_impl(), temp[10] is an uninitialized stack buffer. Explicitly NUL-terminate temp[len] = '\0' and reject empty or non-positive cid_len values when reconstructing clientid strings. Change-Id: I65f381d8ac3e82cd6abdc669930e58f6f76b3e6d Signed-off-by: Jim Chen <czcbangkai@google.com> --- M src/SAL/recovery/recovery_fs_ng.c 1 file changed, 5 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/35/1255635/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255635?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I65f381d8ac3e82cd6abdc669930e58f6f76b3e6d Gerrit-Change-Number: 1255635 Gerrit-PatchSet: 1 Gerrit-Owner: Jim Chen <czcbangkai@google.com>