tests: bhyve: use g_autofree where possible

Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Laine Stump <laine@redhat.com>
Reviewed-by: Roman Bogorodskiy <bogorodskiy@gmail.com>

diff --git a/tests/bhyveargv2xmltest.c b/tests/bhyveargv2xmltest.c
index 735cc4b338f5d6c9bcecf4e0f223963ed182f361..e58a36d17c73647fba768d13f23fb7f84a1f14f8 100644 (file)
--- a/tests/bhyveargv2xmltest.c
+++ b/tests/bhyveargv2xmltest.c
@@ -28,9 +28,9 @@ testCompareXMLToArgvFiles(const char *xmlfile,
                           unsigned int flags)
 
 {
-    char *actualxml = NULL;
-    char *cmd = NULL;
-    char *log = NULL;
+    g_autofree char *actualxml = NULL;
+    g_autofree char *cmd = NULL;
+    g_autofree char *log = NULL;
     int ret = -1;
     virDomainDefPtr vmdef = NULL;
 
@@ -86,9 +86,6 @@ testCompareXMLToArgvFiles(const char *xmlfile,
     ret = 0;
 
  fail:
-    VIR_FREE(actualxml);
-    VIR_FREE(cmd);
-    VIR_FREE(log);
     virDomainDefFree(vmdef);
     return ret;
 }
@@ -103,8 +100,8 @@ testCompareXMLToArgvHelper(const void *data)
 {
     int result = -1;
     const struct testInfo *info = data;
-    char *xml = NULL;
-    char *args = NULL;
+    g_autofree char *xml = NULL;
+    g_autofree char *args = NULL;
 
     xml = g_strdup_printf("%s/bhyveargv2xmldata/bhyveargv2xml-%s.xml",
                           abs_srcdir, info->name);
@@ -112,9 +109,6 @@ testCompareXMLToArgvHelper(const void *data)
                            abs_srcdir, info->name);
 
     result = testCompareXMLToArgvFiles(xml, args, info->flags);
-
-    VIR_FREE(xml);
-    VIR_FREE(args);
     return result;
 }
 

diff --git a/tests/bhyvexml2argvtest.c b/tests/bhyvexml2argvtest.c
index f7ae1db26d75d19fefcea356a36661122da5d870..ce2af08d4ccc67bf5c33431f4474369f2fff6910 100644 (file)
--- a/tests/bhyvexml2argvtest.c
+++ b/tests/bhyvexml2argvtest.c
@@ -26,9 +26,9 @@ static int testCompareXMLToArgvFiles(const char *xml,
                                      const char *dmcmdline,
                                      unsigned int flags)
 {
-    char *actualargv = NULL;
-    char *actualld = NULL;
-    char *actualdm = NULL;
+    g_autofree char *actualargv = NULL;
+    g_autofree char *actualld = NULL;
+    g_autofree char *actualdm = NULL;
     virDomainDefPtr vmdef = NULL;
     virCommandPtr cmd = NULL;
     virCommandPtr ldcmd = NULL;
@@ -99,9 +99,6 @@ static int testCompareXMLToArgvFiles(const char *xml,
         vmdef->graphics[0]->type == VIR_DOMAIN_GRAPHICS_TYPE_VNC)
         virPortAllocatorRelease(vmdef->graphics[0]->data.vnc.port);
 
-    VIR_FREE(actualargv);
-    VIR_FREE(actualld);
-    VIR_FREE(actualdm);
     virCommandFree(cmd);
     virCommandFree(ldcmd);
     virDomainDefFree(vmdef);
@@ -119,10 +116,10 @@ testCompareXMLToArgvHelper(const void *data)
 {
     int ret = -1;
     const struct testInfo *info = data;
-    char *xml = NULL;
-    char *args = NULL;
-    char *ldargs = NULL;
-    char *dmargs = NULL;
+    g_autofree char *xml = NULL;
+    g_autofree char *args = NULL;
+    g_autofree char *ldargs = NULL;
+    g_autofree char *dmargs = NULL;
 
     xml = g_strdup_printf("%s/bhyvexml2argvdata/bhyvexml2argv-%s.xml",
                           abs_srcdir, info->name);
@@ -135,10 +132,6 @@ testCompareXMLToArgvHelper(const void *data)
 
     ret = testCompareXMLToArgvFiles(xml, args, ldargs, dmargs, info->flags);
 
-    VIR_FREE(xml);
-    VIR_FREE(args);
-    VIR_FREE(ldargs);
-    VIR_FREE(dmargs);
     return ret;
 }
 

diff --git a/tests/bhyvexml2xmltest.c b/tests/bhyvexml2xmltest.c
index a0c20a14c1c836bdb57d6101cc02594d4ddcb47c..2ac102b8e3a42a47fc4500bb921682bb5d3d1f8d 100644 (file)
--- a/tests/bhyvexml2xmltest.c
+++ b/tests/bhyvexml2xmltest.c
@@ -26,8 +26,8 @@ static int
 testCompareXMLToXMLHelper(const void *data)
 {
     const struct testInfo *info = data;
-    char *xml_in = NULL;
-    char *xml_out = NULL;
+    g_autofree char *xml_in = NULL;
+    g_autofree char *xml_out = NULL;
     bool is_different = info->flags & FLAG_IS_DIFFERENT;
     int ret = -1;
 
@@ -48,8 +48,6 @@ testCompareXMLToXMLHelper(const void *data)
         virResetLastError();
     }
 
-    VIR_FREE(xml_in);
-    VIR_FREE(xml_out);
     return ret;
 }