andy/glusterfs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

heal: free leaked frames.

Browse Source 
'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>
This commit is contained in:
Ravishankar N 2014-12-01 10:08:53 +00:00 committed by Pranith Kumar Karampuri
parent ac4c203bfb
commit 811abf66d1
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
heal/src/glfs-heal.c
View File

@ -350,6 +350,8 @@ out:
inode_unref (inode);
}
if (frame)
AFR_STACK_DESTROY (frame);
GF_FREE (path);
return ret;
}