From 743a52637258a602eb4ea891b5c3bca0032724c7 Mon Sep 17 00:00:00 2001 From: Brian Paul Date: Fri, 7 Oct 2016 15:21:58 -0600 Subject: [PATCH] mesa: add sanity check assertion in update_array_format MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit At most, one of the normalized, integer, doubles bools can be true. Reviewed-by: Mathias Fröhlich --- src/mesa/main/varray.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/mesa/main/varray.c b/src/mesa/main/varray.c index 9acd8686621..0f7777bfe3c 100644 --- a/src/mesa/main/varray.c +++ b/src/mesa/main/varray.c @@ -333,6 +333,9 @@ update_array_format(struct gl_context *ctx, GLbitfield typeBit; GLenum format = GL_RGBA; + /* at most, one of these bools can be true */ + assert((int) normalized + (int) integer + (int) doubles <= 1); + if (ctx->Array.LegalTypesMask == 0 || ctx->Array.LegalTypesMaskAPI != ctx->API) { /* Compute the LegalTypesMask only once, unless the context API has * changed, in which case we want to compute it again. We can't do this -- 2.30.2