ofir vainshtein has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238446?usp=email ) Change subject: Replace assert with a check for empty client record value. ...................................................................... Replace assert with a check for empty client record value. The assert on `len` could cause a crash if the client record value was empty. This change replaces the assert with a check that logs a critical error and returns NULL when `len` is 0, preventing a potential crash. Change-Id: I8f60ca24db4e00417fd8fb921e0541bb73fd6dfa Signed-off-by: Ofir Vainshtein <ofirvins(a)google.com&gt; --- M src/SAL/nfs4_clientid.c 1 file changed, 4 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/46/1238446/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1238446?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: I8f60ca24db4e00417fd8fb921e0541bb73fd6dfa Gerrit-Change-Number: 1238446 Gerrit-PatchSet: 1 Gerrit-Owner: ofir vainshtein <ofirvins(a)google.com&gt;