From: Ian Campbell <ian.campbell@citrix.com>
Date: Wed, 17 Feb 2016 15:39:55 +0000 (+0000)
Subject: xenpaging: don't try to log via xch if xc_interface_close fails
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=14c679a4000ef5bb2c7e622fa986912b5e4c98a0;p=people%2Fliuw%2Fxen.git

xenpaging: don't try to log via xch if xc_interface_close fails

Since xch may not be valid (enough) any longer, xc_interface_close
already logs anything of any use before it tears down the integrated
logger so there is no need to log any further in the application via
that path.

CID: 1056203

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
---

diff --git a/tools/xenpaging/xenpaging.c b/tools/xenpaging/xenpaging.c
index 6157d3a4b6..d0571cabac 100644
--- a/tools/xenpaging/xenpaging.c
+++ b/tools/xenpaging/xenpaging.c
@@ -550,11 +550,7 @@ static void xenpaging_teardown(struct xenpaging *paging)
     xs_close(paging->xs_handle);
 
     /* Close connection to Xen */
-    rc = xc_interface_close(xch);
-    if ( rc != 0 )
-    {
-        ERROR("Error closing connection to xen");
-    }
+    xc_interface_close(xch);
 }
 
 static void get_request(struct vm_event *vm_event, vm_event_request_t *req)