godotengine/godot
1
0
Fork 0
mirror of https://github.com/godotengine/godot.git synced 2025-11-17 14:11:06 +00:00
Code Releases Wiki Activity

Prevents shader crashing if varying assigned incorrectly

Browse Source
This commit is contained in:
Yuri Roubinsky
2021-07-26 14:38:33 +03:00
parent 04fd284a9b
commit b47b3a9957
2 changed files with 17 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
servers/rendering/shader_language.cpp
View File

@@ -2820,6 +2820,20 @@ bool ShaderLanguage::is_token_operator(TokenType p_type) {
p_type == TK_COLON); p_type == TK_COLON);
} }
bool ShaderLanguage::is_token_operator_assign(TokenType p_type) {
return (p_type == TK_OP_ASSIGN ||
p_type == TK_OP_ASSIGN_ADD ||
p_type == TK_OP_ASSIGN_SUB ||
p_type == TK_OP_ASSIGN_MUL ||
p_type == TK_OP_ASSIGN_DIV ||
p_type == TK_OP_ASSIGN_MOD ||
p_type == TK_OP_ASSIGN_SHIFT_LEFT ||
p_type == TK_OP_ASSIGN_SHIFT_RIGHT ||
p_type == TK_OP_ASSIGN_BIT_AND ||
p_type == TK_OP_ASSIGN_BIT_OR ||
p_type == TK_OP_ASSIGN_BIT_XOR);
}
bool ShaderLanguage::convert_constant(ConstantNode *p_constant, DataType p_to_type, ConstantNode::Value *p_value) { bool ShaderLanguage::convert_constant(ConstantNode *p_constant, DataType p_to_type, ConstantNode::Value *p_value) {
if (p_constant->datatype == p_to_type) { if (p_constant->datatype == p_to_type) {
if (p_value) { if (p_value) {
@@ -4240,7 +4254,8 @@ ShaderLanguage::Node *ShaderLanguage::_parse_expression(BlockNode *p_block, cons
Token next_token = _get_token(); Token next_token = _get_token();
_set_tkpos(prev_pos); _set_tkpos(prev_pos);
String error; String error;
if (next_token.type == TK_OP_ASSIGN) {
if (is_token_operator_assign(next_token.type)) {
if (!_validate_varying_assign(shader->varyings[identifier], &error)) { if (!_validate_varying_assign(shader->varyings[identifier], &error)) {
_set_error(error); _set_error(error);
return nullptr; return nullptr;

1
servers/rendering/shader_language.h
View File

@@ -766,6 +766,7 @@ public:
static String get_datatype_name(DataType p_type); static String get_datatype_name(DataType p_type);
static bool is_token_nonvoid_datatype(TokenType p_type); static bool is_token_nonvoid_datatype(TokenType p_type);
static bool is_token_operator(TokenType p_type); static bool is_token_operator(TokenType p_type);
static bool is_token_operator_assign(TokenType p_type);
static bool convert_constant(ConstantNode *p_constant, DataType p_to_type, ConstantNode::Value *p_value = nullptr); static bool convert_constant(ConstantNode *p_constant, DataType p_to_type, ConstantNode::Value *p_value = nullptr);
static DataType get_scalar_type(DataType p_type); static DataType get_scalar_type(DataType p_type);