On 02/19/2012 06:53 PM, Brian Paul wrote:
From: Brian Paul<[email protected]>

NOTE: This is a candidate for the 8.0 branch.
---
  src/mesa/main/varray.c |    1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/src/mesa/main/varray.c b/src/mesa/main/varray.c
index 77c1d7d..dfe5064 100644
--- a/src/mesa/main/varray.c
+++ b/src/mesa/main/varray.c
@@ -185,6 +185,7 @@ update_array(struct gl_context *ctx,
         (type == GL_UNSIGNED_INT_2_10_10_10_REV ||
          type == GL_INT_2_10_10_10_REV)&&  size != 4) {
        _mesa_error(ctx, GL_INVALID_OPERATION, "%s(size=%d)", func, size);
+      return;
     }

     ASSERT(size<= 4);

Series looks good to me too.
Reviewed-by: Kenneth Graunke <[email protected]>
_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to