diff options
author | nobled <nobled@dreamwidth.org> | 2010-08-19 14:06:21 -0400 |
---|---|---|
committer | Eric Anholt <eric@anholt.net> | 2011-02-11 18:19:10 -0800 |
commit | b5dc40710d0e5edffb9f673dfbf26df4d0043eef (patch) | |
tree | 94571ea6e53cbe459dae5862e64a018d941355f7 /src/glx/glxcmds.c | |
parent | df54b53b7d12a3bca5867b6649cb308feb36f0da (diff) |
glx: Put null check before use
'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.
Diffstat (limited to 'src/glx/glxcmds.c')
-rw-r--r-- | src/glx/glxcmds.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/glx/glxcmds.c b/src/glx/glxcmds.c index d12ff9684e..80eaf72b7d 100644 --- a/src/glx/glxcmds.c +++ b/src/glx/glxcmds.c @@ -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; |