Fix potential memory leak

Bug-AGL：SPEC-4259

Signed-off-by: duerpei <duep.fnst@fujitsu.com>
Change-Id: I4be57103eee67d64893baccf878d2dbebd1f9cf5
Reviewed-on: https://gerrit.automotivelinux.org/gerrit/c/AGL/meta-agl/+/27219
Reviewed-by: Marius Vlad <marius.vlad@collabora.com>
Reviewed-by: Naoto YAMAGUCHI <naoto.yamaguchi@aisin.co.jp>
Reviewed-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Tested-by: Jenkins Job builder account
ci-image-build: Jenkins Job builder account
ci-image-boot-test: Jenkins Job builder account

diff --git a/meta-agl-core/recipes-graphics/wayland/weston/0001-libweston-Remove-source-repaint_timer-in-weston_comp.patch b/meta-agl-core/recipes-graphics/wayland/weston/0001-libweston-Remove-source-repaint_timer-in-weston_comp.patch
new file mode 100644 (file)
index 0000000..bdcc249
--- /dev/null
+++ b/meta-agl-core/recipes-graphics/wayland/weston/0001-libweston-Remove-source-repaint_timer-in-weston_comp.patch
@@ -0,0 +1,34 @@
+From d4b46cdd3cf0b7e9eaa415fc87a4281695f2349c Mon Sep 17 00:00:00 2001
+From: Lujin Wang <luwang@nvidia.com>
+Date: Mon, 12 Aug 2019 12:03:29 -0700
+Subject: [PATCH] libweston: Remove source repaint_timer in
+ weston_compositor_shutdown
+
+Fixes a definitely lost:
+== 56 bytes in 1 blocks are definitely lost in loss record 16 of 45
+==    at 0x48450F8: malloc (vg_replace_malloc.c:309)
+==    by 0x4B55E93: wl_event_loop_add_timer (event-loop.c:197)
+==    by 0x4126CF: weston_compositor_create (in /usr/local/bin/weston)
+==    by 0x409997: main (in /usr/local/bin/weston)
+
+Signed-off-by: Lujin Wang <luwang@nvidia.com>
+Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.com>
+---
+ libweston/compositor.c | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/libweston/compositor.c b/libweston/compositor.c
+index c2da3a48c..90321e860 100644
+--- a/libweston/compositor.c
++++ b/libweston/compositor.c
+@@ -7733,6 +7733,7 @@ weston_compositor_shutdown(struct weston_compositor *ec)
+       struct weston_output *output, *next;
+ 
+       wl_event_source_remove(ec->idle_source);
++      wl_event_source_remove(ec->repaint_timer);
+ 
+       /* Destroy all outputs associated with this compositor */
+       wl_list_for_each_safe(output, next, &ec->output_list, link)
+-- 
+GitLab
+

diff --git a/meta-agl-core/recipes-graphics/wayland/weston_8.0_aglcore.inc b/meta-agl-core/recipes-graphics/wayland/weston_8.0_aglcore.inc
index 71d2700..f970d5f 100644 (file)
--- a/meta-agl-core/recipes-graphics/wayland/weston_8.0_aglcore.inc
+++ b/meta-agl-core/recipes-graphics/wayland/weston_8.0_aglcore.inc
@@ -6,6 +6,7 @@ SRC_URI:append = "\
     file://0001-clients-window-Add-set_app_id-function-change-Waylan.patch \
     file://0001-libweston-backend-drm-Re-order-gbm-destruction-at-DR.patch \
     file://0001-gl-renderer-Avoid-double-free-on-init-failure.patch \
+    file://0001-libweston-Remove-source-repaint_timer-in-weston_comp.patch \
     "
 
 # Workaround for incorrect upstream definition