fdo-mirrors/weston
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/wayland/weston.git synced 2026-02-05 08:50:34 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

weston-test: Fix another mem leak on shutdown path

Browse source 
Fixes the following mem leak:

=================================================================
==191==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 56 byte(s) in 1 object(s) allocated from:
    #0 0x7f6b843f6610 in calloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:77
    #1 0x7f6b8302f499 in output_created_listener ../tests/weston-test.c:145
    #2 0x7f6b83032dcd in wet_module_init ../tests/weston-test.c:857
    #3 0x7f6b842b8425 in wet_load_module ../frontend/main.c:989
    #4 0x7f6b842b89eb in load_modules ../frontend/main.c:1069
    #5 0x7f6b842d2711 in wet_main ../frontend/main.c:4865
    #6 0x562862e934d6 in execute_compositor ../tests/weston-test-fixture-compositor.c:431
    #7 0x562862e9756b in weston_test_harness_execute_as_client ../tests/weston-test-runner.c:570
    #8 0x562862e81e1d in fixture_setup ../tests/drm-writeback-screenshot-test.c:48
    #9 0x562862e81e9e in fixture_setup_run_ ../tests/drm-writeback-screenshot-test.c:50
    #10 0x562862e97bb6 in main ../tests/weston-test-runner.c:726
    #11 0x7f6b83d33ca7 in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
This commit is contained in:
Marius Vlad 2025-07-04 15:33:41 +03:00 • committed by Daniel Stone
parent 02a6e45ce5
commit 41bef3caaa
1 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
tests/weston-test.c
View file

@ -836,6 +836,7 @@ wet_module_init(struct weston_compositor *ec,
struct weston_test *test;
struct weston_output *output;
struct wl_event_loop *loop;
struct weston_test_output *to, *tmp;
test = zalloc(sizeof *test);
if (test == NULL)
@ -881,6 +882,12 @@ out_free:
if (test->log)
weston_log_scope_destroy(test->log);
wl_list_for_each_safe(to, tmp, &test->output_list, link) {
wl_list_remove(&to->repaint_listener.link);
wl_list_remove(&to->link);
free(to);
}
wl_list_remove(&test->output_destroyed_listener.link);
wl_list_remove(&test->destroy_listener.link);
free(test);