Re: [libvirt] [PATCH] snapshot: conf: Forbid using same file as external snapshot for disks and external snapshot for memory

On 10/02/2014 11:17 AM, Shanzhi Yu wrote: This is a case of user stupidity. We can't always protect the user from shooting themselves in the foot. If it is easy, it may be worth adding, but I'm not convinced it is easy. A more foolproof (but still not bulletproof) approach would be to track the memspec via the lease manager, since the lease manager already avoids collisions even across situations that aren't easy to spot by mere string equality. But that has it's own set of design questions (to date, the lease manager covers only disk images, not saved state images). STREQ_NULLABLE is a little more compact. However, this check is very weak - it does not detect things like 'a/b' and 'a//b' being the same file. I also think it is better to detect this via stat() at the point later on in the process where we sanitize the XML, rather than just here at parse time (where a user can leave the disk destination blank and rely on later code supplying a sane default, which would miss a collision with the memspec also using what would be the generated disk name). Thus, I think that if we are going to do this at all, it should be done in virDomainSnapshotAlignDisks(). -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org