Re: [Xen-devel] [PATCH 00/13] Coverity fixes for libxl

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

On 01/12/2013 10:14, Matthew Daley wrote:

All except for patches 4 and 8 (which have followups),

Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

> Matthew Daley (13):
>   libxl: fix unsigned less-than-0 comparison in e820_sanitize
>   libxl: check for xc_domain_setmaxmem failure in libxl__build_pre
>   libxl: correct file open success check in libxl__device_pci_reset
>   libxl: don't leak p in libxl__wait_for_backend
>   libxl: remove unsigned less-than-0 comparison
>   libxl: actually abort if initializing a ctx's lock fails
>   libxl: don't leak output vcpu info on error in libxl_list_vcpu
>   libxl: don't leak ptr in libxl_list_vm error case
>   libxl: don't leak pcidevs in libxl_pcidev_assignable
>   libxl: don't try to fclose file twice on error in
>     libxl_userdata_store
>   libxl: use pipe instead of temporary file for VNC viewer --autopass
>   libxl: don't leak buf in libxl_xen_console_read_start error handling
>   libxl: replace for loop with more idiomatic do-while loop
>
>  tools/libxl/libxl.c        |   64 
> +++++++++++++++++++++-----------------------
>  tools/libxl/libxl_cpuid.c  |    2 +-
>  tools/libxl/libxl_device.c |    3 +++
>  tools/libxl/libxl_dom.c    |   42 ++++++++++++++---------------
>  tools/libxl/libxl_pci.c    |    9 +++----
>  tools/libxl/libxl_x86.c    |    2 +-
>  6 files changed, 59 insertions(+), 63 deletions(-)
>


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel