aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatthew Daley <mattjd@gmail.com>2013-09-10 23:12:45 +1200
committerIan Jackson <Ian.Jackson@eu.citrix.com>2013-10-10 15:49:40 +0100
commit3cd10fd21220f2b814324e6e732004f8f0487d0a (patch)
tree65e8003a4ec5e1c51fc19c3fadbb03d75a17da1f
parentc53702cee1d6f9f1b72f0cae0b412e21bcda8724 (diff)
downloadxen-3cd10fd21220f2b814324e6e732004f8f0487d0a.tar.gz
xen-3cd10fd21220f2b814324e6e732004f8f0487d0a.tar.bz2
xen-3cd10fd21220f2b814324e6e732004f8f0487d0a.zip
tools/ocaml: fix erroneous free of cpumap in stub_xc_vcpu_getaffinity
Not sure how it got there... Coverity-ID: 1056196 This is CVE-2013-4370 / XSA-69 Signed-off-by: Matthew Daley <mattjd@gmail.com> Acked-by: Ian Campbell <ian.campbell@citrix.com>
-rw-r--r--tools/ocaml/libs/xc/xenctrl_stubs.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/tools/ocaml/libs/xc/xenctrl_stubs.c b/tools/ocaml/libs/xc/xenctrl_stubs.c
index df756ad539..f5cf0edb24 100644
--- a/tools/ocaml/libs/xc/xenctrl_stubs.c
+++ b/tools/ocaml/libs/xc/xenctrl_stubs.c
@@ -461,8 +461,6 @@ CAMLprim value stub_xc_vcpu_getaffinity(value xch, value domid,
retval = xc_vcpu_getaffinity(_H(xch), _D(domid),
Int_val(vcpu), c_cpumap);
- free(c_cpumap);
-
if (retval < 0) {
free(c_cpumap);
failwith_xc(_H(xch));