[libvirt] [PATCH 1/3] qemuDomainBuildNamespace: Clean up temp files
Michal Privoznik
mprivozn at redhat.com
Thu Jun 15 08:53:05 UTC 2017
On 06/14/2017 09:50 PM, John Ferlan wrote:
>
>
> On 06/12/2017 11:57 AM, Michal Privoznik wrote:
>> https://bugzilla.redhat.com/show_bug.cgi?id=1431112
>>
>> After 290a00e41d we know how to deal with file mount points.
>> However, when cleaning up the temporary location for preserved
>> mount points we are still calling rmdir(). This won't fly for
>> files. We need to call unlink(). Now, since we don't really care
>> if the cleanup succeeded or not (it's the best effort anyway), we
>> can call both rmdir() and unlink() without need for
>> differentiation between files and directories.
>>
>> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>> ---
>> src/qemu/qemu_domain.c | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>
> But why call both?
I don't think you can call unlink() over a directory, can you? And sure,
I could call stat() just to find out if it's a dir or a file and call
just one of the pair. Or I can call both and ignore any errors. The
result is the same, isn't it?
Michal
More information about the libvir-list
mailing list