summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKrutika Dhananjay <kdhananj@redhat.com>2016-11-16 18:26:52 +0530
committerPranith Kumar Karampuri <pkarampu@redhat.com>2016-11-16 22:26:01 -0800
commit65f785bb9b484e42eac39af0e468a0a9f46c8e99 (patch)
tree6e559dabb4966a27d28f21ca1fe684f5f7fcfd65
parentcb90ef1142da5123f3db74b9af0e1c7bcfe3747c (diff)
protocol/client: Fix iobref and iobuf leaks in COMPOUND fop
Backport of: http://review.gluster.org/15860 Change-Id: I5873bd0ae9a2df91876b1c9c9d8afdec9c5151f9 BUG: 1395694 Signed-off-by: Krutika Dhananjay <kdhananj@redhat.com> Reviewed-on: http://review.gluster.org/15861 Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com> Smoke: Gluster Build System <jenkins@build.gluster.org> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
-rw-r--r--xlators/protocol/client/src/client-rpc-fops.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/xlators/protocol/client/src/client-rpc-fops.c b/xlators/protocol/client/src/client-rpc-fops.c
index 8085aec0765..a7b88baa8bb 100644
--- a/xlators/protocol/client/src/client-rpc-fops.c
+++ b/xlators/protocol/client/src/client-rpc-fops.c
@@ -6158,7 +6158,6 @@ client3_3_compound (call_frame_t *frame, xlator_t *this, void *data)
rsphdr->iov_len = iobuf_pagesize (rsphdr_iobuf);
rsphdr_count = 1;
rsphdr_iobuf = NULL;
- rsphdr_iobref = NULL;
req.compound_fop_enum = c_args->fop_enum;
req.compound_req_array.compound_req_array_len = c_args->fop_length;
@@ -6208,6 +6207,8 @@ client3_3_compound (call_frame_t *frame, xlator_t *this, void *data)
GF_FREE (req.xdata.xdata_val);
+ iobref_unref (rsphdr_iobref);
+
compound_request_cleanup (&req);
return 0;
unwind: