Re: [PATCH 2/6] secret: util: Refactor virSecretGetSecretString

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 05/13/2016 12:04 PM, Peter Krempa wrote:
> Call the internal driver callbacks rather than the public APIs to avoid
> calling unnecessarily the error dispatching code and don't overwrite
> the error messages provided by the APIs. They are good enough to
> describe which secret is missing either by UUID or the usage (basically
> name).
> ---
>  po/POTFILES.in           |  1 -
>  src/libxl/libxl_conf.c   |  3 ---
>  src/qemu/qemu_domain.c   |  4 +---
>  src/secret/secret_util.c | 39 +++++++--------------------------------
>  src/secret/secret_util.h |  1 -
>  5 files changed, 8 insertions(+), 40 deletions(-)
> 

[...]

> --- a/src/secret/secret_util.h
> +++ b/src/secret/secret_util.h
> @@ -26,7 +26,6 @@
>  # include "virstoragefile.h"
> 
>  char *virSecretGetSecretString(virConnectPtr conn,
> -                               const char *scheme,
>                                 bool encoded,
>                                 virStorageAuthDefPtr authdef,
>                                 virSecretUsageType secretUsageType)
> 

Need to adjust the ATTRIBUTE_NONNULL... since (2) is being removed, then
(4) becomes (3) too...

John

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]