[PATCH xwayland] xwayland: Don't load extension list more then once
Peter Hutterer
peter.hutterer at who-t.net
Wed May 31 11:43:19 UTC 2017
On Wed, May 31, 2017 at 01:31:24PM +0900, Michel Dänzer wrote:
> On 31/05/17 05:39 AM, Lyude wrote:
> > When running an Xwayland server from the command line, we end up
> > resetting the server every time all of the clients connected to the
> > server leave. This would be fine, except that xwayland makes the mistake
> > of unconditionally calling LoadExtensionList(). This causes us to setup
> > the glxExtension twice in a row which means that when we lose our last
> > client on the second server generation, we end up trying to call the glx
> > destructors twice in a row resulting in a segfault:
> >
[...]
>
> Good catch.
>
> Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>
remote: I: patch #158991 updated using rev 4f29366f1e5678505fb882143c9b4a892d5b8273.
remote: I: 1 patch(es) updated to state Accepted.
To git+ssh://git.freedesktop.org/git/xorg/xserver
2fbf62b2f..4f29366f1 master -> master
(also fixed a typo in the commit message subject before pushing)
Cheers,
Peter
More information about the xorg-devel
mailing list