diff options
| author | Pranith Kumar K <pranithk@gluster.com> | 2012-06-04 23:23:45 +0530 | 
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2012-12-12 00:10:59 -0500 | 
| commit | 37cb91fd48e302816d74eb2571ff4afd4c64e709 (patch) | |
| tree | a1a97e3d960e62a0289b20384556e6a3d307bc67 /cli/src/cli.h | |
| parent | 771276b01995540550c09eae85d2f13618b84eb0 (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>
Reviewed-on: https://code.engineering.redhat.com/gerrit/1851
Tested-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'cli/src/cli.h')
0 files changed, 0 insertions, 0 deletions
