Переглянути джерело

Merge pull request #32598 from Chaosus/fix_shader_crash

Prevents shader crash if name of variable overrides function name
Yuri Roubinsky 6 роки тому
батько
коміт
b54d1276e6
1 змінених файлів з 10 додано та 0 видалено
  1. 10 0
      servers/visual/shader_language.cpp

+ 10 - 0
servers/visual/shader_language.cpp

@@ -2914,6 +2914,16 @@ ShaderLanguage::Node *ShaderLanguage::_parse_expression(BlockNode *p_block, cons
 
 				bool ok = _parse_function_arguments(p_block, p_builtin_types, func, &carg);
 
+				// Check if block has a variable with the same name as function to prevent shader crash.
+				ShaderLanguage::BlockNode *bnode = p_block;
+				while (bnode) {
+					if (bnode->variables.has(name)) {
+						_set_error("Expected function name");
+						return NULL;
+					}
+					bnode = bnode->parent_block;
+				}
+
 				//test if function was parsed first
 				for (int i = 0; i < shader->functions.size(); i++) {
 					if (shader->functions[i].name == name) {