>From Malahal <mala...@gmail.com>:
Malahal has uploaded this change for review. (
https://review.gerrithub.io/370632
Change subject: Fix a race in using hashtables leading to crashes
......................................................................
Fix a race in using hashtables leading to crashes
get_state_owner, get_nsm_client, get_nlm_client and get_nlm_state
functions use atomic_fetch_int32_t() on the refcount to see if their
hashtable entries are about to be deleted. If not, they later place a
hold and use the entry! It is possible that an entry might have a
positive refcount when atomic_fetch_int32_t() was used, but it could go
to zero by the time these functions place a hold on the entry. In other
words, these functions end up using entries that are about to be deleted
and freed by some other threads leading to ganesha crashes.
Change-Id: Ife9c8b6c5247ae9c902c4259fb8a8c11dc512477
Signed-off-by: Malahal Naineni <mala...@us.ibm.com>
---
M src/SAL/nlm_owner.c
M src/SAL/nlm_state.c
M src/SAL/state_misc.c
3 files changed, 21 insertions(+), 27 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha
refs/changes/32/370632/1
--
To view, visit https://review.gerrithub.io/370632
To unsubscribe, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ife9c8b6c5247ae9c902c4259fb8a8c11dc512477
Gerrit-Change-Number: 370632
Gerrit-PatchSet: 1
Gerrit-Owner: Malahal <mala...@gmail.com>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
