On 09/13/13 17:31, Martin Kletzander wrote: > On 09/13/2013 04:19 PM, Peter Krempa wrote: >> Semantics of the libvirt helper are more clear. This change also allows >> to clean up some pieces of code. >> ---
[...]
>> diff --git a/src/storage/storage_backend_fs.c
>> b/src/storage/storage_backend_fs.c
>> index d305b06..956cfce 100644
>> --- a/src/storage/storage_backend_fs.c
>> +++ b/src/storage/storage_backend_fs.c
>> @@ -503,7 +503,7 @@ virStorageBackendFileSystemCheck(virConnectPtr conn
>> ATTRIBUTE_UNUSED,
>> {
>> *isActive = false;
>> if (pool->def->type == VIR_STORAGE_POOL_DIR) {
>> - if (access(pool->def->target.path, F_OK) == 0)
>> + if (virFileExists(pool->def->target.path))
>> *isActive = true;
>
> Why didn't you use the same (nicer) way here, when you used it
> everywhere else. I'm talking about '*isActive = virFileExists(...)'.
>
> ACK either way, looking forward to [12]/3 ;-)
>
> Martin
>
> P.S.: If you'll need the answer urgently, ping me and I'll see them in
> the archives.
>
I cleaned up the part you've pointed out and pushed this patch.
Thanks
Peter
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
