[[FIX]] Tolerate dangling NewExpression

jugglinmike · rwaldron · commit 7c890aa32b4f · 2021-08-06T15:07:33.000-04:00
diff --git a/src/jshint.js b/src/jshint.js
@@ -2717,11 +2717,16 @@ var JSHINT = (function() {
 
     var opening = state.tokens.next;
     var c = expression(context, 155), i;
+
+    if (!c) {
+      return this;
+    }
+
     if (!c.paren && c.rbp > 160) {
       error("E024", opening, opening.value);
     }
 
-    if (c && c.id !== "function") {
+    if (c.id !== "function") {
       if (c.identifier) {
         switch (c.value) {
         case "Number":
diff --git a/tests/unit/parser.js b/tests/unit/parser.js
@@ -10598,3 +10598,26 @@ exports.optionalChaining = function (test) {
 
   test.done();
 };
+
+// gh-3556: "Crash parsing: throw new"
+exports.loneNew = function (test) {
+  TestRun(test, "as reported")
+    .addError(4, 3, "Expected an identifier and instead saw '}'.")
+    .addError(4, 4, "Missing semicolon.")
+    .addError(1, 21, "Unmatched '{'.")
+    .addError(5, 1, "Unrecoverable syntax error. (100% scanned).")
+    .test([
+      "function code(data) {",
+      "  if (data.request === 'foo') {",
+      "    throw new ",
+      "  }",
+      "}"
+    ]);
+
+  TestRun(test, "simplified")
+    .addError(1, 4, "Expected an identifier and instead saw ';'.")
+    .addError(1, 5, "Missing semicolon.")
+    .test("new;");
+
+  test.done();
+};
