libweston: fix output destroy signal list corruption

This issue was likely introduced by "libweston: add
weston_view_set_output()" which forgot to ensure the output destroy
listener is removed when weston_view is destroyed, leading to freed
memory being left into the list.

This was quite easy to trigger by opening and closing an application
window a few times, leading various memory corruption symptoms.

Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Ian Ray <ian.ray@ge.com>
dev
Pekka Paalanen 6 years ago
parent 99628000cb
commit 944fae8887
  1. 1
      libweston/compositor.c

@ -1931,6 +1931,7 @@ weston_view_destroy(struct weston_view *view)
pixman_region32_fini(&view->transform.opaque); pixman_region32_fini(&view->transform.opaque);
weston_view_set_transform_parent(view, NULL); weston_view_set_transform_parent(view, NULL);
weston_view_set_output(view, NULL);
wl_list_remove(&view->surface_link); wl_list_remove(&view->surface_link);

Loading…
Cancel
Save