Re: [PATCH 01/13] tools: Fix vshControl declaration and initialization

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

 



On 8/3/23 12:36, Michal Privoznik wrote:
> Both virsh and virt-admin have vshControl typed variables and
> also pointers to these variables. In both cases these are
> declared on a single line. Do the following:
> 
>   1) break declaration into two lines,
>   2) use struct zero initializer for vshControl and
>      virshControl/vshAdmControl structs,
>   3) drop explicit memset(.., 0, ...) ;
> 
> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

Reviewed-by: Claudio Fontana <cfontana@xxxxxxx>

> ---
>  tools/virsh.c      | 7 +++----
>  tools/virt-admin.c | 7 +++----
>  2 files changed, 6 insertions(+), 8 deletions(-)
> 
> diff --git a/tools/virsh.c b/tools/virsh.c
> index 963e886860..40c23e4180 100644
> --- a/tools/virsh.c
> +++ b/tools/virsh.c
> @@ -831,12 +831,11 @@ static const vshClientHooks hooks = {
>  int
>  main(int argc, char **argv)
>  {
> -    vshControl _ctl, *ctl = &_ctl;
> -    virshControl virshCtl;
> +    vshControl _ctl = { 0 };
> +    vshControl *ctl = &_ctl;
> +    virshControl virshCtl = { 0 };
>      bool ret = true;
>  
> -    memset(ctl, 0, sizeof(vshControl));
> -    memset(&virshCtl, 0, sizeof(virshControl));
>      ctl->name = "virsh";        /* hardcoded name of the binary */
>      ctl->env_prefix = "VIRSH";
>      ctl->log_fd = -1;           /* Initialize log file descriptor */
> diff --git a/tools/virt-admin.c b/tools/virt-admin.c
> index db246dc3a6..9d01890447 100644
> --- a/tools/virt-admin.c
> +++ b/tools/virt-admin.c
> @@ -1554,12 +1554,11 @@ static const vshClientHooks hooks = {
>  int
>  main(int argc, char **argv)
>  {
> -    vshControl _ctl, *ctl = &_ctl;
> -    vshAdmControl virtAdminCtl;
> +    vshControl _ctl = { 0 };
> +    vshControl *ctl = &_ctl;
> +    vshAdmControl virtAdminCtl = { 0 };
>      bool ret = true;
>  
> -    memset(ctl, 0, sizeof(vshControl));
> -    memset(&virtAdminCtl, 0, sizeof(vshAdmControl));
>      ctl->name = "virt-admin";        /* hardcoded name of the binary */
>      ctl->env_prefix = "VIRT_ADMIN";
>      ctl->log_fd = -1;                /* Initialize log file descriptor */




[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]

  Powered by Linux