diff options
| author | Brian Paul <brianp@vmware.com> | 2017-05-22 11:46:27 -0600 |
|---|---|---|
| committer | Emil Velikov <emil.l.velikov@gmail.com> | 2017-06-14 10:40:22 +0100 |
| commit | 6348a02e275c44164029070c717aa623120a0ad0 (patch) | |
| tree | fb8403c5012228fe0109af05edb488c43cde8a41 | |
| parent | c24bdf046e4608a1d372e27446d4a7e87b5458c3 (diff) | |
xlib: fix glXGetCurrentDisplay() failure
glXGetCurrentDisplay() has been broken for years and nobody noticed until
recently. This change adds a new XMesaGetCurrentDisplay() that the GLX
emulation API can call, just as we did for glXGetCurrentContext().
Tested by hacking glxgears to call glXGetCurrentContext() before and
after glXMakeCurrent() to verify the return value is NULL beforehand and
the same as the opened display afterward.
Also tested by Tom Hudson with his tests programs.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100988
Cc: mesa-stable@lists.freedesktop.org
Tested-by: Tom Hudson <tom.hudson.phd@gmail.com>
Signed-off-by: Brian Paul <brianp@vmware.com>
(cherry picked from commit c6ba85a8c0f02b3b7058dae7afb6c49f56567319)
| -rw-r--r-- | src/mesa/drivers/x11/glxapi.c | 8 | ||||
| -rw-r--r-- | src/mesa/drivers/x11/glxapi.h | 1 | ||||
| -rw-r--r-- | src/mesa/drivers/x11/xm_api.c | 8 | ||||
| -rw-r--r-- | src/mesa/drivers/x11/xmesa.h | 6 |
4 files changed, 18 insertions, 5 deletions
diff --git a/src/mesa/drivers/x11/glxapi.c b/src/mesa/drivers/x11/glxapi.c index a807074a863..52e60265697 100644 --- a/src/mesa/drivers/x11/glxapi.c +++ b/src/mesa/drivers/x11/glxapi.c @@ -379,13 +379,13 @@ glXQueryServerString(Display *dpy, int screen, int name) /*** GLX_VERSION_1_2 ***/ +/* declare here to avoid including xmesa.h */ +extern Display *XMesaGetCurrentDisplay(void); + Display PUBLIC * glXGetCurrentDisplay(void) { - /* Same code as in libGL's glxext.c */ - __GLXcontext *gc = (__GLXcontext *) glXGetCurrentContext(); - if (NULL == gc) return NULL; - return gc->currentDpy; + return XMesaGetCurrentDisplay(); } diff --git a/src/mesa/drivers/x11/glxapi.h b/src/mesa/drivers/x11/glxapi.h index aff38f7531d..cc4f902925b 100644 --- a/src/mesa/drivers/x11/glxapi.h +++ b/src/mesa/drivers/x11/glxapi.h @@ -37,7 +37,6 @@ * work properly. */ typedef struct __GLXcontextRec { - Display *currentDpy; GLboolean isDirect; GLXDrawable currentDrawable; GLXDrawable currentReadable; diff --git a/src/mesa/drivers/x11/xm_api.c b/src/mesa/drivers/x11/xm_api.c index 5be8aa95de1..155d273236f 100644 --- a/src/mesa/drivers/x11/xm_api.c +++ b/src/mesa/drivers/x11/xm_api.c @@ -1304,6 +1304,14 @@ XMesaBuffer XMesaGetCurrentReadBuffer( void ) } +Display *XMesaGetCurrentDisplay(void) +{ + GET_CURRENT_CONTEXT(ctx); + XMesaContext xmctx = XMESA_CONTEXT(ctx); + return xmctx ? xmctx->display : NULL; +} + + GLboolean XMesaSetFXmode( GLint mode ) { diff --git a/src/mesa/drivers/x11/xmesa.h b/src/mesa/drivers/x11/xmesa.h index cc878e7402e..84b2b27006d 100644 --- a/src/mesa/drivers/x11/xmesa.h +++ b/src/mesa/drivers/x11/xmesa.h @@ -241,6 +241,12 @@ extern XMesaBuffer XMesaGetCurrentReadBuffer( void ); /* + * Return display of current context. + */ +extern Display *XMesaGetCurrentDisplay( void ); + + +/* * Swap the front and back buffers for the given buffer. No action is * taken if the buffer is not double buffered. */ |