diff options
author | Krasimir Georgiev <krasimir@google.com> | 2019-08-08 11:56:18 +0000 |
---|---|---|
committer | Krasimir Georgiev <krasimir@google.com> | 2019-08-08 11:56:18 +0000 |
commit | 9ab051bdda8de83df9abbaf00e76500875c3669e (patch) | |
tree | 93d08a71ab6f1f19c4162dd93110406ca0170633 /clang/lib/Format | |
parent | 0de33de81338304492526053d0c1cfa616e58992 (diff) | |
download | bcm5719-llvm-9ab051bdda8de83df9abbaf00e76500875c3669e.tar.gz bcm5719-llvm-9ab051bdda8de83df9abbaf00e76500875c3669e.zip |
[clang-format] fix crash involving invalid preprocessor line
Summary:
This (invalid) fragment is crashing clang-format:
```
#if 1
int x;
#elif
int y;
#endif
```
The reason being that the parser expects a token after `#elif`, and the
subsequent parsing of the next line does not check if `CurrentToken` is null.
Reviewers: gribozavr
Reviewed By: gribozavr
Subscribers: cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D65940
llvm-svn: 368280
Diffstat (limited to 'clang/lib/Format')
-rw-r--r-- | clang/lib/Format/TokenAnnotator.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/clang/lib/Format/TokenAnnotator.cpp b/clang/lib/Format/TokenAnnotator.cpp index 6e0369f27e7..cc0a954dbfb 100644 --- a/clang/lib/Format/TokenAnnotator.cpp +++ b/clang/lib/Format/TokenAnnotator.cpp @@ -1099,6 +1099,8 @@ private: public: LineType parseLine() { + if (!CurrentToken) + return LT_Invalid; NonTemplateLess.clear(); if (CurrentToken->is(tok::hash)) return parsePreprocessorDirective(); |