[PATCH] D15463: [Objective-c] Fix a crash

[Akira Hatanaka via cfe-commits]

ahatanak created this revision.
ahatanak added a subscriber: cfe-commits.

This patch fixes a crash that occurs when __kindof is incorrectly used in the type parameter list of an interface. The crash occurs because ObjCTypeParamList::back() is called in checkTypeParamListConsistency on an empty list:

00762       diagLoc = S.getLocForEndOfToken(newTypeParams->back()->getLocEnd());

http://reviews.llvm.org/D15463

Files:
  lib/Parse/ParseObjc.cpp
  test/SemaObjC/kindof.m

Index: test/SemaObjC/kindof.m
===================================================================
--- test/SemaObjC/kindof.m
+++ test/SemaObjC/kindof.m
@@ -302,3 +302,13 @@
   void processCopyable(__typeof(getSomeCopyable()) string);
   processCopyable(0); // expected-warning{{null passed to a callee that requires a non-null argument}}
 }
+
+// __kinddof cannot be used in parameter list. 
+ at interface Array1<T> : NSObject
+ at end
+
+ at interface I1 : NSObject
+ at end
+
+ at interface Array1<__kindof I1*>(extensions) // // expected-error{{expected type parameter name}}
+ at end
Index: lib/Parse/ParseObjc.cpp
===================================================================
--- lib/Parse/ParseObjc.cpp
+++ lib/Parse/ParseObjc.cpp
@@ -603,7 +603,7 @@
   // whether there are any protocol references.
   lAngleLoc = SourceLocation();
   rAngleLoc = SourceLocation();
-  return list;
+  return invalid ? nullptr : list;
 }
 
 /// Parse an objc-type-parameter-list.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15463.42560.patch
Type: text/x-patch
Size: 958 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20151211/8d1f3c89/attachment.bin>