diff options
| author | Vijay Bellur <vbellur@redhat.com> | 2016-03-13 10:44:12 -0400 |
|---|---|---|
| committer | Niels de Vos <ndevos@redhat.com> | 2016-03-16 10:21:03 -0700 |
| commit | 8fda324df01b6de9c58a1395263ce9755465b26d (patch) | |
| tree | b428ad9511d96f0abed23d663030c9373212c6d6 /xlators/mount/fuse/src | |
| parent | 500ad8f3a72053d33120657e8a2e93d844041cf0 (diff) | |
mount/fuse: cleanup an additional inode_ref()
commit ca515db0127 introduced a check in
fuse_resolve_inode_simple(). This results in an additional
ref being held on inodes which were obtained through readdirp.
As a result, the inode table keeps growing and entries remain in
the active list even after deletion of such inodes.
Change-Id: I780ec5513990d6ef00ea051ec57ff20e4428081e
BUG: 1317948
Signed-off-by: Vijay Bellur <vbellur@redhat.com>
Reviewed-on: http://review.gluster.org/13689
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Smoke: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Niels de Vos <ndevos@redhat.com>
Diffstat (limited to 'xlators/mount/fuse/src')
| -rw-r--r-- | xlators/mount/fuse/src/fuse-resolve.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/xlators/mount/fuse/src/fuse-resolve.c b/xlators/mount/fuse/src/fuse-resolve.c index a0352bcf43f..7d3494f5419 100644 --- a/xlators/mount/fuse/src/fuse-resolve.c +++ b/xlators/mount/fuse/src/fuse-resolve.c @@ -256,8 +256,11 @@ fuse_resolve_parent_simple (fuse_state_t *state) * have been there even though it need not have (bug #804592). */ - if (loc->inode && inode_needs_lookup (loc->inode, THIS)) + if (loc->inode && inode_needs_lookup (loc->inode, THIS)) { + inode_unref (loc->inode); + loc->inode = NULL; return -1; + } if ((loc->inode == NULL) && __is_root_gfid (parent->gfid)) { |