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

Reply via email to