[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