[PATCH weston] xwayland: Fix crash on weston shutdown

Derek Foreman derekf at osg.samsung.com
Wed Feb 7 16:58:45 UTC 2018


On 2018-02-05 04:35 PM, Daniel Stone wrote:
> Hi,
> 
> On 5 February 2018 at 22:02, Derek Foreman <derekf at osg.samsung.com> wrote:
>> commit e7fff215ada3fd3d1b2af664888f960c082f9065 made initializing the
>> selection_listener conditional, but didn't make its clean-up
>> conditional at shutdown.
>>
>> To see this, run weston -Bheadless-backend.so and then connect to it
>> with an X client.  When killing weston it will attempt shutdown but
>> die with a segfault.
> 
> For bonus points, it seems like it should really hang off the seat's
> destroy_signal too ...

Ah, you're probably right.  I might get back to that later.

> wm_list_init() somewhere in init would be more in keeping with the
> rest of the codebase; with that, this is:
> Reviewed-by: Daniel Stone <daniels at collabora.com>

I've switched up to a wm_list_init() and landed this.

Thanks,
Derek

> Cheers,
> Daniel
> 



More information about the wayland-devel mailing list