>From Soumya <skod...@redhat.com>: Soumya has uploaded a new change for review.
https://review.gerrithub.io/296062 Change subject: SAL: Ref leak in case of merge of lock entries ...................................................................... SAL: Ref leak in case of merge of lock entries In state_lock(), we pin the cache_entry before processing the lock operation. This shall be unref'ed at the end of the operation unless it is the first lock entry for that cache entry. But in case of overlapping locks, in merge_lock_entry(), once we merge the locks, we remove the older one. This could lead to empty lock list for that cache_entry, but with the earlier ref not taken out resulting in a leak. The fix is to unpin the cache entry if the lock entry removed was the last one in its lock list. Change-Id: I9ed9e9ad8115d1e3c5dcc28bcc8e733b94b82de2 Signed-off-by: Soumya Koduri <skod...@redhat.com> --- M src/SAL/state_lock.c 1 file changed, 7 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/62/296062/1 -- To view, visit https://review.gerrithub.io/296062 To unsubscribe, visit https://review.gerrithub.io/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I9ed9e9ad8115d1e3c5dcc28bcc8e733b94b82de2 Gerrit-PatchSet: 1 Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Owner: Soumya <skod...@redhat.com> ------------------------------------------------------------------------------ _______________________________________________ Nfs-ganesha-devel mailing list Nfs-ganesha-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
