[libvirt] [PATCH] qemu: Allow loading snapshot configs starting with a dot
Eric Blake
eblake at redhat.com
Mon Jan 21 20:20:30 UTC 2013
On 01/21/2013 12:58 PM, Daniel P. Berrange wrote:
> On Mon, Jan 21, 2013 at 08:46:55PM +0100, Peter Krempa wrote:
>> Relax the check for the period on start of a snapshot name to skip only
>> the '.' and '..' directories.
>> ---
>> src/qemu/qemu_driver.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
>> index 3a54228..637e599 100644
>> --- a/src/qemu/qemu_driver.c
>> +++ b/src/qemu/qemu_driver.c
>> @@ -520,7 +520,7 @@ qemuDomainSnapshotLoad(void *payload,
>> }
>>
>> while ((entry = readdir(dir))) {
>> - if (entry->d_name[0] == '.')
>> + if (STREQ(entry->d_name, ".") || STREQ(entry->d_name, ".."))
>> continue;
>>
>> /* NB: ignoring errors, so one malformed config doesn't
>
> I'm not entirely convinced we should allow snapshots with a leading '.'
> to be created in the first place.
I agree with Dan - I'd rather see a patch that goes the other direction
and explicitly refuse to create a snapshot with a leading dot, just as
we now explicitly refuse to create a snapshot with a slash.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130121/799f0acc/attachment-0001.sig>
More information about the libvir-list
mailing list