compositor-wayland: destroy the appropriate output instead of exiting when receiving an xdg_toplevel::close event
v2: Fix use after free spotted by Daniel Stone
Signed-off-by: Sergi Granell <xerpi.g.12@gmail.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
diff --git a/libweston/compositor-wayland.c b/libweston/compositor-wayland.c
index 00cc4d4..e2f416a 100644
--- a/libweston/compositor-wayland.c
+++ b/libweston/compositor-wayland.c
@@ -1098,8 +1098,12 @@
handle_xdg_toplevel_close(void *data, struct zxdg_toplevel_v6 *xdg_toplevel)
{
struct wayland_output *output = data;
+ struct weston_compositor *compositor = output->base.compositor;
- weston_compositor_exit(output->base.compositor);
+ wayland_output_destroy(&output->base);
+
+ if (wl_list_empty(&compositor->output_list))
+ weston_compositor_exit(compositor);
}
static const struct zxdg_toplevel_v6_listener xdg_toplevel_listener = {