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

Pekka Paalanen ppaalanen at gmail.com
Fri Aug 10 10:32:29 UTC 2018


On Wed, 8 Aug 2018 12:59:45 +0300
Pekka Paalanen <ppaalanen at gmail.com> wrote:

> On Tue, 7 Aug 2018 19:05:01 +0530
> Harsha M M <harsha.manjulamallikarjun at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20180810/259f51d7/attachment.sig>


More information about the wayland-devel mailing list