On 11/19/2012 05:09 PM, Eric Blake wrote:
> When asked to parse a branch snapshot XML definition, we have to
> piece together the definition of the new snapshot from parts of
> the branch point, as well as run some sanity checks that the
> branches are compatible. This patch is rather restrictive in
> what it allows; depending on effort and future needs, we may be
> able to relax some of those restrictions and allow more cases.
>
> + if (other) {
> + /* For now, we only allow branch snapshots of external snapshots. */
> + if (virDomainSnapshotAlignDisks(def,
> + VIR_DOMAIN_SNAPSHOT_LOCATION_EXTERNAL,
> + true) < 0)
> + goto cleanup;
> + for (i = 0; i < def->ndisks; def++)
You can tell I was sending this late yesterday, with blatant bugs like
iterating the wrong variable. Now that I've actually tested the patch,
and had valgrind point out the obvious, my v3 should be something that
actually works.
--
Eric Blake eblake@xxxxxxxxxx +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list