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

[Xen-changelog] [xen master] libxenctrl: Fix xc_interface_close() crash if it gets NULL as an argument



commit 2cebe22e6924439535cbf4a9f82a7d9d30c8f9c7
Author:     Daniel Kiper <daniel.kiper@xxxxxxxxxx>
AuthorDate: Mon Dec 2 20:13:03 2013 +0100
Commit:     Ian Campbell <ian.campbell@xxxxxxxxxx>
CommitDate: Wed Dec 4 14:33:00 2013 +0000

    libxenctrl: Fix xc_interface_close() crash if it gets NULL as an argument
    
    xc_interface_close() crashes if it gets NULL as an argument. However,
    it just calls xc_interface_close_common() which is called by many
    others functions. It means that they are also vulnerable. So fix above
    mentioned issue by adding NULL check in xc_interface_close_common().
    This way we fix similar issue in other functions which calls
    xc_interface_close_common() too.
    
    Signed-off-by: Daniel Kiper <daniel.kiper@xxxxxxxxxx>
    Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
 tools/libxc/xc_private.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/tools/libxc/xc_private.c b/tools/libxc/xc_private.c
index 524862e..838fd21 100644
--- a/tools/libxc/xc_private.c
+++ b/tools/libxc/xc_private.c
@@ -198,6 +198,9 @@ static int xc_interface_close_common(xc_interface *xch)
 {
     int rc = 0;
 
+    if (!xch)
+       return 0;
+
     xc__hypercall_buffer_cache_release(xch);
 
     xtl_logger_destroy(xch->dombuild_logger_tofree);
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
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®.