From: Brian Paul <brian.paul@tungstengraphics.com>
Date: Wed, 6 Aug 2008 17:58:52 +0000 (-0600)
Subject: mesa: glsl: disallow initializers for varying vars
X-Git-Tag: 062012170305~17580^2~390^2~741
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ac553c6ce4718aa1e90f49c910c14e13d0d5de39;p=profile%2Fivi%2Fmesa.git

mesa: glsl: disallow initializers for varying vars
---

diff --git a/src/mesa/shader/slang/slang_codegen.c b/src/mesa/shader/slang/slang_codegen.c
index 13cad6a..e36b689 100644
--- a/src/mesa/shader/slang/slang_codegen.c
+++ b/src/mesa/shader/slang/slang_codegen.c
@@ -3647,7 +3647,7 @@ _slang_codegen_global_variable(slang_assemble_ctx *A, slang_variable *var,
       const GLuint swizzle = _slang_var_swizzle(totalSize, 0);
 
       if (var->initializer) {
-         slang_info_log_error(A->log, "illegal assignment to '%s'", varName);
+         slang_info_log_error(A->log, "illegal initializer for uniform '%s'", varName);
          return GL_FALSE;
       }
 
@@ -3695,13 +3695,21 @@ _slang_codegen_global_variable(slang_assemble_ctx *A, slang_variable *var,
       if (dbg) printf("UNIFORM (sz %d) ", totalSize);
    }
    else if (var->type.qualifier == SLANG_QUAL_VARYING) {
-      if (!_slang_type_is_float_vec_mat(var->type.specifier.type)) {
+      /* varyings must be float, vec or mat */
+      if (!_slang_type_is_float_vec_mat(var->type.specifier.type) &&
+          var->type.specifier.type != SLANG_SPEC_ARRAY) {
          slang_info_log_error(A->log,
                               "varying '%s' must be float/vector/matrix",
                               varName);
          return GL_FALSE;
       }
 
+      if (var->initializer) {
+         slang_info_log_error(A->log, "illegal initializer for varying '%s'",
+                              varName);
+         return GL_FALSE;
+      }
+
       if (prog) {
          /* user-defined varying */
          GLint varyingLoc = _mesa_add_varying(prog->Varying, varName, size);