[PATCH] D16065: Fix infinite recursion for invalid declaration

[Dmitry Polukhin via cfe-commits]

DmitryPolukhin created this revision.
DmitryPolukhin added a reviewer: rsmith.
DmitryPolukhin added subscribers: cfe-commits, kcc.

Fix for a case found by fuzzing PR23057 (comment #25 https://llvm.org/bugs/show_bug.cgi?id=23057#c25).

http://reviews.llvm.org/D16065

Files:
  lib/AST/Decl.cpp
  test/SemaCXX/linkage-invalid-decl.cpp

Index: test/SemaCXX/linkage-invalid-decl.cpp
===================================================================
--- /dev/null
+++ 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 at -1 {{unknown type name 'foo'}}
+// expected-error at -2 {{ISO C++ forbids forward references to 'enum' types}}
+// expected-error at -3 {{expected ')'}}
+// expected-note at -4 {{to match this '('}}
Index: lib/AST/Decl.cpp
===================================================================
--- lib/AST/Decl.cpp
+++ lib/AST/Decl.cpp
@@ -1184,7 +1184,7 @@
     return LinkageInfo::none();
 
   const Decl *OuterD = getOutermostFuncOrBlockContext(D);
-  if (!OuterD)
+  if (!OuterD || OuterD->isInvalidDecl())
     return LinkageInfo::none();
 
   LinkageInfo LV;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16065.44499.patch
Type: text/x-patch
Size: 948 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160111/79b35e71/attachment.bin>