From c42ca36d67939bb1cda8bb362ff8da7c0da24a1d Mon Sep 17 00:00:00 2001 From: Ian Romanick Date: Tue, 18 Oct 2011 16:28:21 -0700 Subject: [PATCH] mesa: Add missing check for glUniform*v count > 1 on non-array Signed-off-by: Ian Romanick Tested-by: Tom Stellard --- src/mesa/main/uniform_query.cpp | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/src/mesa/main/uniform_query.cpp b/src/mesa/main/uniform_query.cpp index 496f60416bf..debe3ae5963 100644 --- a/src/mesa/main/uniform_query.cpp +++ b/src/mesa/main/uniform_query.cpp @@ -399,6 +399,8 @@ validate_uniform_parameters(struct gl_context *ctx, * * - if no variable with a location of location exists in the * program object currently in use and location is not -1, + * - if count is greater than one, and the uniform declared in the + * shader is not an array variable, */ if (location < -1) { _mesa_error(ctx, GL_INVALID_OPERATION, "%s(location=%d)", @@ -414,6 +416,13 @@ validate_uniform_parameters(struct gl_context *ctx, return false; } + if (shProg->UniformStorage[*loc].array_elements == 0 && count > 1) { + _mesa_error(ctx, GL_INVALID_OPERATION, + "%s(count > 1 for non-array, location=%d)", + caller, location); + return false; + } + /* This case should be impossible. The implication is that a call like * glGetUniformLocation(prog, "foo[8]") was successful but "foo" is not an * array. -- 2.30.2