virNWFilterBindingDefCopy: Use automatic memory management

Signed-off-by: Tim Wiederhake <twiederh@redhat.com>
Reviewed-by: Peter Krempa <pkrempa@redhat.com>

diff --git a/src/conf/virnwfilterbindingdef.c b/src/conf/virnwfilterbindingdef.c
index 9704e1bebb56ee259f8a8bb104e25da46cb958c0..4ed3763efd2fc1f34a66d96f7086ac06cc15e341 100644 (file)
--- a/src/conf/virnwfilterbindingdef.c
+++ b/src/conf/virnwfilterbindingdef.c
@@ -49,9 +49,7 @@ virNWFilterBindingDefFree(virNWFilterBindingDef *def)
 virNWFilterBindingDef *
 virNWFilterBindingDefCopy(virNWFilterBindingDef *src)
 {
-    virNWFilterBindingDef *ret;
-
-    ret = g_new0(virNWFilterBindingDef, 1);
+    g_autoptr(virNWFilterBindingDef) ret = g_new0(virNWFilterBindingDef, 1);
 
     ret->ownername = g_strdup(src->ownername);
 
@@ -70,10 +68,9 @@ virNWFilterBindingDefCopy(virNWFilterBindingDef *src)
     if (virNWFilterHashTablePutAll(src->filterparams, ret->filterparams) < 0)
         goto error;
 
-    return ret;
+    return g_steal_pointer(&ret);
 
  error:
-    virNWFilterBindingDefFree(ret);
     return NULL;
 }