Fixed for-in unwanted detection of arrow function.

nginx · Nov 17, 2022 · 36f649b · 36f649b
1 parent 66e4e6c
commit 36f649b
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 6 deletions.
diff --git a/src/njs_parser.c b/src/njs_parser.c
@@ -4291,14 +4291,16 @@ njs_parser_assignment_expression(njs_parser_t *parser,
 {
     njs_int_t  ret;
 
-    ret = njs_parser_match_arrow_expression(parser, token);
-    if (ret == NJS_OK) {
-        njs_parser_next(parser, njs_parser_arrow_function);
+    if (!parser->use_lhs) {
+        ret = njs_parser_match_arrow_expression(parser, token);
+        if (ret == NJS_OK) {
+            njs_parser_next(parser, njs_parser_arrow_function);
 
-        return NJS_OK;
+            return NJS_OK;
 
-    } else if (ret == NJS_ERROR) {
-        return NJS_ERROR;
+        } else if (ret == NJS_ERROR) {
+            return NJS_ERROR;
+        }
     }
 
     njs_parser_next(parser, njs_parser_conditional_expression);

diff --git a/src/test/njs_unit_test.c b/src/test/njs_unit_test.c
@@ -2965,6 +2965,9 @@ static njs_unit_test_t  njs_test[] =
     { njs_str("for(I in``[)8"),
       njs_str("SyntaxError: Unexpected token \")\" in 1") },
 
+    { njs_str("for(9A=>>"),
+      njs_str("SyntaxError: Unexpected token \"A\" in 1") },
+
     /* switch. */
 
     { njs_str("switch"),