[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-changelog] [xen stable-4.2] tools/ocaml: fix erroneous free of cpumap in stub_xc_vcpu_getaffinity



commit 4d747f7c638b6cc6fce8f98ce89b53b4eefc991a
Author:     Matthew Daley <mattjd@xxxxxxxxx>
AuthorDate: Tue Sep 10 23:12:45 2013 +1200
Commit:     Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
CommitDate: Thu Oct 10 16:59:41 2013 +0100

    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@xxxxxxxxx>
    Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
    (cherry picked from commit 3cd10fd21220f2b814324e6e732004f8f0487d0a)
    (cherry picked from commit debfacf7d68de8e39a06ebc7f7b22386b28ce6fb)
---
 tools/ocaml/libs/xc/xenctrl_stubs.c |    2 --
 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 0e26ae9..a144313 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));
--
generated by git-patchbot for /home/xen/git/xen.git#stable-4.2

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.