summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPranith Kumar K <pranithk@gluster.com>2012-03-31 13:30:22 +0530
committerVijay Bellur <vijay@gluster.com>2012-03-31 07:41:14 -0700
commit778c8d616ba2e9b47cafbbe95dbec7f2eb788302 (patch)
treeaec7d552d7c4e5dbb7104170cd7a4e9d3f169fbe
parent06226c19a2b6a8840c0fd88837164f1e2150ba5b (diff)
cluster/afr: Fix frame leak in hardlink self-heal
Change-Id: I8560fc2ba0d989eeb33226f8eb8c7fbad46dfe32 BUG: 765391 Signed-off-by: Pranith Kumar K <pranithk@gluster.com> Reviewed-on: http://review.gluster.com/3056 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
-rw-r--r--xlators/cluster/afr/src/afr-self-heal-entry.c11
1 files changed, 2 insertions, 9 deletions
diff --git a/xlators/cluster/afr/src/afr-self-heal-entry.c b/xlators/cluster/afr/src/afr-self-heal-entry.c
index 766474682be..b57521b9fb9 100644
--- a/xlators/cluster/afr/src/afr-self-heal-entry.c
+++ b/xlators/cluster/afr/src/afr-self-heal-entry.c
@@ -1205,18 +1205,11 @@ afr_sh_entry_impunge_hardlink_cbk (call_frame_t *impunge_frame, void *cookie,
struct iatt *postparent, dict_t *xdata)
{
int call_count = 0;
- call_frame_t *frame = NULL;
- afr_local_t *impunge_local = NULL;
- afr_local_t *local = NULL;
- afr_self_heal_t *impunge_sh = NULL;
- afr_self_heal_t *sh = NULL;
-
- AFR_INIT_SH_FRAME_VALS (impunge_frame, impunge_local, impunge_sh,
- frame, local, sh);
call_count = afr_frame_return (impunge_frame);
if (call_count == 0)
- sh->impunge_done (frame, this, op_ret, op_errno);
+ afr_sh_entry_call_impunge_done (impunge_frame, this,
+ op_ret, op_errno);
return 0;
}