From 0b81f2801b7a72130d86c88da938f288430cd3e5 Mon Sep 17 00:00:00 2001 From: Anand Avati Date: Mon, 25 Mar 2013 20:34:43 -0700 Subject: cluster/afr: fix fd leak with unsafe call_resume() Introduce AFR_CALL_RESUME macro which cleans up frame->local, like how AFR_STACK_UNWIND etc. do. Therefore fix leak in afr_fsync() path. Change-Id: I3855d8e7e84dbc44e05f507563b7f722bf9621b8 BUG: 927146 Signed-off-by: Anand Avati Reviewed-on: http://review.gluster.org/4745 Reviewed-by: Pranith Kumar Karampuri Tested-by: Gluster Build System --- xlators/cluster/afr/src/afr-transaction.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'xlators/cluster/afr/src/afr-transaction.c') diff --git a/xlators/cluster/afr/src/afr-transaction.c b/xlators/cluster/afr/src/afr-transaction.c index e54f9c2cb..e766b65cd 100644 --- a/xlators/cluster/afr/src/afr-transaction.c +++ b/xlators/cluster/afr/src/afr-transaction.c @@ -399,7 +399,7 @@ afr_changelog_post_op_cbk (call_frame_t *frame, void *cookie, xlator_t *this, if (call_count == 0) { if (local->transaction.resume_stub) { - call_resume (local->transaction.resume_stub); + AFR_CALL_RESUME (local->transaction.resume_stub); local->transaction.resume_stub = NULL; } -- cgit