mesa: add missing error check in _mesa_CallLists()
authorBrian Paul <brianp@vmware.com>
Mon, 8 Feb 2016 22:30:39 +0000 (15:30 -0700)
committerBrian Paul <brianp@vmware.com>
Tue, 9 Feb 2016 18:27:48 +0000 (11:27 -0700)
Generate GL_INVALID_VALUE if n < 0.  Return early if n==0 or lists==NULL.

v2: fix formatting, also check for lists==NULL.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/mesa/main/dlist.c

index cd8e3b6a2f20f438006ab16325fefb3219a8c12e..65f092936b3e3ce11ffe80ba37d2a43d64d7637f 100644 (file)
@@ -9105,6 +9105,14 @@ _mesa_CallLists(GLsizei n, GLenum type, const GLvoid * lists)
       return;
    }
 
+   if (n < 0) {
+      _mesa_error(ctx, GL_INVALID_VALUE, "glCallLists(n < 0)");
+      return;
+   } else if (n == 0 || lists == NULL) {
+      /* nothing to do */
+      return;
+   }
+
    /* Save the CompileFlag status, turn it off, execute display list,
     * and restore the CompileFlag.
     */