mainnika/weston
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

tests: fix all leaks in touch-test

Browse Source 
Fixes all ASan reported leaks.

The manual touch release is slightly awkward as we need to open-code a
part of input_destroy() to avoid double-freeing pointer->wl_touch.

Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.com>
This commit is contained in:
Pekka Paalanen
2021-06-18 13:09:55 +03:00
parent 56a8d69b0e
commit a01299a5b8
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/touch-test.c
+8
View File
@@ -89,6 +89,8 @@ TEST(touch_events)
assert(timespec_eq(&touch->up_time_timespec, &t3)); assert(timespec_eq(&touch->up_time_timespec, &t3));
input_timestamps_destroy(input_ts); input_timestamps_destroy(input_ts);
client_destroy(client);
} }
TEST(touch_timestamps_stop_after_input_timestamps_object_is_destroyed) TEST(touch_timestamps_stop_after_input_timestamps_object_is_destroyed)
@@ -107,6 +109,8 @@ TEST(touch_timestamps_stop_after_input_timestamps_object_is_destroyed)
send_touch(client, &t2, WL_TOUCH_UP); send_touch(client, &t2, WL_TOUCH_UP);
assert(touch->up_time_msec == timespec_to_msec(&t2)); assert(touch->up_time_msec == timespec_to_msec(&t2));
assert(timespec_is_zero(&touch->up_time_timespec)); assert(timespec_is_zero(&touch->up_time_timespec));
client_destroy(client);
} }
TEST(touch_timestamps_stop_after_client_releases_wl_touch) TEST(touch_timestamps_stop_after_client_releases_wl_touch)
@@ -132,4 +136,8 @@ TEST(touch_timestamps_stop_after_client_releases_wl_touch)
assert(timespec_eq(&touch->input_timestamp, &t_other)); assert(timespec_eq(&touch->input_timestamp, &t_other));
input_timestamps_destroy(input_ts); input_timestamps_destroy(input_ts);
free(client->input->touch);
client->input->touch = NULL;
client_destroy(client);
} }