>From Frank Filz <ffilz...@mindspring.com>:
Frank Filz has uploaded this change for review. (
https://review.gerrithub.io/359354
Change subject: Don't double deconstruct new entry in mdcache_new_entry
......................................................................
Don't double deconstruct new entry in mdcache_new_entry
The combination of mdcache_put and mdcache_kill_entry will deconstruct
the new entry, so we actually were double deconstructing and causing
problems.
Also clarify the goto labels, and since the old "out" now
"out_release_new_entry" is only called after nentry has been allocated
there is no need to check for nentry being NULL (that may have been a
holdover from before we aborted if memory allocation failed).
Change-Id: Idf1e829d8d5c7258e239824ca91df2e84810e2bb
Signed-off-by: Frank S. Filz <ffilz...@mindspring.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_int.h
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c
3 files changed, 20 insertions(+), 31 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha
refs/changes/54/359354/1
--
To view, visit https://review.gerrithub.io/359354
To unsubscribe, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: Idf1e829d8d5c7258e239824ca91df2e84810e2bb
Gerrit-Change-Number: 359354
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilz...@mindspring.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
