aboutsummaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorChen Qun <kuhn.chenqun@huawei.com>2020-11-18 19:56:45 +0800
committerThomas Huth <thuth@redhat.com>2020-11-20 13:34:22 +0100
commitf369797617297e190e2fba3d1b7ba459580f619c (patch)
tree761f331a0c42c964bf5f961b4f5e79a52ecc0e94 /tests
parent923e931188dcbb72387961d313478b80ffc75231 (diff)
downloadqemu-f369797617297e190e2fba3d1b7ba459580f619c.zip
qemu-f369797617297e190e2fba3d1b7ba459580f619c.tar.gz
qemu-f369797617297e190e2fba3d1b7ba459580f619c.tar.bz2
tests/qtest: variable defined by g_autofree need to be initialized
According to the glib function requirements, we need initialise the variable. Otherwise there will be compilation warnings: glib-autocleanups.h:28:3: warning: ‘full_name’ may be used uninitialized in this function [-Wmaybe-uninitialized] 28 | g_free (*pp); | ^~~~~~~~~~~~ Reported-by: Euler Robot <euler.robot@huawei.com> Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com> Reviewed-by: Thomas Huth <thuth@redhat.com> Reviewed-by: Havard Skinnemoen <hskinnemoen@google.com> Message-Id: <20201118115646.2461726-2-kuhn.chenqun@huawei.com> Signed-off-by: Thomas Huth <thuth@redhat.com>
Diffstat (limited to 'tests')
-rw-r--r--tests/qtest/npcm7xx_timer-test.c8
1 files changed, 3 insertions, 5 deletions
diff --git a/tests/qtest/npcm7xx_timer-test.c b/tests/qtest/npcm7xx_timer-test.c
index f08b0cd..83774a5 100644
--- a/tests/qtest/npcm7xx_timer-test.c
+++ b/tests/qtest/npcm7xx_timer-test.c
@@ -512,11 +512,9 @@ static void test_disable_on_expiration(gconstpointer test_data)
*/
static void tim_add_test(const char *name, const TestData *td, GTestDataFunc fn)
{
- g_autofree char *full_name;
-
- full_name = g_strdup_printf("npcm7xx_timer/tim[%d]/timer[%d]/%s",
- tim_index(td->tim), timer_index(td->timer),
- name);
+ g_autofree char *full_name = g_strdup_printf(
+ "npcm7xx_timer/tim[%d]/timer[%d]/%s", tim_index(td->tim),
+ timer_index(td->timer), name);
qtest_add_data_func(full_name, td, fn);
}