Animesh Javali has uploaded this change for review.

View Change

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 change 1234237. To unsubscribe, or for help writing mail filters, visit settings.

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>