thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-20 08:40:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

QueryNormalizer only checks stack size

Browse Source
This commit is contained in:
vdimir 2023-10-16 10:33:04 +00:00
parent ba5cd016dd
commit d7c77420b6
No known key found for this signature in database
GPG Key ID: 6EE4CE2BEDC51862
2 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Interpreters/QueryNormalizer.cpp
View File

@ -120,7 +120,6 @@ void QueryNormalizer::visit(ASTIdentifier & node, ASTPtr & ast, Data & data)
/// In a construct like "a AS b", where a is an alias, you must set alias b to the result of substituting alias a.
/// Check size of the alias before cloning too large alias AST
alias_node->checkSize(data.settings.max_expanded_ast_elements);
current_asts.insert(alias_node.get());
ast = alias_node->clone();
ast->setAlias(node_alias);
@ -139,7 +138,6 @@ void QueryNormalizer::visit(ASTIdentifier & node, ASTPtr & ast, Data & data)
/// Check size of the alias before cloning too large alias AST
alias_node->checkSize(data.settings.max_expanded_ast_elements);
auto alias_name = ast->getAliasOrColumnName();
current_asts.insert(alias_node.get());
ast = alias_node->clone();
ast->setAlias(alias_name);

2
tests/queries/0_stateless/02896_cyclic_aliases_crash.sql
View File

@ -6,7 +6,7 @@ SELECT
val + 1 as prev,
val + prev as val
FROM ( SELECT 1 as val )
; -- { serverError CYCLIC_ALIASES }
; -- { serverError CYCLIC_ALIASES, TOO_DEEP_RECURSION }
SELECT