On 12/12/2011 10:31 PM, jian.j.z...@intel.com wrote: > From: Jian Zhao<jian.j.z...@intel.com> > > --- > src/mesa/main/texgetimage.c | 8 ++++++++ > 1 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/src/mesa/main/texgetimage.c b/src/mesa/main/texgetimage.c > index ae0d51f..3f24187 100644 > --- a/src/mesa/main/texgetimage.c > +++ b/src/mesa/main/texgetimage.c > @@ -708,6 +708,14 @@ getteximage_error_check(struct gl_context *ctx, GLenum > target, GLint level, > return GL_TRUE; > } > > + if (!_mesa_is_legal_format_and_type(ctx, format, type)) { > + /*GL_INVALID_OPERATION is generated by a format/type > + * mismatch (see the 1.2 spec page 94, sec 3.6.4.) > + */ > + _mesa_error(ctx, GL_INVALID_OPERATION, "glGetTexImage(target)"); > + return GL_TRUE; > + } > + > baseFormat = _mesa_get_format_base_format(texImage->TexFormat); > > /* Make sure the requested image format is compatible with the
Reviewed-by: Brian Paul <bri...@vmware.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev