From fda3696ecf4d17ddcea260ca184c3cc28e085fbc Mon Sep 17 00:00:00 2001 From: Pekka Paalanen Date: Mon, 14 Jun 2021 14:38:04 +0300 Subject: [PATCH] tests: fix leaks in ivi-layout-test-client Everything here was systematically leaking client and iviapp. Discovered by ASan on ./tests/test-ivi-layout-client Signed-off-by: Pekka Paalanen --- tests/ivi-layout-test-client.c | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/tests/ivi-layout-test-client.c b/tests/ivi-layout-test-client.c index b7edf8e8..9ac9a4dd 100644 --- a/tests/ivi-layout-test-client.c +++ b/tests/ivi-layout-test-client.c @@ -241,6 +241,8 @@ TEST_P(ivi_layout_runner, basic_test_names) ivi_window_destroy(wnd); runner_destroy(runner); + ivi_application_destroy(iviapp); + client_destroy(client); } TEST(ivi_layout_surface_create) @@ -267,6 +269,8 @@ TEST(ivi_layout_surface_create) ivi_window_destroy(winds[1]); runner_destroy(runner); + ivi_application_destroy(iviapp); + client_destroy(client); } TEST_P(commit_changes_after_properties_set_surface_destroy, surface_property_commit_changes_test_names) @@ -291,6 +295,8 @@ TEST_P(commit_changes_after_properties_set_surface_destroy, surface_property_com runner_run(runner, "ivi_layout_commit_changes"); runner_destroy(runner); + ivi_application_destroy(iviapp); + client_destroy(client); } TEST(get_surface_after_destroy_ivi_surface) @@ -313,6 +319,8 @@ TEST(get_surface_after_destroy_ivi_surface) wl_surface_destroy(wnd->wl_surface); free(wnd); runner_destroy(runner); + ivi_application_destroy(iviapp); + client_destroy(client); } TEST(get_surface_after_destroy_wl_surface) @@ -335,6 +343,8 @@ TEST(get_surface_after_destroy_wl_surface) ivi_surface_destroy(wnd->ivi_surface); free(wnd); runner_destroy(runner); + ivi_application_destroy(iviapp); + client_destroy(client); } TEST_P(ivi_layout_layer_render_order_runner, render_order_test_names) @@ -363,6 +373,8 @@ TEST_P(ivi_layout_layer_render_order_runner, render_order_test_names) ivi_window_destroy(winds[1]); ivi_window_destroy(winds[2]); runner_destroy(runner); + ivi_application_destroy(iviapp); + client_destroy(client); } TEST(destroy_surface_after_layer_render_order) @@ -392,6 +404,8 @@ TEST(destroy_surface_after_layer_render_order) ivi_window_destroy(winds[0]); ivi_window_destroy(winds[2]); runner_destroy(runner); + ivi_application_destroy(iviapp); + client_destroy(client); } TEST(commit_changes_after_render_order_set_surface_destroy) @@ -422,6 +436,8 @@ TEST(commit_changes_after_render_order_set_surface_destroy) ivi_window_destroy(winds[0]); ivi_window_destroy(winds[2]); runner_destroy(runner); + ivi_application_destroy(iviapp); + client_destroy(client); } TEST(ivi_layout_surface_configure_notification) @@ -457,6 +473,8 @@ TEST(ivi_layout_surface_configure_notification) buffer_destroy(buffer); ivi_window_destroy(wind); runner_destroy(runner); + ivi_application_destroy(iviapp); + client_destroy(client); } TEST(ivi_layout_surface_create_notification) @@ -482,6 +500,8 @@ TEST(ivi_layout_surface_create_notification) ivi_window_destroy(wind); runner_destroy(runner); + ivi_application_destroy(iviapp); + client_destroy(client); } TEST(ivi_layout_surface_remove_notification) @@ -506,4 +526,6 @@ TEST(ivi_layout_surface_remove_notification) runner_run(runner, "surface_remove_notification_p3"); runner_destroy(runner); + ivi_application_destroy(iviapp); + client_destroy(client); }