compositor-fbdev: fix finish_frame_timer leak
authorPekka Paalanen <pekka.paalanen@collabora.co.uk>
Thu, 14 Sep 2017 08:43:29 +0000 (11:43 +0300)
committerPekka Paalanen <pekka.paalanen@collabora.co.uk>
Wed, 4 Oct 2017 13:18:06 +0000 (16:18 +0300)
The timer was never removed anywhere. Remove it in disable() to match
what happens in enable().

Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Ian Ray <ian.ray@ge.com>
Acked-by Daniel Stone <daniels@collabora.com>

libweston/compositor-fbdev.c

index f5aa44b..4b3605c 100644 (file)
@@ -485,6 +485,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);