[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 06/12] tests/qtest: Free paths
From: |
Thomas Huth |
Subject: |
[PULL 06/12] tests/qtest: Free paths |
Date: |
Tue, 2 Jul 2024 12:33:04 +0200 |
From: Akihiko Odaki <akihiko.odaki@daynix.com>
This fixes LeakSanitizer warnings.
Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Message-ID: <20240627-san-v2-14-750bb0946dbd@daynix.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
tests/qtest/qos-test.c | 16 ++++++++++++----
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/tests/qtest/qos-test.c b/tests/qtest/qos-test.c
index 5da4091ec3..114f6bef27 100644
--- a/tests/qtest/qos-test.c
+++ b/tests/qtest/qos-test.c
@@ -33,7 +33,6 @@
static char *old_path;
-
/**
* qos_set_machines_devices_available(): sets availability of qgraph
* machines and devices.
@@ -191,6 +190,12 @@ static void subprocess_run_one_test(const void *arg)
g_test_trap_assert_passed();
}
+static void destroy_pathv(void *arg)
+{
+ g_free(((char **)arg)[0]);
+ g_free(arg);
+}
+
/*
* in this function, 2 path will be built:
* path_str, a one-string path (ex "pc/i440FX-pcihost/...")
@@ -295,10 +300,13 @@ static void walk_path(QOSGraphNode *orig_path, int len)
if (path->u.test.subprocess) {
gchar *subprocess_path = g_strdup_printf("/%s/%s/subprocess",
qtest_get_arch(), path_str);
- qtest_add_data_func(path_str, subprocess_path,
subprocess_run_one_test);
- g_test_add_data_func(subprocess_path, path_vec, run_one_test);
+ qtest_add_data_func_full(path_str, subprocess_path,
+ subprocess_run_one_test, g_free);
+ g_test_add_data_func_full(subprocess_path, path_vec,
+ run_one_test, destroy_pathv);
} else {
- qtest_add_data_func(path_str, path_vec, run_one_test);
+ qtest_add_data_func_full(path_str, path_vec,
+ run_one_test, destroy_pathv);
}
g_free(path_str);
--
2.45.2
- [PULL 00/12] qtest, s390x, avocado and doc patches, Thomas Huth, 2024/07/02
- [PULL 04/12] tests/qtest: Free unused QMP response, Thomas Huth, 2024/07/02
- [PULL 03/12] tests/qtest: Use qtest_add_data_func_full(), Thomas Huth, 2024/07/02
- [PULL 05/12] tests/qtest: Free old machine variable name, Thomas Huth, 2024/07/02
- [PULL 06/12] tests/qtest: Free paths,
Thomas Huth <=
- [PULL 07/12] tests/qtest: Free GThread, Thomas Huth, 2024/07/02
- [PULL 08/12] docs: add precision about capstone for execlog plugin, Thomas Huth, 2024/07/02
- [PULL 09/12] hw/s390x: Attach default virtio-net devices to the /machine/virtual-css-bridge, Thomas Huth, 2024/07/02
- [PULL 11/12] .travis.yml: Install python3-tomli in all build jobs, Thomas Huth, 2024/07/02
- [PULL 10/12] tests/avocado: add hotplug_blk test, Thomas Huth, 2024/07/02
- [PULL 12/12] pc-bios/s390-ccw: Remove duplicated LDFLAGS, Thomas Huth, 2024/07/02
- Re: [PULL 00/12] qtest, s390x, avocado and doc patches, Richard Henderson, 2024/07/03