Re: [PATCH xserver 05/12] glxproxy: Fix __glXActiveScreens allocation

2017-06-20 Thread Adam Jackson
On Fri, 2017-06-16 at 13:26 -0700, Keith Packard wrote: > Adam Jackson writes: > > > Apparently this has been broken for about ten years, eesh. We were never > > allocating any storage for this array, so the first attempt at using GLX > > with Xdmx would crash the server. > > >

Re: [PATCH xserver 05/12] glxproxy: Fix __glXActiveScreens allocation

2017-06-16 Thread Keith Packard
Adam Jackson writes: > Apparently this has been broken for about ten years, eesh. We were never > allocating any storage for this array, so the first attempt at using GLX > with Xdmx would crash the server. > > Promote it to an array and use __glXNumActiveScreens to detect

[PATCH xserver 05/12] glxproxy: Fix __glXActiveScreens allocation

2017-06-16 Thread Adam Jackson
Apparently this has been broken for about ten years, eesh. We were never allocating any storage for this array, so the first attempt at using GLX with Xdmx would crash the server. Promote it to an array and use __glXNumActiveScreens to detect whether GLX is actually enabled. Bugzilla: