From 0239df868e2e09f4963bc310c56ea129f51a2288 Mon Sep 17 00:00:00 2001 From: Jordan Rose Date: Thu, 28 Jun 2012 16:39:28 +0000 Subject: [PATCH] Warn on weak properties declared in protocols as well. Previously this caused a crash, since protocols are not interfaces. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@159357 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Sema/SemaExprObjC.cpp | 6 ++++-- test/SemaObjC/weak-receiver-warn.m | 12 ++++++++++++ 2 files changed, 16 insertions(+), 2 deletions(-) diff --git a/lib/Sema/SemaExprObjC.cpp b/lib/Sema/SemaExprObjC.cpp index 598ad0b27a..a3fe7d3bbe 100644 --- a/lib/Sema/SemaExprObjC.cpp +++ b/lib/Sema/SemaExprObjC.cpp @@ -1379,10 +1379,12 @@ static void DiagnoseARCUseOfWeakReceiver(Sema &S, Expr *Receiver) { ObjCMethodDecl *Method = ME->getMethodDecl(); if (Method && Method->isSynthesized()) { Selector Sel = Method->getSelector(); - if (Sel.getNumArgs() == 0) + if (Sel.getNumArgs() == 0) { + const DeclContext *Container = Method->getDeclContext(); PDecl = - S.LookupPropertyDecl(Method->getClassInterface(), + S.LookupPropertyDecl(cast(Container), Sel.getIdentifierInfoForSlot(0)); + } if (PDecl) T = PDecl->getType(); } diff --git a/test/SemaObjC/weak-receiver-warn.m b/test/SemaObjC/weak-receiver-warn.m index 56d9bc10f9..e6f8eaba8c 100644 --- a/test/SemaObjC/weak-receiver-warn.m +++ b/test/SemaObjC/weak-receiver-warn.m @@ -66,3 +66,15 @@ void test0(Test0 *x) { } @end + + +// Weak properties on protocols can be synthesized by an adopting class. +@protocol MyProtocol +@property (weak) id object; // expected-note 2 {{property declared here}} +@end + +void testProtocol(id input) { + [[input object] Meth]; // expected-warning {{weak property may be unpredictably null in ARC mode}} + [input.object Meth]; // expected-warning {{weak property may be unpredictably null in ARC mode}} +} + -- 2.40.0