Fixed wrong type being passed to eglGetConfigAttrib() (thanks, Alexan…

…der!). This wanted an EGLint*, not a VisualID* cast to an EGLint*. Without this, 64-bit X11 fails here, because the datatype sizes are different. Fixes Bugzilla 1686.
libsdl-org · Jul 19, 2013 · 3bdd85e · 3bdd85e
1 parent cd6c08e
commit 3bdd85e
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/video/x11/SDL_x11opengles.c b/src/video/x11/SDL_x11opengles.c
@@ -212,7 +212,7 @@ X11_GLES_GetVisual(_THIS, Display * display, int screen)
     /* 64 seems nice. */
     EGLint attribs[64];
     EGLint found_configs = 0;
-    VisualID visual_id;
+    EGLint visual_id;
     int i;
 
     if (!_this->gles_data) {
@@ -277,7 +277,7 @@ X11_GLES_GetVisual(_THIS, Display * display, int screen)
     if (_this->gles_data->eglGetConfigAttrib(_this->gles_data->egl_display,
                                              _this->gles_data->egl_config,
                                              EGL_NATIVE_VISUAL_ID,
-                                             (EGLint *) & visual_id) ==
+                                             &visual_id) ==
         EGL_FALSE || !visual_id) {
         /* Use the default visual when all else fails */
         XVisualInfo vi_in;