Skip to content

[PATCH weston 0/3] Fix invalid memory access during de-init

Pekka Paalanen requested to merge pq/weston:MR/memfixes-part1 into master

This MR contains two of the three patches from the mailing list submission:

> From: Harsha M M 
> To: wayland-devel@lists.freedesktop.org
> Cc: Harsha M M 
> Subject: [PATCH weston 0/3] Fix invalid memory access during de-init
> Date: Tue, 7 Aug 2018 19:05:01 +0530
> 
> valgrind reports several invalid memory access during shutdown of
> weston. In general problem is with components which do not remove
> the signals added using wl_signal_emit, but free their memory.
> 
> This patch series fixes invalid memory access during shutdown of weston
> 
> Harsha M M (3):
>   libweston: Remove signals from the list during de-init
>   ivi-shell: Remove the compositor destory listener from list during
>     de-init
>   ivi-shell: clean up remaining ivisurface during de-init
> 
>  compositor/text-backend.c         | 3 +++
>  compositor/weston-screenshooter.c | 2 ++
>  desktop-shell/shell.c             | 1 +
>  ivi-shell/ivi-shell.c             | 4 ++++
>  4 files changed, 10 insertions(+)

This is ready to merge and already has my R-b in place (since I picked it up from the mailing list). Further review comments are on the mailing list.

Merge request reports