shaharhoch@google.com has uploaded this change for review.

View Change

Fix dir chunk ref count leak

A recent change added a skip to a failure flow, but it called `mdcache_lru_unref_chunk` before the skip, but if we goto skip, we also unref the chunk there.

This causes us to to dec-ref the chunk twice and release it while other dirs still reference to it.
This can cause us memory corruption.

Change-Id: I685b8e6977deeaa670cd80ebed041e79c8039252
Signed-off-by: Shahar Hochma <shaharhoch@google.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c
1 file changed, 2 insertions(+), 2 deletions(-)

git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/13/1238513/1

To view, visit change 1238513. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I685b8e6977deeaa670cd80ebed041e79c8039252
Gerrit-Change-Number: 1238513
Gerrit-PatchSet: 1
Gerrit-Owner: shaharhoch@google.com