Mesa (master): glx: Fix glXGetFBConfigFromVisualSGIX.

Brian Paul brianp at kemper.freedesktop.org
Fri May 27 17:59:07 UTC 2011


Module: Mesa
Branch: master
Commit: fc48de4fabcf3a42a1cd7d447790afefc7d27af8
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=fc48de4fabcf3a42a1cd7d447790afefc7d27af8

Author: Stéphane Marchesin <marcheu at chromium.org>
Date:   Thu May 26 17:19:03 2011 -0700

glx: Fix glXGetFBConfigFromVisualSGIX.

We want to check for Success, otherwise it will fail even with the right visual.

NOTE: This is a candidate for the 7.10 branch.

Signed-off-by: Antoine Labour <piman at chromium.org>
Signed-off-by: Stéphane Marchesin <marcheu at chromium.org>
Signed-off-by: Brian Paul <brianp at vmware.com>

---

 src/glx/glxcmds.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/glx/glxcmds.c b/src/glx/glxcmds.c
index 012cd79..dbfa0ed 100644
--- a/src/glx/glxcmds.c
+++ b/src/glx/glxcmds.c
@@ -1890,7 +1890,7 @@ glXGetFBConfigFromVisualSGIX(Display * dpy, XVisualInfo * vis)
    struct glx_display *priv;
    struct glx_screen *psc = NULL;
 
-   if ((GetGLXPrivScreenConfig(dpy, vis->screen, &priv, &psc) != Success)
+   if ((GetGLXPrivScreenConfig(dpy, vis->screen, &priv, &psc) == Success)
        && __glXExtensionBitIsEnabled(psc, SGIX_fbconfig_bit)
        && (psc->configs->fbconfigID != (int) GLX_DONT_CARE)) {
       return (GLXFBConfigSGIX) glx_config_find_visual(psc->configs,




More information about the mesa-commit mailing list