[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