On Thu, 26 Apr 2018 11:08:10 +0200 Fabien Lahoudere <[email protected]> wrote:
> From: Semi Malinen <[email protected]> > > Instead of desktop shell assigning view outputs directly, > use a new method, weston_view_set_output(). The method can > set up an output destroy listener to make sure that views > do not have stale output pointers. > > Without this patch it is possible to end up in a scenario > where, e.g. configure_static_view() accesses memory that > has already been freed. The scenario can be provoked by > repeatedly plugging and unplugging a display. The faulty > memory accesses are reported by valgrind. > > Signed-off-by: Semi Malinen <[email protected]> > Signed-off-by: Fabien Lahoudere <[email protected]> > --- > desktop-shell/shell.c | 11 +++++++---- > libweston/compositor.c | 43 +++++++++++++++++++++++++++++++++++++++++-- > libweston/compositor.h | 4 ++++ > 3 files changed, 52 insertions(+), 6 deletions(-) Pushed with my R-b: 03dc95a1..e7a52fbb master -> master Thanks, pq
pgpcVmnRopGe3.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
