summaryrefslogtreecommitdiffstats
path: root/heal
diff options
context:
space:
mode:
authorRavishankar N <ravishankar@redhat.com>2014-12-01 10:08:53 +0000
committerPranith Kumar Karampuri <pkarampu@redhat.com>2014-12-04 02:31:33 -0800
commit811abf66d158efa037f1314792f7c674a7640c50 (patch)
tree59894506581228ca7caef46259fa8b1c5262f3a3 /heal
parentac4c203bfb4c3ebe48a08ef695ee462ba9b5e2c7 (diff)
heal: free leaked frames.
'gluster v heal <VOLNAME> info` waits for about 10 seconds before printing the heal info and exiting. Problem: glfsh_process_entries() is not freeing the frames it created. Thus when glfs_fini() is called, it busy waits for 10 seconds for background frames to finish before returning error. Fix: Destroy the frames in that are created in glfsh_process_entries(). Change-Id: I5484d1b7301a355b913ae9c15b8a62471036c755 BUG: 1169335 Signed-off-by: Ravishankar N <ravishankar@redhat.com> Reviewed-on: http://review.gluster.org/9224 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com> Tested-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
Diffstat (limited to 'heal')
-rw-r--r--heal/src/glfs-heal.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/heal/src/glfs-heal.c b/heal/src/glfs-heal.c
index e01e98df0e9..1056ea467c4 100644
--- a/heal/src/glfs-heal.c
+++ b/heal/src/glfs-heal.c
@@ -350,6 +350,8 @@ out:
inode_unref (inode);
}
+ if (frame)
+ AFR_STACK_DESTROY (frame);
GF_FREE (path);
return ret;
}