diff options
| author | Pranith Kumar K <pranithk@gluster.com> | 2012-03-31 13:30:22 +0530 | 
|---|---|---|
| committer | Vijay Bellur <vijay@gluster.com> | 2012-03-31 07:41:14 -0700 | 
| commit | 778c8d616ba2e9b47cafbbe95dbec7f2eb788302 (patch) | |
| tree | aec7d552d7c4e5dbb7104170cd7a4e9d3f169fbe | |
| parent | 06226c19a2b6a8840c0fd88837164f1e2150ba5b (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.c | 11 | 
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;  }  | 
