From 05350e451c1f052bd6223fa9fd3ff9346e34103f Mon Sep 17 00:00:00 2001
From: Peter Krempa <pkrempa@redhat.com>
Date: Tue, 23 Mar 2021 17:42:31 +0100
Subject: [PATCH] virResctrlAllocNewFromInfo: Use g_autoptr for 'ret'
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

Remove 'cleanup' and 'error' labels by switching 'ret' to automatic
pointer and stealing it in the return statement.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
---
 src/util/virresctrl.c | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/src/util/virresctrl.c b/src/util/virresctrl.c
index 5e7c391524..c66cf4b087 100644
--- a/src/util/virresctrl.c
+++ b/src/util/virresctrl.c
@@ -1811,7 +1811,7 @@ static virResctrlAllocPtr
 virResctrlAllocNewFromInfo(virResctrlInfoPtr info)
 {
     size_t i = 0;
-    virResctrlAllocPtr ret = virResctrlAllocNew();
+    g_autoptr(virResctrlAlloc) ret = virResctrlAllocNew();
 
     if (!ret)
         return NULL;
@@ -1836,7 +1836,7 @@ virResctrlAllocNewFromInfo(virResctrlInfoPtr info)
 
             for (k = 0; k <= i_type->max_cache_id; k++) {
                 if (virResctrlAllocUpdateMask(ret, i, j, k, mask) < 0)
-                    goto error;
+                    return NULL;
             }
         }
     }
@@ -1854,12 +1854,7 @@ virResctrlAllocNewFromInfo(virResctrlInfoPtr info)
         }
     }
 
- cleanup:
-    return ret;
- error:
-    virObjectUnref(ret);
-    ret = NULL;
-    goto cleanup;
+    return g_steal_pointer(&ret);
 }
 
 /*
-- 
2.47.2