From: Jiang Jiacheng <jiangjiacheng@huawei.com>
Date: Fri, 6 Jan 2023 09:18:39 +0000 (+0800)
Subject: vz: use g_autofree and remove unnecessary label
X-Git-Tag: v9.0.0-rc1~66
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=ffd286ac6f25374d16f4eaa7ff64e30c77541b41;p=thirdparty%2Flibvirt.git

vz: use g_autofree and remove unnecessary label

Signed-off-by: Jiang Jiacheng <jiangjiacheng@huawei.com>
Reviewed-by: Jonathon Jongsma <jjongsma@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
Signed-off-by: Ján Tomko <jtomko@redhat.com>
---

diff --git a/src/vz/vz_utils.c b/src/vz/vz_utils.c
index fcf6d363a9..219d13823b 100644
--- a/src/vz/vz_utils.c
+++ b/src/vz/vz_utils.c
@@ -151,7 +151,8 @@ vzInitCaps(unsigned long vzVersion, struct _vzCapabilities *vzCaps)
 int
 vzInitVersion(struct _vzDriver *driver)
 {
-    char *output, *sVer, *tmp;
+    g_autofree char *output = NULL;
+    char *sVer, *tmp;
     const char *searchStr = "prlsrvctl version ";
     int ret = -1;
 
@@ -159,12 +160,12 @@ vzInitVersion(struct _vzDriver *driver)
 
     if (!output) {
         vzParseError();
-        goto cleanup;
+        return -1;
     }
 
     if (!(sVer = strstr(output, searchStr))) {
         vzParseError();
-        goto cleanup;
+        return -1;
     }
 
     sVer = sVer + strlen(searchStr);
@@ -173,26 +174,23 @@ vzInitVersion(struct _vzDriver *driver)
      * In libvirt we handle only first two numbers. */
     if (!(tmp = strchr(sVer, '.'))) {
         vzParseError();
-        goto cleanup;
+        return -1;
     }
 
     if (!(tmp = strchr(tmp + 1, '.'))) {
         vzParseError();
-        goto cleanup;
+        return -1;
     }
 
     tmp[0] = '\0';
     if (virStringParseVersion(&(driver->vzVersion), sVer, true) < 0) {
         vzParseError();
-        goto cleanup;
+        return -1;
     }
 
     vzInitCaps(driver->vzVersion, &driver->vzCaps);
-    ret = 0;
 
- cleanup:
-    VIR_FREE(output);
-    return ret;
+    return 0;
 }
 
 static int