From 6c19ddaef98d71a0a9a6517478874c31cfb48a64 Mon Sep 17 00:00:00 2001
From: Robert Fancsik <robert.fancsik@h-lab.eu>
Date: Tue, 4 Jan 2022 12:35:27 +0100
Subject: [PATCH] Fix duplicated private identifier lookup

This patch fixes #4921.

JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik robert.fancsik@h-lab.eu
---
 jerry-core/parser/js/js-parser-expr.c | 31 ++++++++++++++++-----------
 tests/jerry/es.next/private_fields.js |  7 ++++++
 2 files changed, 26 insertions(+), 12 deletions(-)

diff --git a/jerry-core/parser/js/js-parser-expr.c b/jerry-core/parser/js/js-parser-expr.c
index 03309c128d..d1a3043f5f 100644
--- a/jerry-core/parser/js/js-parser-expr.c
+++ b/jerry-core/parser/js/js-parser-expr.c
@@ -638,16 +638,23 @@ parser_parse_class_body (parser_context_t *context_p, /**< context */
       continue;
     }
 
-    if (is_private)
-    {
-      parser_check_duplicated_private_field (context_p, SCANNER_PRIVATE_FIELD_PROPERTY_GETTER_SETTER);
-    }
-
-    bool is_constructor_literal = context_p->token.type == LEXER_LITERAL && parser_is_constructor_literal (context_p);
+    bool is_constructor_literal = false;
 
-    if (is_private && is_constructor_literal && lexer_check_next_character (context_p, LIT_CHAR_LEFT_PAREN))
+    if (context_p->token.type == LEXER_LITERAL)
     {
-      parser_raise_error (context_p, PARSER_ERR_CLASS_PRIVATE_CONSTRUCTOR);
+      if (parser_is_constructor_literal (context_p))
+      {
+        is_constructor_literal = true;
+
+        if (is_private && lexer_check_next_character (context_p, LIT_CHAR_LEFT_PAREN))
+        {
+          parser_raise_error (context_p, PARSER_ERR_CLASS_PRIVATE_CONSTRUCTOR);
+        }
+      }
+      else
+      {
+        parser_check_duplicated_private_field (context_p, SCANNER_PRIVATE_FIELD_PROPERTY_GETTER_SETTER);
+      }
     }
 
     if (!is_static && is_constructor_literal)
@@ -809,9 +816,9 @@ parser_parse_class_body (parser_context_t *context_p, /**< context */
         lexer_expect_object_literal_id (context_p, ident_opts);
       }
 
-      if (is_private)
+      if (is_private && context_p->token.type == LEXER_LITERAL)
       {
-        if (context_p->token.type == LEXER_LITERAL && parser_is_constructor_literal (context_p))
+        if (parser_is_constructor_literal (context_p))
         {
           parser_raise_error (context_p, PARSER_ERR_CLASS_PRIVATE_CONSTRUCTOR);
         }
@@ -836,9 +843,9 @@ parser_parse_class_body (parser_context_t *context_p, /**< context */
 
       status_flags |= PARSER_IS_GENERATOR_FUNCTION | PARSER_DISALLOW_AWAIT_YIELD;
 
-      if (is_private)
+      if (is_private && context_p->token.type == LEXER_LITERAL)
       {
-        if (context_p->token.type == LEXER_LITERAL && parser_is_constructor_literal (context_p))
+        if (parser_is_constructor_literal (context_p))
         {
           parser_raise_error (context_p, PARSER_ERR_CLASS_PRIVATE_CONSTRUCTOR);
         }
diff --git a/tests/jerry/es.next/private_fields.js b/tests/jerry/es.next/private_fields.js
index 19d533b8c8..6b51540867 100644
--- a/tests/jerry/es.next/private_fields.js
+++ b/tests/jerry/es.next/private_fields.js
@@ -47,6 +47,13 @@ check_syntax_error("class A { static get #a(){}; set #a(){}; #a; }");
 check_syntax_error("class A { static #a(){}; #a; }");
 check_syntax_error("class A extends B{ foo(){ super.#a }}");
 check_syntax_error("class A extends function() { x = this.#foo; }{ #foo; };");
+check_syntax_error("class A { static async *#bar(x) { } #bar }");
+check_syntax_error("class A { static async #bar(x) { } #bar }");
+check_syntax_error("class A { static *#bar(x) { } #bar }");
+check_syntax_error("class A { async *#bar(x) { } #bar }");
+check_syntax_error("class A { async #bar(x) { } #bar }");
+check_syntax_error("class A { *#bar(x) { } #bar }");
+
 
 class A {
   #a = 1;