Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/488570 ) Change subject: Make fullpath and pseudopath gsh_refstr and keep a ref in op_ctx ...................................................................... Make fullpath and pseudopath gsh_refstr and keep a ref in op_ctx The pseudopath is going to be changeable in a following patch so we must make a stable version easily accessible. Use gsh_refstr to accomplish that. Also, for consistency handle fullpath the same way. It may actually be possible to allow changing of it also and in the meantime, code that picks one or the other of pseudopath and fullpath will not have to deal with different rules for access to those strings. Change-Id: I40270796f6508f1d2f9cc7659347ab24189a03ff Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com> --- M src/FSAL/FSAL_CEPH/export.c M src/FSAL/FSAL_CEPH/main.c M src/FSAL/FSAL_GLUSTER/export.c M src/FSAL/FSAL_GPFS/export.c M src/FSAL/FSAL_GPFS/fsal_up.c M src/FSAL/FSAL_LIZARDFS/export.c M src/FSAL/FSAL_LIZARDFS/main.c M src/FSAL/FSAL_MEM/mem_export.c M src/FSAL/FSAL_PROXY/export.c M src/FSAL/FSAL_PSEUDO/export.c M src/FSAL/FSAL_RGW/main.c M src/FSAL/FSAL_VFS/export.c M src/FSAL/FSAL_VFS/handle.c M src/FSAL/FSAL_VFS/subfsal_helpers.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_export.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_up.c M src/FSAL/commonlib.c M src/FSAL/fsal_helper.c M src/FSAL_UP/fsal_up_top.c M src/MainNFSD/libganesha_nfsd.ver M src/MainNFSD/nfs_worker_thread.c M src/Protocols/9P/9p_attach.c M src/Protocols/9P/9p_proto_tools.c M src/Protocols/NFS/mnt_Export.c M src/Protocols/NFS/mnt_Mnt.c M src/Protocols/NFS/nfs3_create.c M src/Protocols/NFS/nfs3_mkdir.c M src/Protocols/NFS/nfs3_mknod.c M src/Protocols/NFS/nfs3_symlink.c M src/Protocols/NFS/nfs3_write.c M src/Protocols/NFS/nfs4_Compound.c M src/Protocols/NFS/nfs4_op_allocate.c M src/Protocols/NFS/nfs4_op_create.c M src/Protocols/NFS/nfs4_op_free_stateid.c M src/Protocols/NFS/nfs4_op_layoutreturn.c M src/Protocols/NFS/nfs4_op_lookup.c M src/Protocols/NFS/nfs4_op_lookupp.c M src/Protocols/NFS/nfs4_op_putfh.c M src/Protocols/NFS/nfs4_op_putrootfh.c M src/Protocols/NFS/nfs4_op_readdir.c M src/Protocols/NFS/nfs4_op_restorefh.c M src/Protocols/NFS/nfs4_op_secinfo.c M src/Protocols/NFS/nfs4_op_write.c M src/Protocols/NFS/nfs4_pseudo.c M src/Protocols/NFS/nfs_proto_tools.c M src/Protocols/RQUOTA/rquota_common.c M src/Protocols/RQUOTA/rquota_getquota.c M src/Protocols/RQUOTA/rquota_setquota.c M src/SAL/nfs4_state.c M src/SAL/state_deleg.c M src/SAL/state_layout.c M src/SAL/state_lock.c M src/SAL/state_share.c M src/gtest/gtest.hh M src/gtest/test_ci_hash_dist1.cc M src/include/export_mgr.h M src/include/fsal.h M src/include/fsal_api.h M src/include/mount.h M src/include/nfs_exports.h M src/include/nfs_proto_data.h M src/include/sal_data.h M src/support/export_mgr.c M src/support/exports.c M src/support/nfs_creds.c 65 files changed, 881 insertions(+), 427 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/70/488570/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/488570 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: I40270796f6508f1d2f9cc7659347ab24189a03ff Gerrit-Change-Number: 488570 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx@mindspring.com> Gerrit-MessageType: newchange