On 09/11/2014 09:47 AM, Peter Krempa wrote: > If a floppy drive isn't selected for snapshot explicitly and is empty > don't try to snapshot it. For external snapshots this would fail as we > can't generate a name for the snapshot from an empty drive.
Do we need the same for cdrom drives? > > Reported-by: Pavel Hrdina <phrd...@redhat.com> > --- > src/conf/snapshot_conf.c | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/src/conf/snapshot_conf.c b/src/conf/snapshot_conf.c > index c53a66b..cbaff74 100644 > --- a/src/conf/snapshot_conf.c > +++ b/src/conf/snapshot_conf.c > @@ -561,7 +561,14 @@ virDomainSnapshotAlignDisks(virDomainSnapshotDefPtr def, > if (VIR_STRDUP(disk->name, def->dom->disks[i]->dst) < 0) > goto cleanup; > disk->index = i; > - disk->snapshot = def->dom->disks[i]->snapshot; > + > + /* Don't snapshot empty floppy drives */ > + if (def->dom->disks[i]->device == VIR_DOMAIN_DISK_DEVICE_FLOPPY && > + !virDomainDiskGetSource(def->dom->disks[i])) If we are worried about ALL empty drives, it would be simpler to just drop the left side of the &&, making it solely a test of whether there is currently a defined host source. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
