Re: [libvirt] [PATCH] qemu: Allow loading snapshot configs starting with a dot

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.