blob: 5ed28e955d4e0330f0b58c88ca65125a6ca73444 [file] [log] [blame]
Anna Zaks0353aad2012-09-29 00:20:38 +00001//=- IvarInvalidationChecker.cpp - -*- C++ -------------------------------*-==//
Anna Zaks9802f9f2012-09-26 18:55:16 +00002//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
10// This checker implements annotation driven invalidation checking. If a class
11// contains a method annotated with 'objc_instance_variable_invalidator',
12// - (void) foo
13// __attribute__((annotate("objc_instance_variable_invalidator")));
14// all the "ivalidatable" instance variables of this class should be
15// invalidated. We call an instance variable ivalidatable if it is an object of
Anna Zaks0353aad2012-09-29 00:20:38 +000016// a class which contains an invalidation method. There could be multiple
17// methods annotated with such annotations per class, either one can be used
18// to invalidate the ivar. An ivar or property are considered to be
19// invalidated if they are being assigned 'nil' or an invalidation method has
20// been called on them. An invalidation method should either invalidate all
21// the ivars or call another invalidation method (on self).
Anna Zaks9802f9f2012-09-26 18:55:16 +000022//
Anna Zaksa5096f62013-02-08 23:55:43 +000023// Partial invalidor annotation allows to addess cases when ivars are
24// invalidated by other methods, which might or might not be called from
25// the invalidation method. The checker checks that each invalidation
26// method and all the partial methods cumulatively invalidate all ivars.
27// __attribute__((annotate("objc_instance_variable_invalidator_partial")));
28//
Anna Zaks9802f9f2012-09-26 18:55:16 +000029//===----------------------------------------------------------------------===//
30
31#include "ClangSACheckers.h"
Benjamin Kramerea70eb32012-12-01 15:09:41 +000032#include "clang/AST/Attr.h"
Anna Zaks9802f9f2012-09-26 18:55:16 +000033#include "clang/AST/DeclObjC.h"
34#include "clang/AST/StmtVisitor.h"
Chandler Carruth3a022472012-12-04 09:13:33 +000035#include "clang/StaticAnalyzer/Core/BugReporter/BugReporter.h"
36#include "clang/StaticAnalyzer/Core/Checker.h"
37#include "clang/StaticAnalyzer/Core/PathSensitive/AnalysisManager.h"
Anna Zaks9802f9f2012-09-26 18:55:16 +000038#include "llvm/ADT/DenseMap.h"
Anna Zaks0aeb60d2013-01-10 20:59:51 +000039#include "llvm/ADT/SetVector.h"
Anna Zaks9802f9f2012-09-26 18:55:16 +000040#include "llvm/ADT/SmallString.h"
41
42using namespace clang;
43using namespace ento;
44
45namespace {
Anna Zaks91a5fdf2013-02-08 23:55:47 +000046
47struct ChecksFilter {
48 /// Check for missing invalidation method declarations.
49 DefaultBool check_MissingInvalidationMethod;
50 /// Check that all ivars are invalidated.
51 DefaultBool check_InstanceVariableInvalidation;
52};
53
Anna Zaks7811c3e2013-02-09 01:09:27 +000054class IvarInvalidationCheckerImpl {
Anna Zaks9802f9f2012-09-26 18:55:16 +000055
Anna Zaks0aeb60d2013-01-10 20:59:51 +000056 typedef llvm::SmallSetVector<const ObjCMethodDecl*, 2> MethodSet;
Anna Zaks9802f9f2012-09-26 18:55:16 +000057 typedef llvm::DenseMap<const ObjCMethodDecl*,
58 const ObjCIvarDecl*> MethToIvarMapTy;
59 typedef llvm::DenseMap<const ObjCPropertyDecl*,
60 const ObjCIvarDecl*> PropToIvarMapTy;
Anna Zaksa0c83312012-09-27 21:57:14 +000061 typedef llvm::DenseMap<const ObjCIvarDecl*,
62 const ObjCPropertyDecl*> IvarToPropMapTy;
Anna Zaks9802f9f2012-09-26 18:55:16 +000063
Anna Zaks0353aad2012-09-29 00:20:38 +000064
Anna Zaks0aeb60d2013-01-10 20:59:51 +000065 struct InvalidationInfo {
Anna Zaks0353aad2012-09-29 00:20:38 +000066 /// Has the ivar been invalidated?
67 bool IsInvalidated;
68
69 /// The methods which can be used to invalidate the ivar.
70 MethodSet InvalidationMethods;
71
Anna Zaks0aeb60d2013-01-10 20:59:51 +000072 InvalidationInfo() : IsInvalidated(false) {}
Anna Zaks0353aad2012-09-29 00:20:38 +000073 void addInvalidationMethod(const ObjCMethodDecl *MD) {
74 InvalidationMethods.insert(MD);
75 }
76
77 bool needsInvalidation() const {
78 return !InvalidationMethods.empty();
79 }
80
Anna Zaksa5096f62013-02-08 23:55:43 +000081 bool hasMethod(const ObjCMethodDecl *MD) {
Anna Zaks0353aad2012-09-29 00:20:38 +000082 if (IsInvalidated)
83 return true;
84 for (MethodSet::iterator I = InvalidationMethods.begin(),
85 E = InvalidationMethods.end(); I != E; ++I) {
86 if (*I == MD) {
87 IsInvalidated = true;
88 return true;
89 }
90 }
91 return false;
92 }
Anna Zaks0353aad2012-09-29 00:20:38 +000093 };
94
Anna Zaks0aeb60d2013-01-10 20:59:51 +000095 typedef llvm::DenseMap<const ObjCIvarDecl*, InvalidationInfo> IvarSet;
Anna Zaks0353aad2012-09-29 00:20:38 +000096
Anna Zaks9802f9f2012-09-26 18:55:16 +000097 /// Statement visitor, which walks the method body and flags the ivars
98 /// referenced in it (either directly or via property).
99 class MethodCrawler : public ConstStmtVisitor<MethodCrawler> {
Anna Zaks9802f9f2012-09-26 18:55:16 +0000100 /// The set of Ivars which need to be invalidated.
101 IvarSet &IVars;
102
Anna Zaks0353aad2012-09-29 00:20:38 +0000103 /// Flag is set as the result of a message send to another
104 /// invalidation method.
105 bool &CalledAnotherInvalidationMethod;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000106
Anna Zaks0353aad2012-09-29 00:20:38 +0000107 /// Property setter to ivar mapping.
108 const MethToIvarMapTy &PropertySetterToIvarMap;
109
110 /// Property getter to ivar mapping.
111 const MethToIvarMapTy &PropertyGetterToIvarMap;
112
113 /// Property to ivar mapping.
114 const PropToIvarMapTy &PropertyToIvarMap;
115
116 /// The invalidation method being currently processed.
117 const ObjCMethodDecl *InvalidationMethod;
118
Anna Zaksbfacf172012-10-01 20:33:58 +0000119 ASTContext &Ctx;
120
121 /// Peel off parens, casts, OpaqueValueExpr, and PseudoObjectExpr.
Anna Zaks0353aad2012-09-29 00:20:38 +0000122 const Expr *peel(const Expr *E) const;
123
124 /// Does this expression represent zero: '0'?
125 bool isZero(const Expr *E) const;
126
127 /// Mark the given ivar as invalidated.
128 void markInvalidated(const ObjCIvarDecl *Iv);
129
130 /// Checks if IvarRef refers to the tracked IVar, if yes, marks it as
131 /// invalidated.
132 void checkObjCIvarRefExpr(const ObjCIvarRefExpr *IvarRef);
133
134 /// Checks if ObjCPropertyRefExpr refers to the tracked IVar, if yes, marks
135 /// it as invalidated.
136 void checkObjCPropertyRefExpr(const ObjCPropertyRefExpr *PA);
137
138 /// Checks if ObjCMessageExpr refers to (is a getter for) the tracked IVar,
139 /// if yes, marks it as invalidated.
140 void checkObjCMessageExpr(const ObjCMessageExpr *ME);
141
142 /// Checks if the Expr refers to an ivar, if yes, marks it as invalidated.
143 void check(const Expr *E);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000144
145 public:
Anna Zaks97c7ce32012-10-01 20:34:04 +0000146 MethodCrawler(IvarSet &InIVars,
Anna Zaks0353aad2012-09-29 00:20:38 +0000147 bool &InCalledAnotherInvalidationMethod,
148 const MethToIvarMapTy &InPropertySetterToIvarMap,
149 const MethToIvarMapTy &InPropertyGetterToIvarMap,
Anna Zaksbfacf172012-10-01 20:33:58 +0000150 const PropToIvarMapTy &InPropertyToIvarMap,
151 ASTContext &InCtx)
Anna Zaks97c7ce32012-10-01 20:34:04 +0000152 : IVars(InIVars),
Anna Zaks0353aad2012-09-29 00:20:38 +0000153 CalledAnotherInvalidationMethod(InCalledAnotherInvalidationMethod),
154 PropertySetterToIvarMap(InPropertySetterToIvarMap),
155 PropertyGetterToIvarMap(InPropertyGetterToIvarMap),
156 PropertyToIvarMap(InPropertyToIvarMap),
Anna Zaksbfacf172012-10-01 20:33:58 +0000157 InvalidationMethod(0),
158 Ctx(InCtx) {}
Anna Zaks9802f9f2012-09-26 18:55:16 +0000159
160 void VisitStmt(const Stmt *S) { VisitChildren(S); }
161
Anna Zaks0353aad2012-09-29 00:20:38 +0000162 void VisitBinaryOperator(const BinaryOperator *BO);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000163
164 void VisitObjCMessageExpr(const ObjCMessageExpr *ME);
165
Anna Zaks9802f9f2012-09-26 18:55:16 +0000166 void VisitChildren(const Stmt *S) {
Anna Zaks0353aad2012-09-29 00:20:38 +0000167 for (Stmt::const_child_range I = S->children(); I; ++I) {
Anna Zaks9802f9f2012-09-26 18:55:16 +0000168 if (*I)
Anna Zaks8c0dd362012-09-27 19:45:08 +0000169 this->Visit(*I);
Anna Zaks0353aad2012-09-29 00:20:38 +0000170 if (CalledAnotherInvalidationMethod)
171 return;
172 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000173 }
174 };
175
176 /// Check if the any of the methods inside the interface are annotated with
Anna Zaks0353aad2012-09-29 00:20:38 +0000177 /// the invalidation annotation, update the IvarInfo accordingly.
Anna Zaksa5096f62013-02-08 23:55:43 +0000178 /// \param LookForPartial is set when we are searching for partial
179 /// invalidators.
Anna Zaks0353aad2012-09-29 00:20:38 +0000180 static void containsInvalidationMethod(const ObjCContainerDecl *D,
Anna Zaksa5096f62013-02-08 23:55:43 +0000181 InvalidationInfo &Out,
182 bool LookForPartial);
Anna Zaksa0c83312012-09-27 21:57:14 +0000183
184 /// Check if ivar should be tracked and add to TrackedIvars if positive.
185 /// Returns true if ivar should be tracked.
Anna Zaks640123d2013-01-10 22:44:16 +0000186 static bool trackIvar(const ObjCIvarDecl *Iv, IvarSet &TrackedIvars,
187 const ObjCIvarDecl **FirstIvarDecl);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000188
189 /// Given the property declaration, and the list of tracked ivars, finds
190 /// the ivar backing the property when possible. Returns '0' when no such
191 /// ivar could be found.
192 static const ObjCIvarDecl *findPropertyBackingIvar(
193 const ObjCPropertyDecl *Prop,
194 const ObjCInterfaceDecl *InterfaceD,
Anna Zaks640123d2013-01-10 22:44:16 +0000195 IvarSet &TrackedIvars,
196 const ObjCIvarDecl **FirstIvarDecl);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000197
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000198 /// Print ivar name or the property if the given ivar backs a property.
199 static void printIvar(llvm::raw_svector_ostream &os,
200 const ObjCIvarDecl *IvarDecl,
Anna Zaks470543b2013-02-08 23:55:45 +0000201 const IvarToPropMapTy &IvarToPopertyMap);
202
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000203 void reportNoInvalidationMethod(const ObjCIvarDecl *FirstIvarDecl,
204 const IvarToPropMapTy &IvarToPopertyMap,
205 const ObjCInterfaceDecl *InterfaceD,
206 bool MissingDeclaration) const;
207 void reportIvarNeedsInvalidation(const ObjCIvarDecl *IvarD,
208 const IvarToPropMapTy &IvarToPopertyMap,
209 const ObjCMethodDecl *MethodD) const;
Anna Zaks470543b2013-02-08 23:55:45 +0000210
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000211 AnalysisManager& Mgr;
212 BugReporter &BR;
213 /// Filter on the checks performed.
214 const ChecksFilter &Filter;
Anna Zaksa5096f62013-02-08 23:55:43 +0000215
Anna Zaks9802f9f2012-09-26 18:55:16 +0000216public:
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000217 IvarInvalidationCheckerImpl(AnalysisManager& InMgr,
218 BugReporter &InBR,
219 const ChecksFilter &InFilter) :
220 Mgr (InMgr), BR(InBR), Filter(InFilter) {}
221
222 void visit(const ObjCImplementationDecl *D) const;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000223};
224
Anna Zaksa5096f62013-02-08 23:55:43 +0000225static bool isInvalidationMethod(const ObjCMethodDecl *M, bool LookForPartial) {
Anna Zaks8c0dd362012-09-27 19:45:08 +0000226 for (specific_attr_iterator<AnnotateAttr>
227 AI = M->specific_attr_begin<AnnotateAttr>(),
228 AE = M->specific_attr_end<AnnotateAttr>(); AI != AE; ++AI) {
229 const AnnotateAttr *Ann = *AI;
Anna Zaksa5096f62013-02-08 23:55:43 +0000230 if (!LookForPartial &&
231 Ann->getAnnotation() == "objc_instance_variable_invalidator")
232 return true;
233 if (LookForPartial &&
234 Ann->getAnnotation() == "objc_instance_variable_invalidator_partial")
Anna Zaks8c0dd362012-09-27 19:45:08 +0000235 return true;
236 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000237 return false;
238}
239
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000240void IvarInvalidationCheckerImpl::containsInvalidationMethod(
Anna Zaksa5096f62013-02-08 23:55:43 +0000241 const ObjCContainerDecl *D, InvalidationInfo &OutInfo, bool Partial) {
Anna Zaks9802f9f2012-09-26 18:55:16 +0000242
243 if (!D)
Anna Zaks0353aad2012-09-29 00:20:38 +0000244 return;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000245
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000246 assert(!isa<ObjCImplementationDecl>(D));
247 // TODO: Cache the results.
248
Anna Zaks9802f9f2012-09-26 18:55:16 +0000249 // Check all methods.
250 for (ObjCContainerDecl::method_iterator
251 I = D->meth_begin(),
252 E = D->meth_end(); I != E; ++I) {
253 const ObjCMethodDecl *MDI = *I;
Anna Zaksa5096f62013-02-08 23:55:43 +0000254 if (isInvalidationMethod(MDI, Partial))
Anna Zaks0353aad2012-09-29 00:20:38 +0000255 OutInfo.addInvalidationMethod(
256 cast<ObjCMethodDecl>(MDI->getCanonicalDecl()));
Anna Zaks9802f9f2012-09-26 18:55:16 +0000257 }
258
259 // If interface, check all parent protocols and super.
Anna Zaks2975cf22013-01-11 03:52:37 +0000260 if (const ObjCInterfaceDecl *InterfD = dyn_cast<ObjCInterfaceDecl>(D)) {
261
262 // Visit all protocols.
Anna Zaks9802f9f2012-09-26 18:55:16 +0000263 for (ObjCInterfaceDecl::protocol_iterator
Anna Zaks2975cf22013-01-11 03:52:37 +0000264 I = InterfD->protocol_begin(),
265 E = InterfD->protocol_end(); I != E; ++I) {
Anna Zaksa5096f62013-02-08 23:55:43 +0000266 containsInvalidationMethod((*I)->getDefinition(), OutInfo, Partial);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000267 }
Anna Zaks2975cf22013-01-11 03:52:37 +0000268
269 // Visit all categories in case the invalidation method is declared in
270 // a category.
Douglas Gregor048fbfa2013-01-16 23:00:23 +0000271 for (ObjCInterfaceDecl::visible_extensions_iterator
272 Ext = InterfD->visible_extensions_begin(),
273 ExtEnd = InterfD->visible_extensions_end();
274 Ext != ExtEnd; ++Ext) {
Anna Zaksa5096f62013-02-08 23:55:43 +0000275 containsInvalidationMethod(*Ext, OutInfo, Partial);
Anna Zaks2975cf22013-01-11 03:52:37 +0000276 }
277
Anna Zaksa5096f62013-02-08 23:55:43 +0000278 containsInvalidationMethod(InterfD->getSuperClass(), OutInfo, Partial);
Anna Zaks0353aad2012-09-29 00:20:38 +0000279 return;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000280 }
281
282 // If protocol, check all parent protocols.
283 if (const ObjCProtocolDecl *ProtD = dyn_cast<ObjCProtocolDecl>(D)) {
284 for (ObjCInterfaceDecl::protocol_iterator
285 I = ProtD->protocol_begin(),
286 E = ProtD->protocol_end(); I != E; ++I) {
Anna Zaksa5096f62013-02-08 23:55:43 +0000287 containsInvalidationMethod((*I)->getDefinition(), OutInfo, Partial);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000288 }
Anna Zaks0353aad2012-09-29 00:20:38 +0000289 return;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000290 }
291
Anna Zaks2975cf22013-01-11 03:52:37 +0000292 return;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000293}
294
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000295bool IvarInvalidationCheckerImpl::trackIvar(const ObjCIvarDecl *Iv,
Anna Zaks640123d2013-01-10 22:44:16 +0000296 IvarSet &TrackedIvars,
297 const ObjCIvarDecl **FirstIvarDecl) {
Anna Zaksa0c83312012-09-27 21:57:14 +0000298 QualType IvQTy = Iv->getType();
299 const ObjCObjectPointerType *IvTy = IvQTy->getAs<ObjCObjectPointerType>();
300 if (!IvTy)
301 return false;
302 const ObjCInterfaceDecl *IvInterf = IvTy->getInterfaceDecl();
Anna Zaks0353aad2012-09-29 00:20:38 +0000303
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000304 InvalidationInfo Info;
Anna Zaksa5096f62013-02-08 23:55:43 +0000305 containsInvalidationMethod(IvInterf, Info, /*LookForPartial*/ false);
Anna Zaks0353aad2012-09-29 00:20:38 +0000306 if (Info.needsInvalidation()) {
Anna Zaks640123d2013-01-10 22:44:16 +0000307 const ObjCIvarDecl *I = cast<ObjCIvarDecl>(Iv->getCanonicalDecl());
308 TrackedIvars[I] = Info;
309 if (!*FirstIvarDecl)
310 *FirstIvarDecl = I;
Anna Zaksa0c83312012-09-27 21:57:14 +0000311 return true;
312 }
313 return false;
314}
315
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000316const ObjCIvarDecl *IvarInvalidationCheckerImpl::findPropertyBackingIvar(
Anna Zaks9802f9f2012-09-26 18:55:16 +0000317 const ObjCPropertyDecl *Prop,
318 const ObjCInterfaceDecl *InterfaceD,
Anna Zaks640123d2013-01-10 22:44:16 +0000319 IvarSet &TrackedIvars,
320 const ObjCIvarDecl **FirstIvarDecl) {
Anna Zaks9802f9f2012-09-26 18:55:16 +0000321 const ObjCIvarDecl *IvarD = 0;
322
323 // Lookup for the synthesized case.
324 IvarD = Prop->getPropertyIvarDecl();
Anna Zaks5f376432013-01-07 19:12:56 +0000325 // We only track the ivars/properties that are defined in the current
326 // class (not the parent).
327 if (IvarD && IvarD->getContainingInterface() == InterfaceD) {
Anna Zaksa0c83312012-09-27 21:57:14 +0000328 if (TrackedIvars.count(IvarD)) {
329 return IvarD;
330 }
331 // If the ivar is synthesized we still want to track it.
Anna Zaks640123d2013-01-10 22:44:16 +0000332 if (trackIvar(IvarD, TrackedIvars, FirstIvarDecl))
Anna Zaksa0c83312012-09-27 21:57:14 +0000333 return IvarD;
334 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000335
336 // Lookup IVars named "_PropName"or "PropName" among the tracked Ivars.
337 StringRef PropName = Prop->getIdentifier()->getName();
338 for (IvarSet::const_iterator I = TrackedIvars.begin(),
339 E = TrackedIvars.end(); I != E; ++I) {
340 const ObjCIvarDecl *Iv = I->first;
341 StringRef IvarName = Iv->getName();
342
343 if (IvarName == PropName)
344 return Iv;
345
346 SmallString<128> PropNameWithUnderscore;
347 {
348 llvm::raw_svector_ostream os(PropNameWithUnderscore);
349 os << '_' << PropName;
350 }
351 if (IvarName == PropNameWithUnderscore.str())
352 return Iv;
353 }
354
355 // Note, this is a possible source of false positives. We could look at the
356 // getter implementation to find the ivar when its name is not derived from
357 // the property name.
358 return 0;
359}
360
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000361void IvarInvalidationCheckerImpl::printIvar(llvm::raw_svector_ostream &os,
Anna Zaks470543b2013-02-08 23:55:45 +0000362 const ObjCIvarDecl *IvarDecl,
363 const IvarToPropMapTy &IvarToPopertyMap) {
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000364 if (IvarDecl->getSynthesize()) {
Anna Zaks470543b2013-02-08 23:55:45 +0000365 const ObjCPropertyDecl *PD = IvarToPopertyMap.lookup(IvarDecl);
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000366 assert(PD &&"Do we synthesize ivars for something other than properties?");
367 os << "Property "<< PD->getName() << " ";
368 } else {
369 os << "Instance variable "<< IvarDecl->getName() << " ";
370 }
371}
372
373// Check that the invalidatable interfaces with ivars/properties implement the
374// invalidation methods.
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000375void IvarInvalidationCheckerImpl::
376visit(const ObjCImplementationDecl *ImplD) const {
Anna Zaks9802f9f2012-09-26 18:55:16 +0000377 // Collect all ivars that need cleanup.
378 IvarSet Ivars;
Anna Zaks640123d2013-01-10 22:44:16 +0000379 // Record the first Ivar needing invalidation; used in reporting when only
380 // one ivar is sufficient. Cannot grab the first on the Ivars set to ensure
381 // deterministic output.
382 const ObjCIvarDecl *FirstIvarDecl = 0;
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000383 const ObjCInterfaceDecl *InterfaceD = ImplD->getClassInterface();
Anna Zaksb642fc52012-10-16 19:36:37 +0000384
385 // Collect ivars declared in this class, its extensions and its implementation
386 ObjCInterfaceDecl *IDecl = const_cast<ObjCInterfaceDecl *>(InterfaceD);
387 for (const ObjCIvarDecl *Iv = IDecl->all_declared_ivar_begin(); Iv;
388 Iv= Iv->getNextIvar())
Anna Zaks640123d2013-01-10 22:44:16 +0000389 trackIvar(Iv, Ivars, &FirstIvarDecl);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000390
Anna Zaksa0c83312012-09-27 21:57:14 +0000391 // Construct Property/Property Accessor to Ivar maps to assist checking if an
Anna Zaks9802f9f2012-09-26 18:55:16 +0000392 // ivar which is backing a property has been reset.
Anna Zaks0353aad2012-09-29 00:20:38 +0000393 MethToIvarMapTy PropSetterToIvarMap;
394 MethToIvarMapTy PropGetterToIvarMap;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000395 PropToIvarMapTy PropertyToIvarMap;
Anna Zaksa0c83312012-09-27 21:57:14 +0000396 IvarToPropMapTy IvarToPopertyMap;
Anna Zaks92898a72012-10-18 19:17:57 +0000397
398 ObjCInterfaceDecl::PropertyMap PropMap;
Fariborz Jahanianaedaaa42013-02-14 22:33:34 +0000399 ObjCInterfaceDecl::PropertyDeclOrder PropOrder;
400 InterfaceD->collectPropertiesToImplement(PropMap, PropOrder);
Anna Zaks92898a72012-10-18 19:17:57 +0000401
402 for (ObjCInterfaceDecl::PropertyMap::iterator
403 I = PropMap.begin(), E = PropMap.end(); I != E; ++I) {
404 const ObjCPropertyDecl *PD = I->second;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000405
Anna Zaks640123d2013-01-10 22:44:16 +0000406 const ObjCIvarDecl *ID = findPropertyBackingIvar(PD, InterfaceD, Ivars,
407 &FirstIvarDecl);
Anna Zaks470543b2013-02-08 23:55:45 +0000408 if (!ID)
Anna Zaks9802f9f2012-09-26 18:55:16 +0000409 continue;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000410
411 // Store the mappings.
412 PD = cast<ObjCPropertyDecl>(PD->getCanonicalDecl());
Anna Zaks9802f9f2012-09-26 18:55:16 +0000413 PropertyToIvarMap[PD] = ID;
Anna Zaksa0c83312012-09-27 21:57:14 +0000414 IvarToPopertyMap[ID] = PD;
415
416 // Find the setter and the getter.
417 const ObjCMethodDecl *SetterD = PD->getSetterMethodDecl();
418 if (SetterD) {
419 SetterD = cast<ObjCMethodDecl>(SetterD->getCanonicalDecl());
Anna Zaks0353aad2012-09-29 00:20:38 +0000420 PropSetterToIvarMap[SetterD] = ID;
Anna Zaksa0c83312012-09-27 21:57:14 +0000421 }
422
423 const ObjCMethodDecl *GetterD = PD->getGetterMethodDecl();
424 if (GetterD) {
425 GetterD = cast<ObjCMethodDecl>(GetterD->getCanonicalDecl());
Anna Zaks0353aad2012-09-29 00:20:38 +0000426 PropGetterToIvarMap[GetterD] = ID;
Anna Zaksa0c83312012-09-27 21:57:14 +0000427 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000428 }
429
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000430 // If no ivars need invalidation, there is nothing to check here.
431 if (Ivars.empty())
Anna Zaks0353aad2012-09-29 00:20:38 +0000432 return;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000433
Anna Zaksa5096f62013-02-08 23:55:43 +0000434 // Find all partial invalidation methods.
435 InvalidationInfo PartialInfo;
436 containsInvalidationMethod(InterfaceD, PartialInfo, /*LookForPartial*/ true);
437
438 // Remove ivars invalidated by the partial invalidation methods. They do not
439 // need to be invalidated in the regular invalidation methods.
440 for (MethodSet::iterator
441 I = PartialInfo.InvalidationMethods.begin(),
442 E = PartialInfo.InvalidationMethods.end(); I != E; ++I) {
443 const ObjCMethodDecl *InterfD = *I;
444
445 // Get the corresponding method in the @implementation.
446 const ObjCMethodDecl *D = ImplD->getMethod(InterfD->getSelector(),
447 InterfD->isInstanceMethod());
448 if (D && D->hasBody()) {
449 bool CalledAnotherInvalidationMethod = false;
450 // The MethodCrowler is going to remove the invalidated ivars.
451 MethodCrawler(Ivars,
452 CalledAnotherInvalidationMethod,
453 PropSetterToIvarMap,
454 PropGetterToIvarMap,
455 PropertyToIvarMap,
456 BR.getContext()).VisitStmt(D->getBody());
457 // If another invalidation method was called, trust that full invalidation
458 // has occurred.
459 if (CalledAnotherInvalidationMethod)
460 Ivars.clear();
461 }
462 }
463
464 // If all ivars have been invalidated by partial invalidators, there is
465 // nothing to check here.
466 if (Ivars.empty())
467 return;
468
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000469 // Find all invalidation methods in this @interface declaration and parents.
470 InvalidationInfo Info;
Anna Zaksa5096f62013-02-08 23:55:43 +0000471 containsInvalidationMethod(InterfaceD, Info, /*LookForPartial*/ false);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000472
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000473 // Report an error in case none of the invalidation methods are declared.
Anna Zaks7811c3e2013-02-09 01:09:27 +0000474 if (!Info.needsInvalidation()) {
475 if (Filter.check_MissingInvalidationMethod)
476 reportNoInvalidationMethod(FirstIvarDecl, IvarToPopertyMap, InterfaceD,
477 /*MissingDeclaration*/ true);
478 // If there are no invalidation methods, there is no ivar validation work
479 // to be done.
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000480 return;
481 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000482
Anna Zaks7811c3e2013-02-09 01:09:27 +0000483 // Only check if Ivars are invalidated when InstanceVariableInvalidation
484 // has been requested.
485 if (!Filter.check_InstanceVariableInvalidation)
486 return;
487
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000488 // Check that all ivars are invalidated by the invalidation methods.
489 bool AtImplementationContainsAtLeastOneInvalidationMethod = false;
490 for (MethodSet::iterator I = Info.InvalidationMethods.begin(),
491 E = Info.InvalidationMethods.end(); I != E; ++I) {
492 const ObjCMethodDecl *InterfD = *I;
493
494 // Get the corresponding method in the @implementation.
495 const ObjCMethodDecl *D = ImplD->getMethod(InterfD->getSelector(),
496 InterfD->isInstanceMethod());
497 if (D && D->hasBody()) {
498 AtImplementationContainsAtLeastOneInvalidationMethod = true;
499
500 // Get a copy of ivars needing invalidation.
501 IvarSet IvarsI = Ivars;
502
503 bool CalledAnotherInvalidationMethod = false;
504 MethodCrawler(IvarsI,
505 CalledAnotherInvalidationMethod,
506 PropSetterToIvarMap,
507 PropGetterToIvarMap,
508 PropertyToIvarMap,
509 BR.getContext()).VisitStmt(D->getBody());
510 // If another invalidation method was called, trust that full invalidation
511 // has occurred.
512 if (CalledAnotherInvalidationMethod)
513 continue;
514
515 // Warn on the ivars that were not invalidated by the method.
Anna Zaks470543b2013-02-08 23:55:45 +0000516 for (IvarSet::const_iterator
517 I = IvarsI.begin(), E = IvarsI.end(); I != E; ++I)
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000518 reportIvarNeedsInvalidation(I->first, IvarToPopertyMap, D);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000519 }
520 }
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000521
522 // Report an error in case none of the invalidation methods are implemented.
Anna Zaks7811c3e2013-02-09 01:09:27 +0000523 if (!AtImplementationContainsAtLeastOneInvalidationMethod)
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000524 reportNoInvalidationMethod(FirstIvarDecl, IvarToPopertyMap, InterfaceD,
Anna Zaks470543b2013-02-08 23:55:45 +0000525 /*MissingDeclaration*/ false);
526}
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000527
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000528void IvarInvalidationCheckerImpl::
Anna Zaks470543b2013-02-08 23:55:45 +0000529reportNoInvalidationMethod(const ObjCIvarDecl *FirstIvarDecl,
530 const IvarToPropMapTy &IvarToPopertyMap,
531 const ObjCInterfaceDecl *InterfaceD,
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000532 bool MissingDeclaration) const {
Anna Zaks470543b2013-02-08 23:55:45 +0000533 SmallString<128> sbuf;
534 llvm::raw_svector_ostream os(sbuf);
535 assert(FirstIvarDecl);
536 printIvar(os, FirstIvarDecl, IvarToPopertyMap);
537 os << "needs to be invalidated; ";
538 if (MissingDeclaration)
539 os << "no invalidation method is declared for ";
540 else
541 os << "no invalidation method is defined in the @implementation for ";
542 os << InterfaceD->getName();
543
544 PathDiagnosticLocation IvarDecLocation =
545 PathDiagnosticLocation::createBegin(FirstIvarDecl, BR.getSourceManager());
546
547 BR.EmitBasicReport(FirstIvarDecl, "Incomplete invalidation",
548 categories::CoreFoundationObjectiveC, os.str(),
549 IvarDecLocation);
550}
551
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000552void IvarInvalidationCheckerImpl::
Anna Zaks470543b2013-02-08 23:55:45 +0000553reportIvarNeedsInvalidation(const ObjCIvarDecl *IvarD,
554 const IvarToPropMapTy &IvarToPopertyMap,
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000555 const ObjCMethodDecl *MethodD) const {
Anna Zaks470543b2013-02-08 23:55:45 +0000556 SmallString<128> sbuf;
557 llvm::raw_svector_ostream os(sbuf);
558 printIvar(os, IvarD, IvarToPopertyMap);
559 os << "needs to be invalidated or set to nil";
560 PathDiagnosticLocation MethodDecLocation =
561 PathDiagnosticLocation::createEnd(MethodD->getBody(),
562 BR.getSourceManager(),
563 Mgr.getAnalysisDeclContext(MethodD));
564 BR.EmitBasicReport(MethodD, "Incomplete invalidation",
565 categories::CoreFoundationObjectiveC, os.str(),
566 MethodDecLocation);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000567}
568
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000569void IvarInvalidationCheckerImpl::MethodCrawler::markInvalidated(
Anna Zaks0353aad2012-09-29 00:20:38 +0000570 const ObjCIvarDecl *Iv) {
571 IvarSet::iterator I = IVars.find(Iv);
572 if (I != IVars.end()) {
573 // If InvalidationMethod is present, we are processing the message send and
574 // should ensure we are invalidating with the appropriate method,
575 // otherwise, we are processing setting to 'nil'.
Anna Zaksa5096f62013-02-08 23:55:43 +0000576 if (!InvalidationMethod ||
577 (InvalidationMethod && I->second.hasMethod(InvalidationMethod)))
578 IVars.erase(I);
Anna Zaks0353aad2012-09-29 00:20:38 +0000579 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000580}
581
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000582const Expr *IvarInvalidationCheckerImpl::MethodCrawler::peel(const Expr *E) const {
Anna Zaks0353aad2012-09-29 00:20:38 +0000583 E = E->IgnoreParenCasts();
584 if (const PseudoObjectExpr *POE = dyn_cast<PseudoObjectExpr>(E))
585 E = POE->getSyntacticForm()->IgnoreParenCasts();
586 if (const OpaqueValueExpr *OVE = dyn_cast<OpaqueValueExpr>(E))
587 E = OVE->getSourceExpr()->IgnoreParenCasts();
588 return E;
589}
Anna Zaks9802f9f2012-09-26 18:55:16 +0000590
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000591void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCIvarRefExpr(
Anna Zaks0353aad2012-09-29 00:20:38 +0000592 const ObjCIvarRefExpr *IvarRef) {
593 if (const Decl *D = IvarRef->getDecl())
594 markInvalidated(cast<ObjCIvarDecl>(D->getCanonicalDecl()));
595}
596
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000597void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCMessageExpr(
Anna Zaks9802f9f2012-09-26 18:55:16 +0000598 const ObjCMessageExpr *ME) {
599 const ObjCMethodDecl *MD = ME->getMethodDecl();
600 if (MD) {
601 MD = cast<ObjCMethodDecl>(MD->getCanonicalDecl());
Anna Zaks0353aad2012-09-29 00:20:38 +0000602 MethToIvarMapTy::const_iterator IvI = PropertyGetterToIvarMap.find(MD);
603 if (IvI != PropertyGetterToIvarMap.end())
604 markInvalidated(IvI->second);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000605 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000606}
607
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000608void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCPropertyRefExpr(
Anna Zaks9802f9f2012-09-26 18:55:16 +0000609 const ObjCPropertyRefExpr *PA) {
610
611 if (PA->isExplicitProperty()) {
612 const ObjCPropertyDecl *PD = PA->getExplicitProperty();
613 if (PD) {
614 PD = cast<ObjCPropertyDecl>(PD->getCanonicalDecl());
Anna Zaks0353aad2012-09-29 00:20:38 +0000615 PropToIvarMapTy::const_iterator IvI = PropertyToIvarMap.find(PD);
616 if (IvI != PropertyToIvarMap.end())
617 markInvalidated(IvI->second);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000618 return;
619 }
620 }
621
622 if (PA->isImplicitProperty()) {
623 const ObjCMethodDecl *MD = PA->getImplicitPropertySetter();
624 if (MD) {
625 MD = cast<ObjCMethodDecl>(MD->getCanonicalDecl());
Anna Zaks0353aad2012-09-29 00:20:38 +0000626 MethToIvarMapTy::const_iterator IvI =PropertyGetterToIvarMap.find(MD);
627 if (IvI != PropertyGetterToIvarMap.end())
628 markInvalidated(IvI->second);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000629 return;
630 }
631 }
Anna Zaks0353aad2012-09-29 00:20:38 +0000632}
633
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000634bool IvarInvalidationCheckerImpl::MethodCrawler::isZero(const Expr *E) const {
Anna Zaks0353aad2012-09-29 00:20:38 +0000635 E = peel(E);
Anna Zaks0353aad2012-09-29 00:20:38 +0000636
Anna Zaksbfacf172012-10-01 20:33:58 +0000637 return (E->isNullPointerConstant(Ctx, Expr::NPC_ValueDependentIsNotNull)
638 != Expr::NPCK_NotNull);
Anna Zaks0353aad2012-09-29 00:20:38 +0000639}
640
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000641void IvarInvalidationCheckerImpl::MethodCrawler::check(const Expr *E) {
Anna Zaks0353aad2012-09-29 00:20:38 +0000642 E = peel(E);
643
Anna Zaks0353aad2012-09-29 00:20:38 +0000644 if (const ObjCIvarRefExpr *IvarRef = dyn_cast<ObjCIvarRefExpr>(E)) {
645 checkObjCIvarRefExpr(IvarRef);
646 return;
647 }
648
649 if (const ObjCPropertyRefExpr *PropRef = dyn_cast<ObjCPropertyRefExpr>(E)) {
650 checkObjCPropertyRefExpr(PropRef);
651 return;
652 }
653
654 if (const ObjCMessageExpr *MsgExpr = dyn_cast<ObjCMessageExpr>(E)) {
655 checkObjCMessageExpr(MsgExpr);
656 return;
657 }
658}
659
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000660void IvarInvalidationCheckerImpl::MethodCrawler::VisitBinaryOperator(
Anna Zaks0353aad2012-09-29 00:20:38 +0000661 const BinaryOperator *BO) {
Anna Zaksbfacf172012-10-01 20:33:58 +0000662 VisitStmt(BO);
663
Anna Zaksa96a9ef2013-01-10 23:34:16 +0000664 // Do we assign/compare against zero? If yes, check the variable we are
665 // assigning to.
666 BinaryOperatorKind Opcode = BO->getOpcode();
667 if (Opcode != BO_Assign &&
668 Opcode != BO_EQ &&
669 Opcode != BO_NE)
Anna Zaks0353aad2012-09-29 00:20:38 +0000670 return;
671
Anna Zaksa96a9ef2013-01-10 23:34:16 +0000672 if (isZero(BO->getRHS())) {
673 check(BO->getLHS());
674 return;
675 }
Anna Zaks0353aad2012-09-29 00:20:38 +0000676
Anna Zaksa96a9ef2013-01-10 23:34:16 +0000677 if (Opcode != BO_Assign && isZero(BO->getLHS())) {
678 check(BO->getRHS());
679 return;
680 }
Anna Zaks0353aad2012-09-29 00:20:38 +0000681}
682
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000683void IvarInvalidationCheckerImpl::MethodCrawler::VisitObjCMessageExpr(
Anna Zaks640123d2013-01-10 22:44:16 +0000684 const ObjCMessageExpr *ME) {
Anna Zaks0353aad2012-09-29 00:20:38 +0000685 const ObjCMethodDecl *MD = ME->getMethodDecl();
686 const Expr *Receiver = ME->getInstanceReceiver();
687
688 // Stop if we are calling '[self invalidate]'.
Anna Zaksa5096f62013-02-08 23:55:43 +0000689 if (Receiver && isInvalidationMethod(MD, /*LookForPartial*/ false))
Anna Zaks97c7ce32012-10-01 20:34:04 +0000690 if (Receiver->isObjCSelfExpr()) {
691 CalledAnotherInvalidationMethod = true;
692 return;
Anna Zaks0353aad2012-09-29 00:20:38 +0000693 }
694
695 // Check if we call a setter and set the property to 'nil'.
696 if (MD && (ME->getNumArgs() == 1) && isZero(ME->getArg(0))) {
697 MD = cast<ObjCMethodDecl>(MD->getCanonicalDecl());
698 MethToIvarMapTy::const_iterator IvI = PropertySetterToIvarMap.find(MD);
699 if (IvI != PropertySetterToIvarMap.end()) {
700 markInvalidated(IvI->second);
701 return;
702 }
703 }
704
705 // Check if we call the 'invalidation' routine on the ivar.
706 if (Receiver) {
707 InvalidationMethod = MD;
708 check(Receiver->IgnoreParenCasts());
709 InvalidationMethod = 0;
710 }
711
712 VisitStmt(ME);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000713}
714}
715
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000716// Register the checkers.
717namespace {
718
719class IvarInvalidationChecker :
720 public Checker<check::ASTDecl<ObjCImplementationDecl> > {
721public:
722 ChecksFilter Filter;
723public:
724 void checkASTDecl(const ObjCImplementationDecl *D, AnalysisManager& Mgr,
725 BugReporter &BR) const {
726 IvarInvalidationCheckerImpl Walker(Mgr, BR, Filter);
727 Walker.visit(D);
728 }
729};
Anna Zaks9802f9f2012-09-26 18:55:16 +0000730}
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000731
732#define REGISTER_CHECKER(name) \
733void ento::register##name(CheckerManager &mgr) {\
734 mgr.registerChecker<IvarInvalidationChecker>()->Filter.check_##name = true;\
735}
736
737REGISTER_CHECKER(InstanceVariableInvalidation)
738REGISTER_CHECKER(MissingInvalidationMethod)
739