On 02/08/2016 05:06 PM, Brian Paul wrote: > Generate GL_INVALID_VALUE if n < 0. Return early if n==0 or lists==NULL. > > v2: fix formatting, also check for lists==NULL.
You were correct that I was looking at save_CallLists before. Reviewed-by: Ian Romanick <ian.d.roman...@intel.com> > --- > src/mesa/main/dlist.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c > index cd8e3b6..65f0929 100644 > --- a/src/mesa/main/dlist.c > +++ b/src/mesa/main/dlist.c > @@ -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. > */ > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev