[PATCH 5/6] virDomainResctrlNew: Refactor allocation to remove 'cleanup' label

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

 



If we use g_new0 there's no need for the 'cleanup' label as there's
nothing to fail after the allocation.

Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
 src/conf/domain_conf.c | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index f716338efd..0d93ae3341 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -20900,7 +20900,6 @@ virDomainResctrlNew(xmlNodePtr node,
                     unsigned int flags)
 {
     virDomainResctrlDefPtr resctrl = NULL;
-    virDomainResctrlDefPtr ret = NULL;
     g_autofree char *vcpus_str = NULL;
     g_autofree char *alloc_id = NULL;

@@ -20923,18 +20922,13 @@ virDomainResctrlNew(xmlNodePtr node,
     }

     if (virResctrlAllocSetID(alloc, alloc_id) < 0)
-        goto cleanup;
-
-    if (VIR_ALLOC(resctrl) < 0)
-        goto cleanup;
+        return NULL;

+    resctrl = g_new0(virDomainResctrlDef, 1);
     resctrl->vcpus = virBitmapNewCopy(vcpus);
     resctrl->alloc = virObjectRef(alloc);

-    ret = g_steal_pointer(&resctrl);
- cleanup:
-    virDomainResctrlDefFree(resctrl);
-    return ret;
+    return g_steal_pointer(&resctrl);
 }


-- 
2.26.2




[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