[PATCH weston 18/21] compositor-fbdev: fix finish_frame_timer leak

Pekka Paalanen ppaalanen at gmail.com
Fri Sep 22 14:31:26 UTC 2017


From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>

The timer was never removed anywhere. Remove it in disable() to match
what happens in enable().

Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
---
 libweston/compositor-fbdev.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/libweston/compositor-fbdev.c b/libweston/compositor-fbdev.c
index 7e0e7eef..c981c529 100644
--- a/libweston/compositor-fbdev.c
+++ b/libweston/compositor-fbdev.c
@@ -486,6 +486,9 @@ fbdev_output_disable(struct weston_output *base)
 	if (!base->enabled)
 		return 0;
 
+	wl_event_source_remove(output->finish_frame_timer);
+	output->finish_frame_timer = NULL;
+
 	pixman_renderer_output_destroy(&output->base);
 	fbdev_frame_buffer_unmap(output);
 
-- 
2.13.5



More information about the wayland-devel mailing list