diff options
| author | Atin Mukherjee <amukherj@redhat.com> | 2015-03-28 21:09:57 +0530 |
|---|---|---|
| committer | Krishnan Parthasarathi <kparthas@redhat.com> | 2015-03-29 19:40:28 -0700 |
| commit | d5a9607598cf02e50bce306cffe97990a50129c0 (patch) | |
| tree | ad776b338366c4d4f8925b8516b92095537d7d31 /xlators/mgmt/glusterd | |
| parent | 0934432c51aef8ae7e890c3fc2ff2ae843b46698 (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>
Diffstat (limited to 'xlators/mgmt/glusterd')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-handler.c | 2 |
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; |