glx: fix a macro being invoked with the wrong parameter name
authorAdam Stylinski <kungfujesus06@gmail.com>
Sat, 21 Jan 2023 17:51:28 +0000 (12:51 -0500)
committerMarge Bot <emma+marge@anholt.net>
Sun, 22 Jan 2023 04:22:08 +0000 (04:22 +0000)
Strangely, this was somehow compiling with GCC but my futile efforts to
build mesa with msan caused me to find clang refusing to compile because
of this. Unknown how many bugs this could fix or how GCC did manage to
find "config" in scope but it's fairly obvious that this is the correct
parameter that should be used.

Reviewed-by: Adam Jackson <ajax@redhat.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/20830>

src/glx/glxcmds.c

index df960e3..37c968b 100644 (file)
@@ -1771,7 +1771,7 @@ _GLX_PUBLIC GLX_ALIAS(XVisualInfo *, glXGetVisualFromFBConfigSGIX,
 _GLX_PUBLIC GLX_ALIAS(GLXContext, glXCreateContextWithConfigSGIX,
                       (Display *dpy, GLXFBConfigSGIX fbconfig,
                        int renderType, GLXContext shareList, Bool direct),
-                      (dpy, config, renderType, shareList, direct),
+                      (dpy, fbconfig, renderType, shareList, direct),
                       glXCreateNewContext)
 
 _GLX_PUBLIC GLXFBConfigSGIX