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(a)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(a)ibm.com>