]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commitdiff
Ignore listen attribute of <graphics> for type network listens
authorJán Tomko <jtomko@redhat.com>
Thu, 26 Feb 2015 13:33:26 +0000 (14:33 +0100)
committerJán Tomko <jtomko@redhat.com>
Fri, 27 Feb 2015 07:27:21 +0000 (08:27 +0100)
Commit 6992994 started filling the listen attribute
of the parent <graphics> elements from type='network' listens.

When this XML is passed to UpdateDevice, parsing fails:
XML error: graphics listen attribute 10.20.30.40 must match
address attribute of first listen element (found none)

Ignore the address in the parent <graphics> attribute
when no type='address' listens are found,
the same we ignore the address for the <listen> subelements
when parsing inactive XML.

src/conf/domain_conf.c

index d95dd3ebd54d8361524b492749cdfde81a3f3592..9b7ae3f2ae7f246a7acb3397dae93f239cb0bffe 100644 (file)
@@ -9614,12 +9614,16 @@ virDomainGraphicsDefParseXML(xmlNodePtr node,
                         break;
                     }
                 }
-                if (!matched) {
+                if (found && !matched) {
                     virReportError(VIR_ERR_XML_ERROR,
                                    _("graphics listen attribute %s must match address "
                                      "attribute of first listen element (found %s)"),
-                                   listenAddr, found ? found : "none");
+                                   listenAddr, found);
                     goto error;
+                } else if (!found) {
+                    /* quietly ignore listen address if none of the listens
+                     * are of type address */
+                    VIR_FREE(listenAddr);
                 }
             }
         }