[Xen-devel] [PATCH 1/2] tools/xenconsole: Use xc_domain_getinfo() correctly

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

Coverity ID: 1055018

Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CC: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
CC: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
---
 tools/console/client/main.c |    6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/tools/console/client/main.c b/tools/console/client/main.c
index 523fc23..38c856a 100644
--- a/tools/console/client/main.c
+++ b/tools/console/client/main.c
@@ -339,7 +339,11 @@ int main(int argc, char **argv)
                xc_interface *xc_handle = xc_interface_open(0,0,0);
                if (xc_handle == NULL)
                        err(errno, "Could not open xc interface");
-               xc_domain_getinfo(xc_handle, domid, 1, &xcinfo);
+               if ( (xc_domain_getinfo(xc_handle, domid, 1, &xcinfo) != 1) ||
+                    (xcinfo.domid != domid) ) {
+                       xc_interface_close(xc_handle);
+                       err(errno, "Failed to get domain information");
+               }
                /* default to pv console for pv guests and serial for hvm 
guests */
                if (xcinfo.hvm)
                        type = CONSOLE_SERIAL;
-- 
1.7.10.4


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