summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAtin Mukherjee <amukherj@redhat.com>2015-03-28 21:09:57 +0530
committerKrishnan Parthasarathi <kparthas@redhat.com>2015-03-29 19:40:28 -0700
commitd5a9607598cf02e50bce306cffe97990a50129c0 (patch)
treead776b338366c4d4f8925b8516b92095537d7d31
parent0934432c51aef8ae7e890c3fc2ff2ae843b46698 (diff)
glusterd : fix memory overrun for opinfo.local_xaction_peers
Due to incorrect sizeof local_xaction_peers was not allocated as intended which lead a memory overrun in gf_free while freeing local_xaction_peers. Change-Id: Ie4d63289aae51727eea091bce0e6cb0a496ce7cf BUG: 1204727 Signed-off-by: Atin Mukherjee <amukherj@redhat.com> Reviewed-on: http://review.gluster.org/10032 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Emmanuel Dreyfus <manu@netbsd.org> Tested-by: Emmanuel Dreyfus <manu@netbsd.org> Reviewed-by: Krishnan Parthasarathi <kparthas@redhat.com> Tested-by: Krishnan Parthasarathi <kparthas@redhat.com>
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-handler.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c
index 52143bb3fb8..62fc0e6f053 100644
--- a/xlators/mgmt/glusterd/src/glusterd-handler.c
+++ b/xlators/mgmt/glusterd/src/glusterd-handler.c
@@ -626,7 +626,7 @@ glusterd_op_txn_begin (rpcsvc_request_t *req, glusterd_op_t op, void *ctx,
local_locking_done:
txn_op_info.local_xaction_peers =
- GF_CALLOC (1, sizeof (struct cds_list_head *),
+ GF_CALLOC (1, sizeof (struct cds_list_head),
gf_common_mt_list_head_t);
if (!txn_op_info.local_xaction_peers) {
ret = -1;