mirrors/wlroots
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/wlroots/wlroots.git synced 2025-11-03 09:01:40 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

gtk primary selection: destroy devices before manager

Browse source 
Since device_destroy() calls wl_list_remove() on the device's link,
the manager must still be valid. However if the manager is destroyed
before the seat as the wl_display destroy handlers are called, devices
listening for the seat to be destroyed will access free'd memory in
wl_list_remove().
This commit is contained in:
Isaac Freund 2021-01-05 12:51:31 +01:00 • committed by Simon Ser
parent bf926e31a0
commit 0cba1ce747
1 changed files with 6 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

6
types/wlr_gtk_primary_selection.c
View file

@ -452,6 +452,12 @@ static void primary_selection_device_manager_bind(struct wl_client *client,
static void handle_display_destroy(struct wl_listener *listener, void *data) { static void handle_display_destroy(struct wl_listener *listener, void *data) {
struct wlr_gtk_primary_selection_device_manager *manager = struct wlr_gtk_primary_selection_device_manager *manager =
wl_container_of(listener, manager, display_destroy); wl_container_of(listener, manager, display_destroy);
struct wlr_gtk_primary_selection_device *device, *tmp;
wl_list_for_each_safe(device, tmp, &manager->devices, link) {
device_destroy(device);
}
wlr_signal_emit_safe(&manager->events.destroy, manager); wlr_signal_emit_safe(&manager->events.destroy, manager);
wl_list_remove(&manager->display_destroy.link); wl_list_remove(&manager->display_destroy.link);
wl_global_destroy(manager->global); wl_global_destroy(manager->global);