>From Gui Hecheng <[email protected]>: Gui Hecheng has uploaded a new change for review.
https://review.gerrithub.io/301535 Change subject: FSAL_MDCACHE: fix duplicated put on creation of mdcache export ...................................................................... FSAL_MDCACHE: fix duplicated put on creation of mdcache export I hit an assert() while adding a vfs export after commit: FSAL: fix segfault while adding export for a non-exist subdirectory Then I find that there is a duplicated fsal_put() on the failure code path of mdcache_fsal_create_export(), because the fsal parameter is expected to be freed in the caller where it gets alloced. Also I add some frees for the mdcache_fsal_export which is alloced right in mdcache_fsal_create_export(). Some out-of-date comment is corrected as well. Change-Id: Iffee669ebbdb890ea1b35a494e365e4c40bc4718 Signed-off-by: Gui Hecheng <[email protected]> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_main.c 1 file changed, 4 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/35/301535/1 -- To view, visit https://review.gerrithub.io/301535 To unsubscribe, visit https://review.gerrithub.io/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Iffee669ebbdb890ea1b35a494e365e4c40bc4718 Gerrit-PatchSet: 1 Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Owner: Gui Hecheng <[email protected]> ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today. http://sdm.link/xeonphi _______________________________________________ Nfs-ganesha-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
