Animesh Javali has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1234237?usp=email ) Change subject: Fix for Coverity IDs: 509127,509133 ...................................................................... Fix for Coverity IDs: 509127,509133 Fixed big parameter passed by value. Coverity flagged that struct sau_attr_reply is being passed by value in isOffsetChangedByClient() and hasRecentModificationTime(), leading to unnecessary copying of a large structure. This change converts the function parameters to pass by reference(pointer), avoiding the overhead of copying while maintaining the same functionality. Change-Id: Ifca860124a05984b60c97e481aec7d85c90cc84e Signed-off-by: Animesh Javali <Animesh.Javali@ibm.com> --- M src/FSAL/FSAL_SAUNAFS/mds_handle.c 1 file changed, 8 insertions(+), 8 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/37/1234237/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1234237?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ifca860124a05984b60c97e481aec7d85c90cc84e Gerrit-Change-Number: 1234237 Gerrit-PatchSet: 1 Gerrit-Owner: Animesh Javali <Animesh.Javali@ibm.com>