beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 07:30:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

LibJS: Fix runaway let scope when parsing for-in/of statements

Browse source 
This requires variables that should be exported to the script host
or to other scripts to be declared as var (such as in test-common.js),
since lexically-scoped variables won't be visible.
This commit is contained in:
Hendi 2021-07-05 21:31:51 +02:00 committed by Linus Groh
parent 793e1bf28a
commit c194afd17c
Notes: sideshowbarker 2024-07-18 10:20:06 +09:00 
Author: https://github.com/Hendi48
Commit: https://github.com/SerenityOS/serenity/commit/c194afd17c5
Pull-request: https://github.com/SerenityOS/serenity/pull/8439
Issue: https://github.com/SerenityOS/serenity/issues/8299
Reviewed-by: https://github.com/linusg ✅
2 changed files with 10 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
Userland/Libraries/LibJS/Parser.cpp
View file

@ -2033,6 +2033,11 @@ NonnullRefPtr<Statement> Parser::parse_for_statement()
consume(TokenType::ParenOpen);
bool in_scope = false;
ScopeGuard guard([&]() {
if (in_scope)
m_state.let_scopes.take_last();
});
RefPtr<ASTNode> init;
if (!match(TokenType::Semicolon)) {
if (match_expression()) {
@ -2075,10 +2080,6 @@ NonnullRefPtr<Statement> Parser::parse_for_statement()
TemporaryChange continue_change(m_state.in_continue_context, true);
auto body = parse_statement();
if (in_scope) {
m_state.let_scopes.take_last();
}
return create_ast_node<ForStatement>({ m_state.current_token.filename(), rule_start.position(), position() }, move(init), move(test), move(update), move(body));
}

10
Userland/Libraries/LibJS/Tests/test-common.js
View file

@ -1,10 +1,10 @@
let describe;
let test;
let expect;
var describe;
var test;
var expect;
// Stores the results of each test and suite. Has a terrible
// name to avoid name collision.
let __TestResults__ = {};
var __TestResults__ = {};
// So test names like "toString" don't automatically produce an error
Object.setPrototypeOf(__TestResults__, null);
@ -12,7 +12,7 @@ Object.setPrototypeOf(__TestResults__, null);
// This array is used to communicate with the C++ program. It treats
// each message in this array as a separate message. Has a terrible
// name to avoid name collision.
let __UserOutput__ = [];
var __UserOutput__ = [];
// We also rebind console.log here to use the array above
console.log = (...args) => {