On Wed, 8 Aug 2018 12:59:45 +0300 Pekka Paalanen <ppaala...@gmail.com> wrote:
> On Tue, 7 Aug 2018 19:05:01 +0530 > Harsha M M <harsha.manjulamallikar...@in.bosch.com> wrote: > > > 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(+) > > > > Hi, > > I made a MR from patches 1 and 2 as I think those are fine: > https://gitlab.freedesktop.org/wayland/weston/merge_requests/3 After I fixed my own fork to enable CI, force-pushed to trigger it, and it passed, I merged the MR upstream. > Patch 3 needs more work. Thanks, pq
pgpuV31hEaAed.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel