Matthew DeVore has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/505149 ) Change subject: state_deleg: Grab lock after confirming reg file ...................................................................... state_deleg: Grab lock after confirming reg file Rather than grab the lock then check the object type, check the object type first. Fixes crash that occurs on macOS when running st_setattr.testInodeLocking from pynfs, because it prevents access of the lock in the wrong side of the lock union in state_hdl. Change-Id: I4de8ddef3a7fdca9b27902a9b43abe926e64d545 Signed-off-by: Matthew DeVore <matvore@google.com> --- M src/SAL/state_deleg.c 1 file changed, 7 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/49/505149/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/505149 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: I4de8ddef3a7fdca9b27902a9b43abe926e64d545 Gerrit-Change-Number: 505149 Gerrit-PatchSet: 1 Gerrit-Owner: Matthew DeVore <matvore@google.com> Gerrit-MessageType: newchange