From 93f82137fd96afa079d0e763a7bf173387547838 Mon Sep 17 00:00:00 2001 From: "Ryan C. Gordon" Date: Fri, 19 Jul 2013 00:22:57 -0400 Subject: [PATCH] Fixed wrong type being passed to eglGetConfigAttrib() (thanks, Alexander!). 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. --- src/video/x11/SDL_x11opengles.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/video/x11/SDL_x11opengles.c b/src/video/x11/SDL_x11opengles.c index 218421b32..ba72aff17 100644 --- 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;