[libvirt] [PATCH] storage_fs: Create directory with UID if needed

[Ján Tomko]

On Wed, May 27, 2015 at 10:10:53AM +0200, Martin Kletzander wrote:
> The code already exists there, it just modified different flags.  I just
> noticed this when looking at the code.  This patch is better to view
> with bigger context or '-W'.
> 
> Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
> ---
>  src/storage/storage_backend_fs.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/storage/storage_backend_fs.c b/src/storage/storage_backend_fs.c
> index bcbbb3ae252a..5dc712925b27 100644
> --- a/src/storage/storage_backend_fs.c
> +++ b/src/storage/storage_backend_fs.c
> @@ -1,7 +1,7 @@
>  /*
>   * storage_backend_fs.c: storage backend for FS and directory handling
>   *
> - * Copyright (C) 2007-2014 Red Hat, Inc.
> + * Copyright (C) 2007-2015 Red Hat, Inc.
>   * Copyright (C) 2007-2008 Daniel P. Berrange
>   *
>   * This library is free software; you can redistribute it and/or

Please configure your editor to not generate noise.

> @@ -807,7 +807,7 @@ virStorageBackendFileSystemBuild(virConnectPtr conn ATTRIBUTE_UNUSED,
>          (needs_create_as_uid || !virFileExists(pool->def->target.path)))
>          mode = VIR_STORAGE_DEFAULT_POOL_PERM_MODE;
>      if (needs_create_as_uid)
> -        flags |= VIR_DIR_CREATE_AS_UID;
> +        dir_create_flags |= VIR_DIR_CREATE_AS_UID;
> 
>      /* Now create the final dir in the path with the uid/gid/mode
>       * requested in the config. If the dir already exists, just set

ACK to this hunk.

Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150527/1bd4c50b/attachment-0001.sig>