[PATCH weston] xwm: remove the create surface listener in weston_wm_destroy

Bryce Harrington bryce at osg.samsung.com
Fri Feb 6 17:32:01 PST 2015


On Fri, Feb 06, 2015 at 04:34:10PM +0200, Giulio Camuffo wrote:
> 2015-02-03 19:05 GMT+02:00 Derek Foreman <derekf at osg.samsung.com>:
> > Failing to remove this can result in a crash when the signal is sent
> > after the window manager is destroyed.
> 
> Simple enough,
> 
> Reviewed-By: Giulio Camuffo <giuliocamuffo at gmail.com>

Thanks, applied:
   82cc25b..f10e06c  master -> master
 
> >
> > Signed-off-by: Derek Foreman <derekf at osg.samsung.com>
> > ---
> >  xwayland/window-manager.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/xwayland/window-manager.c b/xwayland/window-manager.c
> > index 07c2ef3..ec3dab9 100644
> > --- a/xwayland/window-manager.c
> > +++ b/xwayland/window-manager.c
> > @@ -2229,6 +2229,7 @@ weston_wm_destroy(struct weston_wm *wm)
> >         wl_list_remove(&wm->activate_listener.link);
> >         wl_list_remove(&wm->kill_listener.link);
> >         wl_list_remove(&wm->transform_listener.link);
> > +       wl_list_remove(&wm->create_surface_listener.link);
> >
> >         free(wm);
> >  }
> > --
> > 2.1.4
> >
> > _______________________________________________
> > wayland-devel mailing list
> > wayland-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list