From 9323af2e81be7c7697db024bdd5680a8d47c17e4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= Date: Tue, 21 Jun 2022 12:34:20 +0400 Subject: [PATCH] tests: fix test-cutils leaks MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Reported by ASAN. Fixes commit cfb34489 ("cutils: add functions for IEC and SI prefixes"). Signed-off-by: Marc-André Lureau Reviewed-by: Peter Maydell Message-Id: <20220621083420.66365-1-marcandre.lureau@redhat.com> Signed-off-by: Thomas Huth --- tests/unit/test-cutils.c | 42 ++++++++++++++++++++++++++++++++-------- 1 file changed, 34 insertions(+), 8 deletions(-) diff --git a/tests/unit/test-cutils.c b/tests/unit/test-cutils.c index f5b780f012..86caddcf64 100644 --- a/tests/unit/test-cutils.c +++ b/tests/unit/test-cutils.c @@ -2452,18 +2452,44 @@ static void test_qemu_strtosz_metric(void) static void test_freq_to_str(void) { - g_assert_cmpstr(freq_to_str(999), ==, "999 Hz"); - g_assert_cmpstr(freq_to_str(1000), ==, "1 KHz"); - g_assert_cmpstr(freq_to_str(1010), ==, "1.01 KHz"); + char *str; + + str = freq_to_str(999); + g_assert_cmpstr(str, ==, "999 Hz"); + g_free(str); + + str = freq_to_str(1000); + g_assert_cmpstr(str, ==, "1 KHz"); + g_free(str); + + str = freq_to_str(1010); + g_assert_cmpstr(str, ==, "1.01 KHz"); + g_free(str); } static void test_size_to_str(void) { - g_assert_cmpstr(size_to_str(0), ==, "0 B"); - g_assert_cmpstr(size_to_str(1), ==, "1 B"); - g_assert_cmpstr(size_to_str(1016), ==, "0.992 KiB"); - g_assert_cmpstr(size_to_str(1024), ==, "1 KiB"); - g_assert_cmpstr(size_to_str(512ull << 20), ==, "512 MiB"); + char *str; + + str = size_to_str(0); + g_assert_cmpstr(str, ==, "0 B"); + g_free(str); + + str = size_to_str(1); + g_assert_cmpstr(str, ==, "1 B"); + g_free(str); + + str = size_to_str(1016); + g_assert_cmpstr(str, ==, "0.992 KiB"); + g_free(str); + + str = size_to_str(1024); + g_assert_cmpstr(str, ==, "1 KiB"); + g_free(str); + + str = size_to_str(512ull << 20); + g_assert_cmpstr(str, ==, "512 MiB"); + g_free(str); } static void test_iec_binary_prefix(void)