From: Peter Krempa <pkrempa@redhat.com>
Date: Thu, 11 Jun 2020 08:24:54 +0000 (+0200)
Subject: cputest: Avoid use of temporary variable in DO_TEST macro
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=473c54e788d2ef517307daad45a1c0c9d0d7fc52;p=libvirt.git

cputest: Avoid use of temporary variable in DO_TEST macro

Use g_free directly to free the returned pointer from
virTestLogContentAndReset rather than store it in a temp variable which
was necessary when we only allowed VIR_FREE.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com
---

diff --git a/tests/cputest.c b/tests/cputest.c
index b66ea7850e..0cf6870574 100644
--- a/tests/cputest.c
+++ b/tests/cputest.c
@@ -997,10 +997,8 @@ mymain(void)
             flags, result \
         }; \
         char *testLabel; \
-        char *tmp; \
  \
-        tmp = virTestLogContentAndReset(); \
-        VIR_FREE(tmp); \
+        g_free(virTestLogContentAndReset());\
  \
         testLabel = g_strdup_printf("%s(%s): %s", #api, \
                                     virArchToString(arch), name); \