diff options
author | Pranith Kumar K <pranithk@gluster.com> | 2012-06-04 23:23:45 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2012-08-30 03:25:50 -0700 |
commit | e908ee20e8a87ba1170f228dd4dd5993f36a0da6 (patch) | |
tree | 3c46db9f78b37ca84a1c6212bb2a2d6fba962020 /glusterfsd | |
parent | f382d5c5656301d9b426dc50a95e8c54db42c90e (diff) |
Self-heald: Fix inode leak
RCA:
There is an inode-leak because inode_link returns
linked inode by taking a reference. That needs to be
unreffed.
Fix:
Added the code to perform unrefs. In addition to that
updated the loc inode with the linked-inode because that is
the best practice. The code to update the input inode's
gfid can be removed later, its already removed in master.
Tests:
Checked that opendir comes with an loc with valid inode
Checked that re-opendir happens successfully. Tested index,
full self-heal work fine with the fix.
BUG: 826580
Change-Id: I0c68192ff98f76152ed112b393d497b8fee93355
Signed-off-by: Pranith Kumar K <pranithk@gluster.com>
Reviewed-on: http://review.gluster.org/3518
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Raghavendra Bhat <raghavendra@redhat.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'glusterfsd')
0 files changed, 0 insertions, 0 deletions