[libvirt] [PATCH 03/42] rpc: Use g_autofree on virNetClientNewLibSSH2()

Daniel Henrique Barboza danielhb413 at gmail.com
Thu Dec 19 19:00:38 UTC 2019



On 12/19/19 7:04 AM, Fabiano Fidêncio wrote:
> Signed-off-by: Fabiano Fidêncio <fidencio at redhat.com>
> ---
>   src/rpc/virnetclient.c | 18 ++++++------------
>   1 file changed, 6 insertions(+), 12 deletions(-)
> 
[...]

>       if (knownHostsPath) {
> @@ -517,12 +517,6 @@ virNetClientPtr virNetClientNewLibSSH2(const char *host,
>           goto cleanup;
>   
>    cleanup:
> -    VIR_FREE(command);
> -    VIR_FREE(privkey);
> -    VIR_FREE(knownhosts);
> -    VIR_FREE(homedir);
> -    VIR_FREE(confdir);
> -    VIR_FREE(nc);
>       return ret;
>   

This will leave a now unneeded 'cleanup' label:

  cleanup:
     return ret;


In a quick glance at the patch series I noticed that Patch 41 also leaves
an unused 'cleanup' label as well after the changes.


Thanks,


DHB

>    no_memory:
> 





More information about the libvir-list mailing list