diff options
author | Christopher R. Hertel <crh@redhat.com> | 2014-02-06 12:39:24 -0600 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2014-02-10 08:38:57 -0800 |
commit | c1c3ab9e4c96e059ba1abc1a0b5ba221ef6e3d8d (patch) | |
tree | 47ed8e39d9af184a5e356bc76253168fc5147964 /extras/benchmarking/glfs-bm.c | |
parent | 0d23c99e46b3752b39487cb9268457a2f5e768fe (diff) |
cluster/afr: goto statements may cause exit before memory is freed.
Memory is allocated for pump_priv and for pump_priv->resume_path, but if
an error is detected the references to that memory go out of scope and
the memory is never freed.
This patch assures that the memory is freed on error.
Patchset 2: These are Kaleb's recommended changes which, compared to my
original fix, are more comprehensive and provide a more
complete resolution to the memory leakage bugs in this
function. The bug reported by Coverity was limited to a
single memory allocation.
BUG: 789278
CID: 1124737
Change-Id: Ie239e3b5d28d97308bf948efec6a92f107bc648b
Signed-off-by: Christopher R. Hertel <crh@redhat.com>
Reviewed-on: http://review.gluster.org/6929
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'extras/benchmarking/glfs-bm.c')
0 files changed, 0 insertions, 0 deletions