From: Ian Romanick Date: Tue, 29 Nov 2011 02:31:21 +0000 (-0800) Subject: glx: Explicitly reject servers that only support GLX 1.0 X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e4be406f09fcb3756cd0e8dd0539f62bafb1bcca;p=mesa.git glx: Explicitly reject servers that only support GLX 1.0 __glXInitialize calls AllocAndFetchScreenConfigs. AllocAndFetchScreenConfigs unconditionally sends a glXQuerySeverString request to the server. This request is only supported with GLX 1.1 or later, so we were already implicitly incompatible with GLX 1.0 servers. How many more similar bugs lurk in the code that nobody has noticed in years? Signed-off-by: Ian Romanick Reviewed-by: Adam Jackson Reviewed-by: Eric Anholt --- diff --git a/src/glx/glxext.c b/src/glx/glxext.c index 8254544d1c0..baa2489fd19 100644 --- a/src/glx/glxext.c +++ b/src/glx/glxext.c @@ -836,9 +836,12 @@ __glXInitialize(Display * dpy) dpyPriv->serverGLXvendor = 0x0; dpyPriv->serverGLXversion = 0x0; - /* See if the versions are compatible */ + /* See if the versions are compatible. This GLX implementation does not + * work with servers that only support GLX 1.0. + */ if (!QueryVersion(dpy, dpyPriv->majorOpcode, - &dpyPriv->majorVersion, &dpyPriv->minorVersion)) { + &dpyPriv->majorVersion, &dpyPriv->minorVersion) + || (dpyPriv->majorVersion == 1 && dpyPriv->minorVersion < 1)) { Xfree(dpyPriv); _XUnlockMutex(_Xglobal_lock); return NULL; @@ -884,8 +887,7 @@ __glXInitialize(Display * dpy) return NULL; } - if (dpyPriv->majorVersion == 1 && dpyPriv->minorVersion >= 1) - __glXClientInfo(dpy, dpyPriv->majorOpcode); + __glXClientInfo(dpy, dpyPriv->majorOpcode); /* Grab the lock again and add the dispay private, unless somebody * beat us to initializing on this display in the meantime. */