[libvirt] [PATCH 34/26] snapshot: add <disks> to snapshot xml

Eric Blake eblake at redhat.com
Tue Aug 23 14:18:37 UTC 2011


On 08/19/2011 03:58 PM, Eric Blake wrote:
> Adds an optional element to<domainsnapshot>, which will be used
> to give user control over external snapshot filenames on input,
> and specify generated filenames on output.
>
> @@ -11044,6 +11110,19 @@ virDomainSnapshotDefParseString(const char *xmlStr,
>
>       def->description = virXPathString("string(./description)", ctxt);
>
> +    if ((i = virXPathNodeSet("./disks/*)", ctxt,&nodes))<  0)

Typo.  Squash this in:

diff --git i/src/conf/domain_conf.c w/src/conf/domain_conf.c
index c9daebb..40ba564 100644
--- i/src/conf/domain_conf.c
+++ w/src/conf/domain_conf.c
@@ -11137,7 +11137,7 @@ virDomainSnapshotDefParseString(const char *xmlStr,

      def->description = virXPathString("string(./description)", ctxt);

-    if ((i = virXPathNodeSet("./disks/*)", ctxt, &nodes)) < 0)
+    if ((i = virXPathNodeSet("./disks/*", ctxt, &nodes)) < 0)
          goto cleanup;
      def->ndisks = i;
      if (def->ndisks && VIR_ALLOC_N(def->disks, def->ndisks) < 0) {

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org




More information about the libvir-list mailing list