[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