1340089 - Check the binding name in comprehensionFor.

This commit is contained in:
Fedor 2019-09-05 20:06:15 +03:00
parent 2bb2a37638
commit 0e170d0294
2 changed files with 109 additions and 0 deletions

View File

@ -8101,6 +8101,8 @@ Parser<ParseHandler>::comprehensionFor(GeneratorKind comprehensionKind)
MUST_MATCH_TOKEN_FUNC(TokenKindIsPossibleIdentifier, JSMSG_NO_VARIABLE_NAME);
RootedPropertyName name(context, bindingIdentifier(YieldIsKeyword));
if (!name)
return null();
if (name == context->names().let) {
error(JSMSG_LET_COMP_BINDING);
return null();

View File

@ -0,0 +1,107 @@
var BUGNUMBER = 1340089;
var summary = "Comprehension should check the binding names";
print(BUGNUMBER + ": " + summary);
// Non strict mode.
// Keywords, literals, 'let', and 'yield' are not allowed.
assertThrowsInstanceOf(function () {
eval("[for (true of [1]) 2]");
}, SyntaxError);
assertThrowsInstanceOf(function () {
eval("(for (true of [1]) 2)");
}, SyntaxError);
assertThrowsInstanceOf(function () {
eval("[for (throw of [1]) 2]");
}, SyntaxError);
assertThrowsInstanceOf(function () {
eval("(for (throw of [1]) 2)");
}, SyntaxError);
assertThrowsInstanceOf(function () {
eval("[for (let of [1]) 2]");
}, SyntaxError);
assertThrowsInstanceOf(function () {
eval("(for (let of [1]) 2)");
}, SyntaxError);
assertThrowsInstanceOf(function () {
eval("[for (yield of [1]) 2]");
}, SyntaxError);
assertThrowsInstanceOf(function () {
eval("(for (yield of [1]) 2)");
}, SyntaxError);
eval("[for (public of [1]) 2]");
eval("(for (public of [1]) 2)");
eval("[for (static of [1]) 2]");
eval("(for (static of [1]) 2)");
// Strict mode.
// All reserved words are not allowed.
assertThrowsInstanceOf(function () {
"use strict";
eval("[for (true of [1]) 2]");
}, SyntaxError);
assertThrowsInstanceOf(function () {
"use strict";
eval("(for (true of [1]) 2)");
}, SyntaxError);
assertThrowsInstanceOf(function () {
"use strict";
eval("[for (throw of [1]) 2]");
}, SyntaxError);
assertThrowsInstanceOf(function () {
"use strict";
eval("(for (throw of [1]) 2)");
}, SyntaxError);
assertThrowsInstanceOf(function () {
"use strict";
eval("[for (let of [1]) 2]");
}, SyntaxError);
assertThrowsInstanceOf(function () {
"use strict";
eval("(for (let of [1]) 2)");
}, SyntaxError);
assertThrowsInstanceOf(function () {
"use strict";
eval("[for (yield of [1]) 2]");
}, SyntaxError);
assertThrowsInstanceOf(function () {
"use strict";
eval("(for (yield of [1]) 2)");
}, SyntaxError);
assertThrowsInstanceOf(function () {
"use strict";
eval("[for (public of [1]) 2]");
}, SyntaxError);
assertThrowsInstanceOf(function () {
"use strict";
eval("(for (public of [1]) 2)");
}, SyntaxError);
assertThrowsInstanceOf(function () {
"use strict";
eval("[for (static of [1]) 2]");
}, SyntaxError);
assertThrowsInstanceOf(function () {
"use strict";
eval("(for (static of [1]) 2)");
}, SyntaxError);
(function () {
"use strict";
eval("[for (await of [1]) 2]");
eval("(for (await of [1]) 2)");
})();
if (typeof reportCompare === "function")
reportCompare(true, true);