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 16:06:17 +0100
commitdebfacf7d68de8e39a06ebc7f7b22386b28ce6fb (patch)
tree234ec0a03d8489f59f1f49122a8b2c1d273b08bf
parent60aefd150bc0ad0c7d325da5ffea0bf4e0544130 (diff)
downloadxen-debfacf7d68de8e39a06ebc7f7b22386b28ce6fb.tar.gz
xen-debfacf7d68de8e39a06ebc7f7b22386b28ce6fb.tar.bz2
xen-debfacf7d68de8e39a06ebc7f7b22386b28ce6fb.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> (cherry picked from commit 3cd10fd21220f2b814324e6e732004f8f0487d0a)
-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));