fix clang testsuite crash (cppcheck-opensource#2341)

IOBYTE · danmar · commit 0fed6f009155 · 2019-11-09T18:00:21.000+01:00
diff --git a/lib/tokenize.cpp b/lib/tokenize.cpp
@@ -10890,6 +10890,8 @@ void Tokenizer::simplifyOperatorName()
                 op += par->strAt(1);
                 par = par->tokAt(2);
                 done = true;
+            } else if (par->str() != "(") {
+                syntaxError(par);
             }
         }
 
diff --git a/test/testgarbage.cpp b/test/testgarbage.cpp
@@ -237,6 +237,7 @@ class TestGarbage : public TestFixture {
         TEST_CASE(garbageCode203); // #8972
         TEST_CASE(garbageCode204);
         TEST_CASE(garbageCode205);
+        TEST_CASE(garbageCode206);
 
         TEST_CASE(garbageCodeFuzzerClientMode1); // test cases created with the fuzzer client, mode 1
 
@@ -1625,6 +1626,15 @@ class TestGarbage : public TestFixture {
                   "}"); // don't crash
     }
 
+    void garbageCode206() {
+        {
+            ASSERT_EQUALS("[test.cpp:1] syntax error", getSyntaxError("void foo() { for (auto operator new : int); }"));
+        }
+        {
+            ASSERT_EQUALS("[test.cpp:1] syntax error", getSyntaxError("void foo() { for (a operator== :) }"));
+        }
+    }
+
     void syntaxErrorFirstToken() {
         ASSERT_THROW(checkCode("&operator(){[]};"), InternalError); // #7818
         ASSERT_THROW(checkCode("*(*const<> (size_t); foo) { } *(*const (size_t)() ; foo) { }"), InternalError); // #6858

Original file line number	Diff line number	Diff line change
`@@ -10890,6 +10890,8 @@ void Tokenizer::simplifyOperatorName()`
`10890`	`10890`	`op += par->strAt(1);`
`10891`	`10891`	`par = par->tokAt(2);`
`10892`	`10892`	`done = true;`
	`10893`	`+ } else if (par->str() != "(") {`
	`10894`	`+ syntaxError(par);`
`10893`	`10895`	`}`
`10894`	`10896`	`}`
`10895`	`10897`