diff options
| author | Anand Avati <avati@gluster.com> | 2010-12-29 12:23:08 +0000 |
|---|---|---|
| committer | Anand V. Avati <avati@dev.gluster.com> | 2010-12-29 10:01:09 -0800 |
| commit | 818a4d128048721d70be2d6c8eafd2f41034b260 (patch) | |
| tree | cef7c1ce73fe962d53aae2b797f5f1f279c140a4 /xlators/cluster/stripe/src/stripe.c | |
| parent | 98a1f381ba15dfc9c19ea3f9d778e8efb9e6be8e (diff) | |
stripe: fix memory leak
destroy the frame copied and clean the local structure in stripe to avoid memory leak
Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com>
Signed-off-by: Anand V. Avati <avati@amp.gluster.com>
Signed-off-by: Anand V. Avati <avati@dev.gluster.com>
BUG: 2253 (Memory leak in glusterfs)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2253
Diffstat (limited to 'xlators/cluster/stripe/src/stripe.c')
| -rw-r--r-- | xlators/cluster/stripe/src/stripe.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/xlators/cluster/stripe/src/stripe.c b/xlators/cluster/stripe/src/stripe.c index bda38298f..639fbfd8b 100644 --- a/xlators/cluster/stripe/src/stripe.c +++ b/xlators/cluster/stripe/src/stripe.c @@ -203,6 +203,8 @@ stripe_entry_self_heal (call_frame_t *frame, xlator_t *this, } out: + if (rframe) + STRIPE_STACK_DESTROY (rframe); if (dict) dict_unref (dict); @@ -3389,6 +3391,8 @@ stripe_readv (call_frame_t *frame, xlator_t *this, fd_t *fd, err: if (local && local->fd) fd_unref (local->fd); + if (rframe) + STRIPE_STACK_DESTROY (rframe); STRIPE_STACK_UNWIND (readv, frame, -1, op_errno, NULL, 0, NULL, NULL); return 0; |