[libvirt] [PATCH v1 12/32] util: netdevip: use VIR_AUTOFREE instead of VIR_FREE for scalar types

Erik Skultety eskultet at redhat.com
Fri Aug 3 12:54:37 UTC 2018


On Sat, Jul 28, 2018 at 11:31:27PM +0530, Sukrit Bhatnagar wrote:
> By making use of GNU C's cleanup attribute handled by the
> VIR_AUTOFREE macro for declaring scalar variables, majority
> of the VIR_FREE calls can be dropped, which in turn leads to
> getting rid of most of our cleanup sections.
>
> Signed-off-by: Sukrit Bhatnagar <skrtbhtngr at gmail.com>
> ---
>  src/util/virnetdevip.c | 95 ++++++++++++++++++--------------------------------
>  1 file changed, 33 insertions(+), 62 deletions(-)
>
> diff --git a/src/util/virnetdevip.c b/src/util/virnetdevip.c
> index fdb0b74..8f1081b 100644
> --- a/src/util/virnetdevip.c
> +++ b/src/util/virnetdevip.c
> @@ -171,11 +171,11 @@ virNetDevIPAddrAdd(const char *ifname,
>      virSocketAddr *broadcast = NULL;
>      int ret = -1;
>      struct nl_msg *nlmsg = NULL;

As I pointed out earlier, ^this could be converted to VIR_AUTOPTR. Otherwise
it's fine.

Erik




More information about the libvir-list mailing list