diff options
author | Emil Velikov <[email protected]> | 2016-09-30 11:01:27 +0100 |
---|---|---|
committer | Emil Velikov <[email protected]> | 2016-10-06 15:03:47 +0100 |
commit | 2e9e05dfca18c7f09caa40396d6dd4f2b3ddc1d4 (patch) | |
tree | 94ba46d4a060d88116db80bb2cdf33fdebbfd50c /src/glx/glxext.c | |
parent | e542ed463dd5ae04e730901208344477afdc6a62 (diff) |
glx: return GL_FALSE from glx_screen_init where applicable.
Return GL_FALSE if we fail to find any fb/visual configs, otherwise we
end up with all sorts of chaos further down the GLX stack.
Signed-off-by: Emil Velikov <[email protected]>
Reviewed-by: Marek Olšák <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>
Diffstat (limited to 'src/glx/glxext.c')
-rw-r--r-- | src/glx/glxext.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/glx/glxext.c b/src/glx/glxext.c index f6bc377add5..9ef7ff5f694 100644 --- a/src/glx/glxext.c +++ b/src/glx/glxext.c @@ -748,8 +748,11 @@ glx_screen_init(struct glx_screen *psc, psc->dpy = priv->dpy; psc->display = priv; - getVisualConfigs(psc, priv, screen); - getFBConfigs(psc, priv, screen); + if (!getVisualConfigs(psc, priv, screen)) + return GL_FALSE; + + if (!getFBConfigs(psc, priv, screen)) + return GL_FALSE; return GL_TRUE; } |