commit d78e33a8da3130f424a10c6053dad05a010beb6e
author Corentin Wallez <cwallez@chromium.org> Mon Oct 30 12:33:52 2017 -0400
committer Commit Bot <commit-bot@chromium.org> Wed Nov 01 17:01:08 2017 +0000
tree 60701d32c7f8b30aaaccc1a6a4ecf904a810d76f
parent 3590555ddbda4b55ed27bb9f898f477290ef4cfe

preprocessor: Fix lineno overflow on line continuations

BUG=chromium:774807

Change-Id: I4b3fbee31683f411810080572cfff0f8307b93bf
Reviewed-on: https://chromium-review.googlesource.com/744183
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

src/compiler/preprocessor/Tokenizer.cpp

diff --git a/src/compiler/preprocessor/Tokenizer.cpp b/src/compiler/preprocessor/Tokenizer.cpp
index 76b376b..d8a9b9a 100644
--- a/src/compiler/preprocessor/Tokenizer.cpp
+++ b/src/compiler/preprocessor/Tokenizer.cpp
@@ -1680,11 +1680,18 @@
     yylloc->line = yylineno;
     yylval->clear();
 
-    if (YY_START == COMMENT)
+    // Line number overflows fake EOFs to exit early, check for this case.
+    if (yylineno == INT_MAX)
+    {
+        yyextra->diagnostics->report(pp::Diagnostics::PP_TOKENIZER_ERROR,
+                                     pp::SourceLocation(yyfileno, yylineno),
+                                     "Integer overflow on line number");
+    }
+    else if (YY_START == COMMENT)
     {
         yyextra->diagnostics->report(pp::Diagnostics::PP_EOF_IN_COMMENT,
                                      pp::SourceLocation(yyfileno, yylineno),
-                                     "");
+                                     "EOF while in a comment");
     }
     yyterminate();
 }