glx: Put null check before use
authornobled <nobled@dreamwidth.org>
Thu, 19 Aug 2010 18:06:21 +0000 (14:06 -0400)
committerEric Anholt <eric@anholt.net>
Sat, 12 Feb 2011 02:19:10 +0000 (18:19 -0800)
'dpy' was being checked for null *after* it was already used once.

Also add a null check for psc, and drop gc's redundant initialization.

src/glx/glxcmds.c

index d12ff9684e2579b8d29473d3ecb3645e28f70630..80eaf72b7d504b7abb1181d8c23e57ee0ac0a4c6 100644 (file)
@@ -215,12 +215,16 @@ CreateContext(Display * dpy, int generic_id,
               Bool allowDirect,
              unsigned code, int renderType, int screen)
 {
-   struct glx_context *gc = NULL;
-   struct glx_screen *const psc = GetGLXScreenConfigs(dpy, screen);
+   struct glx_context *gc;
+   struct glx_screen *psc;
    struct glx_context *shareList = (struct glx_context *) shareList_user;
    if (dpy == NULL)
       return NULL;
 
+   psc = GetGLXScreenConfigs(dpy, screen);
+   if (psc == NULL)
+      return NULL;
+
    if (generic_id == None)
       return NULL;