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(a)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(a)google.com>
Gerrit-MessageType: newchange