Module: Mesa Branch: main Commit: c50bdacbda6dc63d4c794e79357ffebf9756aa8a URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=c50bdacbda6dc63d4c794e79357ffebf9756aa8a
Author: Qiang Yu <[email protected]> Date: Wed Dec 8 10:57:45 2021 +0800 glx: fix regression for drawable type detection Newer version of XServer supporting GLX_DRAWABLE_TYPE query also support query with raw X11 window ID besides GLXWindow ID. So we should not limit the suppported type to GLXPbuffer when query success. Otherwise can't start GLX application on newer XServer with: libGL error: GLX drawable type is not supported libGL error: GLX drawable type is not supported X Error of failed request: GLXBadContext Major opcode of failed request: 149 (GLX) Minor opcode of failed request: 5 (X_GLXMakeCurrent) Serial number of failed request: 35 Current serial number in output stream: 35 Fixes: 6625c960c58 ("glx: check drawable type before create drawble") Tested-by: Mike Lothian <[email protected]> Reviewed-by: Adam Jackson <[email protected]> Signed-off-by: Qiang Yu <[email protected]> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14120> --- src/glx/dri_common.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/glx/dri_common.c b/src/glx/dri_common.c index 0be684dda95..27e78fb3799 100644 --- a/src/glx/dri_common.c +++ b/src/glx/dri_common.c @@ -392,15 +392,13 @@ driFetchDrawable(struct glx_context *gc, GLXDrawable glxDrawable) /* Infer the GLX drawable type. */ if (__glXGetDrawableAttribute(dpy, glxDrawable, GLX_DRAWABLE_TYPE, &type)) { - if (type != GLX_PBUFFER_BIT) { - ErrorMessageF("GLX drawable type is not supported\n"); + /* Xserver may support query with raw X11 window. */ + if (type == GLX_PIXMAP_BIT) { + ErrorMessageF("GLXPixmap drawable type is not supported\n"); return NULL; } } else { - /* Xserver may not implement GLX_DRAWABLE_TYPE query yet, or glxDrawable - * is a X window. Assume it's a GLXPbuffer in former case, because we don't - * know GLXPixmap and GLXWindow's X drawable ID anyway. - */ + /* Xserver may not implement GLX_DRAWABLE_TYPE query yet. */ type = GLX_PBUFFER_BIT | GLX_WINDOW_BIT; }
