From 2502ee673893394cfa01f1f26ab9b657a3ccb2dc Mon Sep 17 00:00:00 2001
From: Brian Paul <brianp@vmware.com>
Date: Sat, 23 Oct 2010 09:35:01 -0600
Subject: mesa: new glDrawPixels error check for integer formats

---
 src/mesa/main/drawpix.c | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

(limited to 'src/mesa/main/drawpix.c')

diff --git a/src/mesa/main/drawpix.c b/src/mesa/main/drawpix.c
index ac92a3fb08..890d0ab5a7 100644
--- a/src/mesa/main/drawpix.c
+++ b/src/mesa/main/drawpix.c
@@ -30,9 +30,11 @@
 #include "enums.h"
 #include "feedback.h"
 #include "framebuffer.h"
+#include "image.h"
 #include "readpix.h"
+#include "shaderobj.h"
 #include "state.h"
-#include "main/dispatch.h"
+#include "dispatch.h"
 
 
 #if FEATURE_drawpix
@@ -94,6 +96,14 @@ _mesa_DrawPixels( GLsizei width, GLsizei height,
       goto end; /* no-op, not an error */
    }
 
+   if (_mesa_is_integer_format(format) &&
+       !_mesa_is_fragment_shader_active(ctx)) {
+      /* A fragment shader is required when drawing integer formats */
+      _mesa_error(ctx, GL_INVALID_OPERATION,
+                  "glDrawPixels(integer format but no fragment shader)");
+      goto end;
+   }
+
    if (ctx->RenderMode == GL_RENDER) {
       if (width > 0 && height > 0) {
          /* Round, to satisfy conformance tests (matches SGI's OpenGL) */
-- 
cgit v1.2.3