On Thu, Sep 08, 2011 at 02:17:10PM +0100, Eric Blake wrote: > Regression introduced in commit d6f6b2d194c. > > * src/conf/domain_conf.c (virDomainSnapshotDefParseString): Only > give error about no disk support when <disk> was found. > --- > src/conf/domain_conf.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c > index 74f8d6a..560c773 100644 > --- a/src/conf/domain_conf.c > +++ b/src/conf/domain_conf.c > @@ -11622,7 +11622,7 @@ virDomainSnapshotDefParseString(const char *xmlStr, > goto cleanup; > } > VIR_FREE(nodes); > - } else { > + } else if (i) { > virDomainReportError(VIR_ERR_ARGUMENT_UNSUPPORTED, "%s", > _("unable to handle disk requests in snapshot")); > goto cleanup; Okay, ACK, Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@xxxxxxxxxxxx | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/ -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list