deps: update ChakraCore to chakra-core/ChakraCore@4336e29ca2

[1.8>1.9] [MERGE #4568 @aneeshdk] Fixing usage of arguments inside eval in arrow functions Merge pull request #4568 from aneeshdk:ArrowEvalArgumentsIssue When an inner arrow function has eval it may end up using arguments symbol. So we have to mark the parent that arguments is in use. Reviewed-By: chakrabot <[email protected]>
nodejs · Jan 18, 2018 · 3571d06 · 3571d06
1 parent 03ffff1
commit 3571d06
Show file tree

Hide file tree

Showing 2 changed files with 93 additions and 1 deletion.
diff --git a/deps/chakrashim/core/lib/Parser/Parse.cpp b/deps/chakrashim/core/lib/Parser/Parse.cpp
@@ -3787,6 +3787,7 @@ ParseNodePtr Parser::ParsePostfixOperators(
                             ReferenceSpecialName(wellKnownPropertyPids._newTarget);
                             ReferenceSpecialName(wellKnownPropertyPids._super);
                             ReferenceSpecialName(wellKnownPropertyPids._superConstructor);
+                            ReferenceSpecialName(wellKnownPropertyPids.arguments);
                         }
 
                         pnode->sxCall.callOfConstants = callOfConstants;
@@ -3809,6 +3810,7 @@ ParseNodePtr Parser::ParsePostfixOperators(
                             ReferenceSpecialName(wellKnownPropertyPids._newTarget);
                             ReferenceSpecialName(wellKnownPropertyPids._super);
                             ReferenceSpecialName(wellKnownPropertyPids._superConstructor);
+                            ReferenceSpecialName(wellKnownPropertyPids.arguments);
                         }
                         pToken->tk = tkNone; // This is no longer an identifier
                     }
@@ -5116,7 +5118,7 @@ ParseNodePtr Parser::ParseFncDecl(ushort flags, LPCOLESTR pNameHint, const bool
     // binding of "arguments.  To ensure the arguments object of the enclosing
     // non-lambda function is loaded propagate the UsesArguments flag up to
     // the parent function
-    if (fLambda && pnodeFnc->sxFnc.UsesArguments())
+    if (fLambda && (pnodeFnc->sxFnc.UsesArguments() || pnodeFnc->sxFnc.CallsEval()))
     {
         ParseNodePtr pnodeFncParent = GetCurrentFunctionNode();
 

diff --git a/deps/chakrashim/core/test/es6/lambda1.js b/deps/chakrashim/core/test/es6/lambda1.js
@@ -557,6 +557,96 @@ var tests = [
             var l = () => await (5)
             assert.areEqual('() => await (5)', '' + l, "Regular lambda should also work, though this await looks like a function call");
         }
+    },
+    {
+        name: "Accessing arguments symbol in eval inside lambda",
+        body: function () {
+            function f1(a) {
+                b = () => eval("arguments[0]");
+                assert.areEqual(b(), 1, "Eval should be able to access the arguments symbol");
+            }
+            f1(1);
+
+            function f2(a) {
+                b = (x = eval("arguments[0]")) => x;
+                assert.areEqual(b(), 2, "Eval in param scope should be able to access the arguments symbol")
+            }
+            f2(2);
+
+            function f3(arguments = [3]) {
+                b = () => eval("arguments[0]");
+                assert.areEqual(b(), 3, "Eval should be able to access the arguments param symbol");
+            }
+            f3();
+
+            function f4() {
+                var arguments = [4];
+                b = () => eval("arguments[0]");
+                assert.areEqual(b(), 4, "Eval should be able to access the arguments from the body");
+            }
+            f4(100);
+
+            function f5() {
+                b = () => {
+                    return eval("arguments()");
+                }
+                assert.areEqual(b()[0], 5, "Eval should be able to access the arguments function definition from the body");
+                function arguments() {
+                    return [5];
+                }
+            }
+            f5();
+
+            function f6() {
+                b = () => {
+                    return eval("arguments");
+                }
+                assert.areEqual(b()[0], 6, "Eval should be able to access the arguments function definition in block");
+
+                {
+                    function arguments() {
+                        return [100];
+                    }
+                }
+            }
+            f6(6);
+
+            function f7() {
+                b = () => eval("arguments[0]");
+                assert.areEqual(b(), 7, "Eval should be able to access the built-in arguments from the body");
+                var arguments = [100];
+            }
+            f7(7);
+
+            function f8() {
+                b = (arguments = [8]) => eval("arguments");
+                assert.areEqual(b()[0], 8, "Eval should be able to access the arguments lambda param");
+            }
+            f8(100);
+
+            function f9() {
+                b = (arguments = [9], c = eval("arguments")) => c;
+                assert.areEqual(b()[0], 9, "Eval should be able to access the arguments lambda param in the param scope");
+            }
+            f9();
+
+            function f10() {
+                b = () => {
+                    assert.areEqual(eval("arguments"), undefined, "Eval should access arguments from the lambda body itself");
+                    var arguments = 100;
+                }
+            }
+            f10(100);
+
+            function f11() {
+                arguments;
+                b = () => {
+                    assert.areEqual(eval("arguments"), undefined, "Eval should access arguments from the lambda body itself when the parent function uses arguments");
+                    var arguments = 100;
+                }
+            }
+            f11(100);
+        }
     }
 ];