[Libvirt-cim] [PATCH] device_parsing: Use default values for vnc graphics device

[Eduardo Lima (Etrunko)]

 libxkutil/device_parsing.c |  24 ++++++++++++++++++++----
 1 files changed, 20 insertions(+), 4 deletions(-)


# HG changeset patch
# User Eduardo Lima (Etrunko) <eblima at br.ibm.com>
# Date 1317234028 10800
# Node ID 2313e472149a557143228949878f946278d0dd3a
# Parent  adc78792781448aca7a1356bd253cbdd689839cb
device_parsing: Use default values for vnc graphics device

This patch fixes the behavior where libvirt-cim loses the graphics device
description after a call to ModifyResourceSettings method. Actually it has
nothing to do with the fact that the domain is running or not. What happens is
that if somehow we can't read either 'listen' or 'port' attributes, the
function will fail and return immediately, skipping the device inclusion.

The default values are based on the ones found in the default_graphics_device
function in src/Virt_VirtualSystemManagementService.c.

Signed-off-by: Eduardo Lima (Etrunko) <eblima at br.ibm.com>

diff --git a/libxkutil/device_parsing.c b/libxkutil/device_parsing.c
--- a/libxkutil/device_parsing.c
+++ b/libxkutil/device_parsing.c
@@ -527,6 +527,17 @@
         return 0;
 }
 
+static char *get_attr_value_default(xmlNode *node, char *attrname,
+                                    const char *default_value)
+{
+        char *ret = get_attr_value(node, attrname);
+
+        if (ret == NULL && default_value != NULL)
+                ret = strdup(default_value);
+
+        return ret;
+}
+
 static int parse_graphics_device(xmlNode *node, struct virt_device **vdevs)
 {
         struct virt_device *vdev = NULL;
@@ -547,13 +558,18 @@
         CU_DEBUG("graphics device type = %s", gdev->type);
 
         if (STREQC(gdev->type, "vnc")) {
-                gdev->dev.vnc.port = get_attr_value(node, "port");
-                gdev->dev.vnc.host = get_attr_value(node, "listen");
+                gdev->dev.vnc.port = get_attr_value_default(node, "port",
+                                                            "-1");
+                gdev->dev.vnc.host = get_attr_value_default(node, "listen",
+                                                            "127.0.0.1");
                 gdev->dev.vnc.keymap = get_attr_value(node, "keymap");
                 gdev->dev.vnc.passwd = get_attr_value(node, "passwd");
-        
-                if (gdev->dev.vnc.port == NULL || gdev->dev.vnc.host == NULL)
+
+                if (gdev->dev.vnc.port == NULL || gdev->dev.vnc.host == NULL) {
+                        CU_DEBUG("Error vnc port '%p' host '%p'",
+                                 gdev->dev.vnc.port, gdev->dev.vnc.host);
                         goto err;
+                }
         }
         else if (STREQC(gdev->type, "sdl")) {
                 gdev->dev.sdl.display = get_attr_value(node, "display");