Re: [Mesa-dev] [PATCH 2/5] glx: add extra null check in getFBConfigs

2014-02-17 Thread Ian Romanick
I think if __glXQueryServerString returns NULL, getFBConfigs should return failure. It also looks like __glXQueryServerString could use some NULL checks. On 02/14/2014 05:23 AM, Juha-Pekka Heikkila wrote: > Signed-off-by: Juha-Pekka Heikkila > --- > src/glx/glxext.c | 3 ++- > 1 file changed, 2

[Mesa-dev] [PATCH 2/5] glx: add extra null check in getFBConfigs

2014-02-14 Thread Juha-Pekka Heikkila
Signed-off-by: Juha-Pekka Heikkila --- src/glx/glxext.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/glx/glxext.c b/src/glx/glxext.c index 4a195bd..837c5b0 100644 --- a/src/glx/glxext.c +++ b/src/glx/glxext.c @@ -686,7 +686,8 @@ static GLboolean fb_req->glxCode