diff options
| author | Alexey Bataev <a.bataev@hotmail.com> | 2016-01-12 09:01:25 +0000 |
|---|---|---|
| committer | Alexey Bataev <a.bataev@hotmail.com> | 2016-01-12 09:01:25 +0000 |
| commit | 0a47e65d75aa3b200f80edbba67d69697023f027 (patch) | |
| tree | b6b5b77afbdd86ca7377468dad4f32e7abaa4e02 | |
| parent | 484f0a31b34c929bc0499502b0b81577a25b510a (diff) | |
| download | bcm5719-llvm-0a47e65d75aa3b200f80edbba67d69697023f027.tar.gz bcm5719-llvm-0a47e65d75aa3b200f80edbba67d69697023f027.zip | |
Fix infinite recursion for invalid declaration, by Dmitry Polukhin
Fix for a case found by fuzzing PR23057 (comment #25 https://llvm.org/bugs/show_bug.cgi?id=23057#c25).
Differential Revision: http://reviews.llvm.org/D16065
llvm-svn: 257461
| -rw-r--r-- | clang/lib/AST/Decl.cpp | 2 | ||||
| -rw-r--r-- | clang/test/SemaCXX/linkage-invalid-decl.cpp | 9 |
2 files changed, 10 insertions, 1 deletions
diff --git a/clang/lib/AST/Decl.cpp b/clang/lib/AST/Decl.cpp index 797e07b4b78..427ca5efcd6 100644 --- a/clang/lib/AST/Decl.cpp +++ b/clang/lib/AST/Decl.cpp @@ -1184,7 +1184,7 @@ static LinkageInfo getLVForLocalDecl(const NamedDecl *D, return LinkageInfo::none(); const Decl *OuterD = getOutermostFuncOrBlockContext(D); - if (!OuterD) + if (!OuterD || OuterD->isInvalidDecl()) return LinkageInfo::none(); LinkageInfo LV; diff --git a/clang/test/SemaCXX/linkage-invalid-decl.cpp b/clang/test/SemaCXX/linkage-invalid-decl.cpp new file mode 100644 index 00000000000..0a991baf9d7 --- /dev/null +++ b/clang/test/SemaCXX/linkage-invalid-decl.cpp @@ -0,0 +1,9 @@ +// RUN: %clang_cc1 -fsyntax-only -verify %s + +// This invalid declaration used to call infinite recursion in linkage +// calculation for enum as a function argument. +inline foo(A)(enum E; +// expected-error@-1 {{unknown type name 'foo'}} +// expected-error@-2 {{ISO C++ forbids forward references to 'enum' types}} +// expected-error@-3 {{expected ')'}} +// expected-note@-4 {{to match this '('}} |
