[libvirt] [PATCHv9] snapshot: sanity check when reusing file for snapshot

Peter Krempa pkrempa at redhat.com
Mon Oct 22 21:03:02 UTC 2012


On 10/22/12 22:34, Eric Blake wrote:
> The snapshot code when reusing an existing file had hard-to-read
> logic, as well as a missing sanity check: REUSE_EXT should require
> the destination to already be present.
>
> * src/qemu/qemu_driver.c (qemuDomainSnapshotDiskPrepare): Require
> destination on REUSE_EXT, rename variable for legibility.
> ---
>
> v9: separate out from larger series, as this one is ready now;
> provide diff with more context and a renamed variable.
> https://www.redhat.com/archives/libvir-list/2012-October/msg01107.html
>
>   src/qemu/qemu_driver.c | 11 ++++++++---
>   1 file changed, 8 insertions(+), 3 deletions(-)
>

ACK

Peter




More information about the libvir-list mailing list