blob: 1926600b6d6ea2d8c8c9eef2153417decbe4a859 [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;
Alexander Kornienko4aca9b12014-02-11 21:49:21 +000052
53 CheckName checkName_MissingInvalidationMethod;
54 CheckName checkName_InstanceVariableInvalidation;
Anna Zaks91a5fdf2013-02-08 23:55:47 +000055};
56
Anna Zaks7811c3e2013-02-09 01:09:27 +000057class IvarInvalidationCheckerImpl {
Anna Zaks9802f9f2012-09-26 18:55:16 +000058
Anna Zaks0aeb60d2013-01-10 20:59:51 +000059 typedef llvm::SmallSetVector<const ObjCMethodDecl*, 2> MethodSet;
Anna Zaks9802f9f2012-09-26 18:55:16 +000060 typedef llvm::DenseMap<const ObjCMethodDecl*,
61 const ObjCIvarDecl*> MethToIvarMapTy;
62 typedef llvm::DenseMap<const ObjCPropertyDecl*,
63 const ObjCIvarDecl*> PropToIvarMapTy;
Anna Zaksa0c83312012-09-27 21:57:14 +000064 typedef llvm::DenseMap<const ObjCIvarDecl*,
65 const ObjCPropertyDecl*> IvarToPropMapTy;
Anna Zaks9802f9f2012-09-26 18:55:16 +000066
Anna Zaks0353aad2012-09-29 00:20:38 +000067
Anna Zaks0aeb60d2013-01-10 20:59:51 +000068 struct InvalidationInfo {
Anna Zaks0353aad2012-09-29 00:20:38 +000069 /// Has the ivar been invalidated?
70 bool IsInvalidated;
71
72 /// The methods which can be used to invalidate the ivar.
73 MethodSet InvalidationMethods;
74
Anna Zaks0aeb60d2013-01-10 20:59:51 +000075 InvalidationInfo() : IsInvalidated(false) {}
Anna Zaks0353aad2012-09-29 00:20:38 +000076 void addInvalidationMethod(const ObjCMethodDecl *MD) {
77 InvalidationMethods.insert(MD);
78 }
79
80 bool needsInvalidation() const {
81 return !InvalidationMethods.empty();
82 }
83
Anna Zaksa5096f62013-02-08 23:55:43 +000084 bool hasMethod(const ObjCMethodDecl *MD) {
Anna Zaks0353aad2012-09-29 00:20:38 +000085 if (IsInvalidated)
86 return true;
87 for (MethodSet::iterator I = InvalidationMethods.begin(),
88 E = InvalidationMethods.end(); I != E; ++I) {
89 if (*I == MD) {
90 IsInvalidated = true;
91 return true;
92 }
93 }
94 return false;
95 }
Anna Zaks0353aad2012-09-29 00:20:38 +000096 };
97
Anna Zaks0aeb60d2013-01-10 20:59:51 +000098 typedef llvm::DenseMap<const ObjCIvarDecl*, InvalidationInfo> IvarSet;
Anna Zaks0353aad2012-09-29 00:20:38 +000099
Anna Zaks9802f9f2012-09-26 18:55:16 +0000100 /// Statement visitor, which walks the method body and flags the ivars
101 /// referenced in it (either directly or via property).
102 class MethodCrawler : public ConstStmtVisitor<MethodCrawler> {
Anna Zaks9802f9f2012-09-26 18:55:16 +0000103 /// The set of Ivars which need to be invalidated.
104 IvarSet &IVars;
105
Anna Zaks0353aad2012-09-29 00:20:38 +0000106 /// Flag is set as the result of a message send to another
107 /// invalidation method.
108 bool &CalledAnotherInvalidationMethod;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000109
Anna Zaks0353aad2012-09-29 00:20:38 +0000110 /// Property setter to ivar mapping.
111 const MethToIvarMapTy &PropertySetterToIvarMap;
112
113 /// Property getter to ivar mapping.
114 const MethToIvarMapTy &PropertyGetterToIvarMap;
115
116 /// Property to ivar mapping.
117 const PropToIvarMapTy &PropertyToIvarMap;
118
119 /// The invalidation method being currently processed.
120 const ObjCMethodDecl *InvalidationMethod;
121
Anna Zaksbfacf172012-10-01 20:33:58 +0000122 ASTContext &Ctx;
123
124 /// Peel off parens, casts, OpaqueValueExpr, and PseudoObjectExpr.
Anna Zaks0353aad2012-09-29 00:20:38 +0000125 const Expr *peel(const Expr *E) const;
126
127 /// Does this expression represent zero: '0'?
128 bool isZero(const Expr *E) const;
129
130 /// Mark the given ivar as invalidated.
131 void markInvalidated(const ObjCIvarDecl *Iv);
132
133 /// Checks if IvarRef refers to the tracked IVar, if yes, marks it as
134 /// invalidated.
135 void checkObjCIvarRefExpr(const ObjCIvarRefExpr *IvarRef);
136
137 /// Checks if ObjCPropertyRefExpr refers to the tracked IVar, if yes, marks
138 /// it as invalidated.
139 void checkObjCPropertyRefExpr(const ObjCPropertyRefExpr *PA);
140
141 /// Checks if ObjCMessageExpr refers to (is a getter for) the tracked IVar,
142 /// if yes, marks it as invalidated.
143 void checkObjCMessageExpr(const ObjCMessageExpr *ME);
144
145 /// Checks if the Expr refers to an ivar, if yes, marks it as invalidated.
146 void check(const Expr *E);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000147
148 public:
Anna Zaks97c7ce32012-10-01 20:34:04 +0000149 MethodCrawler(IvarSet &InIVars,
Anna Zaks0353aad2012-09-29 00:20:38 +0000150 bool &InCalledAnotherInvalidationMethod,
151 const MethToIvarMapTy &InPropertySetterToIvarMap,
152 const MethToIvarMapTy &InPropertyGetterToIvarMap,
Anna Zaksbfacf172012-10-01 20:33:58 +0000153 const PropToIvarMapTy &InPropertyToIvarMap,
154 ASTContext &InCtx)
Anna Zaks97c7ce32012-10-01 20:34:04 +0000155 : IVars(InIVars),
Anna Zaks0353aad2012-09-29 00:20:38 +0000156 CalledAnotherInvalidationMethod(InCalledAnotherInvalidationMethod),
157 PropertySetterToIvarMap(InPropertySetterToIvarMap),
158 PropertyGetterToIvarMap(InPropertyGetterToIvarMap),
159 PropertyToIvarMap(InPropertyToIvarMap),
Craig Topper0dbb7832014-05-27 02:45:47 +0000160 InvalidationMethod(nullptr),
Anna Zaksbfacf172012-10-01 20:33:58 +0000161 Ctx(InCtx) {}
Anna Zaks9802f9f2012-09-26 18:55:16 +0000162
163 void VisitStmt(const Stmt *S) { VisitChildren(S); }
164
Anna Zaks0353aad2012-09-29 00:20:38 +0000165 void VisitBinaryOperator(const BinaryOperator *BO);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000166
167 void VisitObjCMessageExpr(const ObjCMessageExpr *ME);
168
Anna Zaks9802f9f2012-09-26 18:55:16 +0000169 void VisitChildren(const Stmt *S) {
Anna Zaks0353aad2012-09-29 00:20:38 +0000170 for (Stmt::const_child_range I = S->children(); I; ++I) {
Anna Zaks9802f9f2012-09-26 18:55:16 +0000171 if (*I)
Anna Zaks8c0dd362012-09-27 19:45:08 +0000172 this->Visit(*I);
Anna Zaks0353aad2012-09-29 00:20:38 +0000173 if (CalledAnotherInvalidationMethod)
174 return;
175 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000176 }
177 };
178
179 /// Check if the any of the methods inside the interface are annotated with
Anna Zaks0353aad2012-09-29 00:20:38 +0000180 /// the invalidation annotation, update the IvarInfo accordingly.
Anna Zaksa5096f62013-02-08 23:55:43 +0000181 /// \param LookForPartial is set when we are searching for partial
182 /// invalidators.
Anna Zaks0353aad2012-09-29 00:20:38 +0000183 static void containsInvalidationMethod(const ObjCContainerDecl *D,
Anna Zaksa5096f62013-02-08 23:55:43 +0000184 InvalidationInfo &Out,
185 bool LookForPartial);
Anna Zaksa0c83312012-09-27 21:57:14 +0000186
187 /// Check if ivar should be tracked and add to TrackedIvars if positive.
188 /// Returns true if ivar should be tracked.
Anna Zaks640123d2013-01-10 22:44:16 +0000189 static bool trackIvar(const ObjCIvarDecl *Iv, IvarSet &TrackedIvars,
190 const ObjCIvarDecl **FirstIvarDecl);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000191
192 /// Given the property declaration, and the list of tracked ivars, finds
193 /// the ivar backing the property when possible. Returns '0' when no such
194 /// ivar could be found.
195 static const ObjCIvarDecl *findPropertyBackingIvar(
196 const ObjCPropertyDecl *Prop,
197 const ObjCInterfaceDecl *InterfaceD,
Anna Zaks640123d2013-01-10 22:44:16 +0000198 IvarSet &TrackedIvars,
199 const ObjCIvarDecl **FirstIvarDecl);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000200
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000201 /// Print ivar name or the property if the given ivar backs a property.
202 static void printIvar(llvm::raw_svector_ostream &os,
203 const ObjCIvarDecl *IvarDecl,
Anna Zaks470543b2013-02-08 23:55:45 +0000204 const IvarToPropMapTy &IvarToPopertyMap);
205
Alexander Kornienko4aca9b12014-02-11 21:49:21 +0000206 void reportNoInvalidationMethod(CheckName CheckName,
207 const ObjCIvarDecl *FirstIvarDecl,
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000208 const IvarToPropMapTy &IvarToPopertyMap,
209 const ObjCInterfaceDecl *InterfaceD,
210 bool MissingDeclaration) const;
211 void reportIvarNeedsInvalidation(const ObjCIvarDecl *IvarD,
212 const IvarToPropMapTy &IvarToPopertyMap,
213 const ObjCMethodDecl *MethodD) const;
Anna Zaks470543b2013-02-08 23:55:45 +0000214
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000215 AnalysisManager& Mgr;
216 BugReporter &BR;
217 /// Filter on the checks performed.
218 const ChecksFilter &Filter;
Anna Zaksa5096f62013-02-08 23:55:43 +0000219
Anna Zaks9802f9f2012-09-26 18:55:16 +0000220public:
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000221 IvarInvalidationCheckerImpl(AnalysisManager& InMgr,
222 BugReporter &InBR,
223 const ChecksFilter &InFilter) :
224 Mgr (InMgr), BR(InBR), Filter(InFilter) {}
225
226 void visit(const ObjCImplementationDecl *D) const;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000227};
228
Anna Zaksa5096f62013-02-08 23:55:43 +0000229static bool isInvalidationMethod(const ObjCMethodDecl *M, bool LookForPartial) {
Aaron Ballmanbe22bcb2014-03-10 17:08:28 +0000230 for (const auto *Ann : M->specific_attrs<AnnotateAttr>()) {
Anna Zaksa5096f62013-02-08 23:55:43 +0000231 if (!LookForPartial &&
232 Ann->getAnnotation() == "objc_instance_variable_invalidator")
233 return true;
234 if (LookForPartial &&
235 Ann->getAnnotation() == "objc_instance_variable_invalidator_partial")
Anna Zaks8c0dd362012-09-27 19:45:08 +0000236 return true;
237 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000238 return false;
239}
240
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000241void IvarInvalidationCheckerImpl::containsInvalidationMethod(
Anna Zaksa5096f62013-02-08 23:55:43 +0000242 const ObjCContainerDecl *D, InvalidationInfo &OutInfo, bool Partial) {
Anna Zaks9802f9f2012-09-26 18:55:16 +0000243
244 if (!D)
Anna Zaks0353aad2012-09-29 00:20:38 +0000245 return;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000246
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000247 assert(!isa<ObjCImplementationDecl>(D));
248 // TODO: Cache the results.
249
Anna Zaks9802f9f2012-09-26 18:55:16 +0000250 // Check all methods.
Aaron Ballmanaff18c02014-03-13 19:03:34 +0000251 for (const auto *MDI : D->methods())
252 if (isInvalidationMethod(MDI, Partial))
253 OutInfo.addInvalidationMethod(
254 cast<ObjCMethodDecl>(MDI->getCanonicalDecl()));
Anna Zaks9802f9f2012-09-26 18:55:16 +0000255
256 // If interface, check all parent protocols and super.
Anna Zaks2975cf22013-01-11 03:52:37 +0000257 if (const ObjCInterfaceDecl *InterfD = dyn_cast<ObjCInterfaceDecl>(D)) {
258
259 // Visit all protocols.
Aaron Ballmana49c5062014-03-13 20:29:09 +0000260 for (const auto *I : InterfD->protocols())
261 containsInvalidationMethod(I->getDefinition(), OutInfo, Partial);
Anna Zaks2975cf22013-01-11 03:52:37 +0000262
263 // Visit all categories in case the invalidation method is declared in
264 // a category.
Aaron Ballmanf53d8dd2014-03-13 21:47:07 +0000265 for (const auto *Ext : InterfD->visible_extensions())
266 containsInvalidationMethod(Ext, OutInfo, Partial);
Anna Zaks2975cf22013-01-11 03:52:37 +0000267
Anna Zaksa5096f62013-02-08 23:55:43 +0000268 containsInvalidationMethod(InterfD->getSuperClass(), OutInfo, Partial);
Anna Zaks0353aad2012-09-29 00:20:38 +0000269 return;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000270 }
271
272 // If protocol, check all parent protocols.
273 if (const ObjCProtocolDecl *ProtD = dyn_cast<ObjCProtocolDecl>(D)) {
Aaron Ballman0f6e64d2014-03-13 22:58:06 +0000274 for (const auto *I : ProtD->protocols()) {
275 containsInvalidationMethod(I->getDefinition(), OutInfo, Partial);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000276 }
Anna Zaks0353aad2012-09-29 00:20:38 +0000277 return;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000278 }
279
Anna Zaks2975cf22013-01-11 03:52:37 +0000280 return;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000281}
282
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000283bool IvarInvalidationCheckerImpl::trackIvar(const ObjCIvarDecl *Iv,
Anna Zaks640123d2013-01-10 22:44:16 +0000284 IvarSet &TrackedIvars,
285 const ObjCIvarDecl **FirstIvarDecl) {
Anna Zaksa0c83312012-09-27 21:57:14 +0000286 QualType IvQTy = Iv->getType();
287 const ObjCObjectPointerType *IvTy = IvQTy->getAs<ObjCObjectPointerType>();
288 if (!IvTy)
289 return false;
290 const ObjCInterfaceDecl *IvInterf = IvTy->getInterfaceDecl();
Anna Zaks0353aad2012-09-29 00:20:38 +0000291
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000292 InvalidationInfo Info;
Anna Zaksa5096f62013-02-08 23:55:43 +0000293 containsInvalidationMethod(IvInterf, Info, /*LookForPartial*/ false);
Anna Zaks0353aad2012-09-29 00:20:38 +0000294 if (Info.needsInvalidation()) {
Anna Zaks640123d2013-01-10 22:44:16 +0000295 const ObjCIvarDecl *I = cast<ObjCIvarDecl>(Iv->getCanonicalDecl());
296 TrackedIvars[I] = Info;
297 if (!*FirstIvarDecl)
298 *FirstIvarDecl = I;
Anna Zaksa0c83312012-09-27 21:57:14 +0000299 return true;
300 }
301 return false;
302}
303
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000304const ObjCIvarDecl *IvarInvalidationCheckerImpl::findPropertyBackingIvar(
Anna Zaks9802f9f2012-09-26 18:55:16 +0000305 const ObjCPropertyDecl *Prop,
306 const ObjCInterfaceDecl *InterfaceD,
Anna Zaks640123d2013-01-10 22:44:16 +0000307 IvarSet &TrackedIvars,
308 const ObjCIvarDecl **FirstIvarDecl) {
Craig Topper0dbb7832014-05-27 02:45:47 +0000309 const ObjCIvarDecl *IvarD = nullptr;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000310
311 // Lookup for the synthesized case.
312 IvarD = Prop->getPropertyIvarDecl();
Anna Zaks5f376432013-01-07 19:12:56 +0000313 // We only track the ivars/properties that are defined in the current
314 // class (not the parent).
315 if (IvarD && IvarD->getContainingInterface() == InterfaceD) {
Anna Zaksa0c83312012-09-27 21:57:14 +0000316 if (TrackedIvars.count(IvarD)) {
317 return IvarD;
318 }
319 // If the ivar is synthesized we still want to track it.
Anna Zaks640123d2013-01-10 22:44:16 +0000320 if (trackIvar(IvarD, TrackedIvars, FirstIvarDecl))
Anna Zaksa0c83312012-09-27 21:57:14 +0000321 return IvarD;
322 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000323
324 // Lookup IVars named "_PropName"or "PropName" among the tracked Ivars.
325 StringRef PropName = Prop->getIdentifier()->getName();
326 for (IvarSet::const_iterator I = TrackedIvars.begin(),
327 E = TrackedIvars.end(); I != E; ++I) {
328 const ObjCIvarDecl *Iv = I->first;
329 StringRef IvarName = Iv->getName();
330
331 if (IvarName == PropName)
332 return Iv;
333
334 SmallString<128> PropNameWithUnderscore;
335 {
336 llvm::raw_svector_ostream os(PropNameWithUnderscore);
337 os << '_' << PropName;
338 }
339 if (IvarName == PropNameWithUnderscore.str())
340 return Iv;
341 }
342
343 // Note, this is a possible source of false positives. We could look at the
344 // getter implementation to find the ivar when its name is not derived from
345 // the property name.
Craig Topper0dbb7832014-05-27 02:45:47 +0000346 return nullptr;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000347}
348
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000349void IvarInvalidationCheckerImpl::printIvar(llvm::raw_svector_ostream &os,
Anna Zaks470543b2013-02-08 23:55:45 +0000350 const ObjCIvarDecl *IvarDecl,
351 const IvarToPropMapTy &IvarToPopertyMap) {
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000352 if (IvarDecl->getSynthesize()) {
Anna Zaks470543b2013-02-08 23:55:45 +0000353 const ObjCPropertyDecl *PD = IvarToPopertyMap.lookup(IvarDecl);
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000354 assert(PD &&"Do we synthesize ivars for something other than properties?");
355 os << "Property "<< PD->getName() << " ";
356 } else {
357 os << "Instance variable "<< IvarDecl->getName() << " ";
358 }
359}
360
361// Check that the invalidatable interfaces with ivars/properties implement the
362// invalidation methods.
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000363void IvarInvalidationCheckerImpl::
364visit(const ObjCImplementationDecl *ImplD) const {
Anna Zaks9802f9f2012-09-26 18:55:16 +0000365 // Collect all ivars that need cleanup.
366 IvarSet Ivars;
Anna Zaks640123d2013-01-10 22:44:16 +0000367 // Record the first Ivar needing invalidation; used in reporting when only
368 // one ivar is sufficient. Cannot grab the first on the Ivars set to ensure
369 // deterministic output.
Craig Topper0dbb7832014-05-27 02:45:47 +0000370 const ObjCIvarDecl *FirstIvarDecl = nullptr;
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000371 const ObjCInterfaceDecl *InterfaceD = ImplD->getClassInterface();
Anna Zaksb642fc52012-10-16 19:36:37 +0000372
373 // Collect ivars declared in this class, its extensions and its implementation
374 ObjCInterfaceDecl *IDecl = const_cast<ObjCInterfaceDecl *>(InterfaceD);
375 for (const ObjCIvarDecl *Iv = IDecl->all_declared_ivar_begin(); Iv;
376 Iv= Iv->getNextIvar())
Anna Zaks640123d2013-01-10 22:44:16 +0000377 trackIvar(Iv, Ivars, &FirstIvarDecl);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000378
Anna Zaksa0c83312012-09-27 21:57:14 +0000379 // Construct Property/Property Accessor to Ivar maps to assist checking if an
Anna Zaks9802f9f2012-09-26 18:55:16 +0000380 // ivar which is backing a property has been reset.
Anna Zaks0353aad2012-09-29 00:20:38 +0000381 MethToIvarMapTy PropSetterToIvarMap;
382 MethToIvarMapTy PropGetterToIvarMap;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000383 PropToIvarMapTy PropertyToIvarMap;
Anna Zaksa0c83312012-09-27 21:57:14 +0000384 IvarToPropMapTy IvarToPopertyMap;
Anna Zaks92898a72012-10-18 19:17:57 +0000385
386 ObjCInterfaceDecl::PropertyMap PropMap;
Fariborz Jahanianaedaaa42013-02-14 22:33:34 +0000387 ObjCInterfaceDecl::PropertyDeclOrder PropOrder;
388 InterfaceD->collectPropertiesToImplement(PropMap, PropOrder);
Anna Zaks92898a72012-10-18 19:17:57 +0000389
390 for (ObjCInterfaceDecl::PropertyMap::iterator
391 I = PropMap.begin(), E = PropMap.end(); I != E; ++I) {
392 const ObjCPropertyDecl *PD = I->second;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000393
Anna Zaks640123d2013-01-10 22:44:16 +0000394 const ObjCIvarDecl *ID = findPropertyBackingIvar(PD, InterfaceD, Ivars,
395 &FirstIvarDecl);
Anna Zaks470543b2013-02-08 23:55:45 +0000396 if (!ID)
Anna Zaks9802f9f2012-09-26 18:55:16 +0000397 continue;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000398
399 // Store the mappings.
400 PD = cast<ObjCPropertyDecl>(PD->getCanonicalDecl());
Anna Zaks9802f9f2012-09-26 18:55:16 +0000401 PropertyToIvarMap[PD] = ID;
Anna Zaksa0c83312012-09-27 21:57:14 +0000402 IvarToPopertyMap[ID] = PD;
403
404 // Find the setter and the getter.
405 const ObjCMethodDecl *SetterD = PD->getSetterMethodDecl();
406 if (SetterD) {
407 SetterD = cast<ObjCMethodDecl>(SetterD->getCanonicalDecl());
Anna Zaks0353aad2012-09-29 00:20:38 +0000408 PropSetterToIvarMap[SetterD] = ID;
Anna Zaksa0c83312012-09-27 21:57:14 +0000409 }
410
411 const ObjCMethodDecl *GetterD = PD->getGetterMethodDecl();
412 if (GetterD) {
413 GetterD = cast<ObjCMethodDecl>(GetterD->getCanonicalDecl());
Anna Zaks0353aad2012-09-29 00:20:38 +0000414 PropGetterToIvarMap[GetterD] = ID;
Anna Zaksa0c83312012-09-27 21:57:14 +0000415 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000416 }
417
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000418 // If no ivars need invalidation, there is nothing to check here.
419 if (Ivars.empty())
Anna Zaks0353aad2012-09-29 00:20:38 +0000420 return;
Anna Zaks9802f9f2012-09-26 18:55:16 +0000421
Anna Zaksa5096f62013-02-08 23:55:43 +0000422 // Find all partial invalidation methods.
423 InvalidationInfo PartialInfo;
424 containsInvalidationMethod(InterfaceD, PartialInfo, /*LookForPartial*/ true);
425
426 // Remove ivars invalidated by the partial invalidation methods. They do not
427 // need to be invalidated in the regular invalidation methods.
Anna Zaks7712f382013-04-24 02:49:16 +0000428 bool AtImplementationContainsAtLeastOnePartialInvalidationMethod = false;
Anna Zaksa5096f62013-02-08 23:55:43 +0000429 for (MethodSet::iterator
430 I = PartialInfo.InvalidationMethods.begin(),
431 E = PartialInfo.InvalidationMethods.end(); I != E; ++I) {
432 const ObjCMethodDecl *InterfD = *I;
433
434 // Get the corresponding method in the @implementation.
435 const ObjCMethodDecl *D = ImplD->getMethod(InterfD->getSelector(),
436 InterfD->isInstanceMethod());
437 if (D && D->hasBody()) {
Anna Zaks7712f382013-04-24 02:49:16 +0000438 AtImplementationContainsAtLeastOnePartialInvalidationMethod = true;
439
Anna Zaksa5096f62013-02-08 23:55:43 +0000440 bool CalledAnotherInvalidationMethod = false;
441 // The MethodCrowler is going to remove the invalidated ivars.
442 MethodCrawler(Ivars,
443 CalledAnotherInvalidationMethod,
444 PropSetterToIvarMap,
445 PropGetterToIvarMap,
446 PropertyToIvarMap,
447 BR.getContext()).VisitStmt(D->getBody());
448 // If another invalidation method was called, trust that full invalidation
449 // has occurred.
450 if (CalledAnotherInvalidationMethod)
451 Ivars.clear();
452 }
453 }
454
455 // If all ivars have been invalidated by partial invalidators, there is
456 // nothing to check here.
457 if (Ivars.empty())
458 return;
459
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000460 // Find all invalidation methods in this @interface declaration and parents.
461 InvalidationInfo Info;
Anna Zaksa5096f62013-02-08 23:55:43 +0000462 containsInvalidationMethod(InterfaceD, Info, /*LookForPartial*/ false);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000463
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000464 // Report an error in case none of the invalidation methods are declared.
Anna Zaks7712f382013-04-24 02:49:16 +0000465 if (!Info.needsInvalidation() && !PartialInfo.needsInvalidation()) {
Anna Zaks7811c3e2013-02-09 01:09:27 +0000466 if (Filter.check_MissingInvalidationMethod)
Alexander Kornienko4aca9b12014-02-11 21:49:21 +0000467 reportNoInvalidationMethod(Filter.checkName_MissingInvalidationMethod,
468 FirstIvarDecl, IvarToPopertyMap, InterfaceD,
Anna Zaks7811c3e2013-02-09 01:09:27 +0000469 /*MissingDeclaration*/ true);
470 // If there are no invalidation methods, there is no ivar validation work
471 // to be done.
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000472 return;
473 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000474
Anna Zaks7811c3e2013-02-09 01:09:27 +0000475 // Only check if Ivars are invalidated when InstanceVariableInvalidation
476 // has been requested.
477 if (!Filter.check_InstanceVariableInvalidation)
478 return;
479
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000480 // Check that all ivars are invalidated by the invalidation methods.
481 bool AtImplementationContainsAtLeastOneInvalidationMethod = false;
482 for (MethodSet::iterator I = Info.InvalidationMethods.begin(),
483 E = Info.InvalidationMethods.end(); I != E; ++I) {
484 const ObjCMethodDecl *InterfD = *I;
485
486 // Get the corresponding method in the @implementation.
487 const ObjCMethodDecl *D = ImplD->getMethod(InterfD->getSelector(),
488 InterfD->isInstanceMethod());
489 if (D && D->hasBody()) {
490 AtImplementationContainsAtLeastOneInvalidationMethod = true;
491
492 // Get a copy of ivars needing invalidation.
493 IvarSet IvarsI = Ivars;
494
495 bool CalledAnotherInvalidationMethod = false;
496 MethodCrawler(IvarsI,
497 CalledAnotherInvalidationMethod,
498 PropSetterToIvarMap,
499 PropGetterToIvarMap,
500 PropertyToIvarMap,
501 BR.getContext()).VisitStmt(D->getBody());
502 // If another invalidation method was called, trust that full invalidation
503 // has occurred.
504 if (CalledAnotherInvalidationMethod)
505 continue;
506
507 // Warn on the ivars that were not invalidated by the method.
Anna Zaks470543b2013-02-08 23:55:45 +0000508 for (IvarSet::const_iterator
509 I = IvarsI.begin(), E = IvarsI.end(); I != E; ++I)
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000510 reportIvarNeedsInvalidation(I->first, IvarToPopertyMap, D);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000511 }
512 }
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000513
514 // Report an error in case none of the invalidation methods are implemented.
Anna Zaks7712f382013-04-24 02:49:16 +0000515 if (!AtImplementationContainsAtLeastOneInvalidationMethod) {
516 if (AtImplementationContainsAtLeastOnePartialInvalidationMethod) {
517 // Warn on the ivars that were not invalidated by the prrtial
518 // invalidation methods.
519 for (IvarSet::const_iterator
520 I = Ivars.begin(), E = Ivars.end(); I != E; ++I)
Craig Topper0dbb7832014-05-27 02:45:47 +0000521 reportIvarNeedsInvalidation(I->first, IvarToPopertyMap, nullptr);
Anna Zaks7712f382013-04-24 02:49:16 +0000522 } else {
523 // Otherwise, no invalidation methods were implemented.
Alexander Kornienko4aca9b12014-02-11 21:49:21 +0000524 reportNoInvalidationMethod(Filter.checkName_InstanceVariableInvalidation,
525 FirstIvarDecl, IvarToPopertyMap, InterfaceD,
Anna Zaks7712f382013-04-24 02:49:16 +0000526 /*MissingDeclaration*/ false);
527 }
528 }
Anna Zaks470543b2013-02-08 23:55:45 +0000529}
Anna Zaks0aeb60d2013-01-10 20:59:51 +0000530
Alexander Kornienko4aca9b12014-02-11 21:49:21 +0000531void IvarInvalidationCheckerImpl::reportNoInvalidationMethod(
532 CheckName CheckName, const ObjCIvarDecl *FirstIvarDecl,
533 const IvarToPropMapTy &IvarToPopertyMap,
534 const ObjCInterfaceDecl *InterfaceD, bool MissingDeclaration) const {
Anna Zaks470543b2013-02-08 23:55:45 +0000535 SmallString<128> sbuf;
536 llvm::raw_svector_ostream os(sbuf);
537 assert(FirstIvarDecl);
538 printIvar(os, FirstIvarDecl, IvarToPopertyMap);
539 os << "needs to be invalidated; ";
540 if (MissingDeclaration)
541 os << "no invalidation method is declared for ";
542 else
543 os << "no invalidation method is defined in the @implementation for ";
544 os << InterfaceD->getName();
545
546 PathDiagnosticLocation IvarDecLocation =
547 PathDiagnosticLocation::createBegin(FirstIvarDecl, BR.getSourceManager());
548
Alexander Kornienko4aca9b12014-02-11 21:49:21 +0000549 BR.EmitBasicReport(FirstIvarDecl, CheckName, "Incomplete invalidation",
Anna Zaks470543b2013-02-08 23:55:45 +0000550 categories::CoreFoundationObjectiveC, os.str(),
551 IvarDecLocation);
552}
553
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000554void IvarInvalidationCheckerImpl::
Anna Zaks470543b2013-02-08 23:55:45 +0000555reportIvarNeedsInvalidation(const ObjCIvarDecl *IvarD,
Anna Zaks7712f382013-04-24 02:49:16 +0000556 const IvarToPropMapTy &IvarToPopertyMap,
557 const ObjCMethodDecl *MethodD) const {
Anna Zaks470543b2013-02-08 23:55:45 +0000558 SmallString<128> sbuf;
559 llvm::raw_svector_ostream os(sbuf);
560 printIvar(os, IvarD, IvarToPopertyMap);
561 os << "needs to be invalidated or set to nil";
Anna Zaks7712f382013-04-24 02:49:16 +0000562 if (MethodD) {
563 PathDiagnosticLocation MethodDecLocation =
564 PathDiagnosticLocation::createEnd(MethodD->getBody(),
565 BR.getSourceManager(),
566 Mgr.getAnalysisDeclContext(MethodD));
Alexander Kornienko4aca9b12014-02-11 21:49:21 +0000567 BR.EmitBasicReport(MethodD, Filter.checkName_InstanceVariableInvalidation,
568 "Incomplete invalidation",
Anna Zaks7712f382013-04-24 02:49:16 +0000569 categories::CoreFoundationObjectiveC, os.str(),
570 MethodDecLocation);
571 } else {
Alexander Kornienko4aca9b12014-02-11 21:49:21 +0000572 BR.EmitBasicReport(
573 IvarD, Filter.checkName_InstanceVariableInvalidation,
574 "Incomplete invalidation", categories::CoreFoundationObjectiveC,
575 os.str(),
576 PathDiagnosticLocation::createBegin(IvarD, BR.getSourceManager()));
Anna Zaks7712f382013-04-24 02:49:16 +0000577 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000578}
579
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000580void IvarInvalidationCheckerImpl::MethodCrawler::markInvalidated(
Anna Zaks0353aad2012-09-29 00:20:38 +0000581 const ObjCIvarDecl *Iv) {
582 IvarSet::iterator I = IVars.find(Iv);
583 if (I != IVars.end()) {
584 // If InvalidationMethod is present, we are processing the message send and
585 // should ensure we are invalidating with the appropriate method,
586 // otherwise, we are processing setting to 'nil'.
Anna Zaksa5096f62013-02-08 23:55:43 +0000587 if (!InvalidationMethod ||
588 (InvalidationMethod && I->second.hasMethod(InvalidationMethod)))
589 IVars.erase(I);
Anna Zaks0353aad2012-09-29 00:20:38 +0000590 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000591}
592
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000593const Expr *IvarInvalidationCheckerImpl::MethodCrawler::peel(const Expr *E) const {
Anna Zaks0353aad2012-09-29 00:20:38 +0000594 E = E->IgnoreParenCasts();
595 if (const PseudoObjectExpr *POE = dyn_cast<PseudoObjectExpr>(E))
596 E = POE->getSyntacticForm()->IgnoreParenCasts();
597 if (const OpaqueValueExpr *OVE = dyn_cast<OpaqueValueExpr>(E))
598 E = OVE->getSourceExpr()->IgnoreParenCasts();
599 return E;
600}
Anna Zaks9802f9f2012-09-26 18:55:16 +0000601
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000602void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCIvarRefExpr(
Anna Zaks0353aad2012-09-29 00:20:38 +0000603 const ObjCIvarRefExpr *IvarRef) {
604 if (const Decl *D = IvarRef->getDecl())
605 markInvalidated(cast<ObjCIvarDecl>(D->getCanonicalDecl()));
606}
607
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000608void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCMessageExpr(
Anna Zaks9802f9f2012-09-26 18:55:16 +0000609 const ObjCMessageExpr *ME) {
610 const ObjCMethodDecl *MD = ME->getMethodDecl();
611 if (MD) {
612 MD = cast<ObjCMethodDecl>(MD->getCanonicalDecl());
Anna Zaks0353aad2012-09-29 00:20:38 +0000613 MethToIvarMapTy::const_iterator IvI = PropertyGetterToIvarMap.find(MD);
614 if (IvI != PropertyGetterToIvarMap.end())
615 markInvalidated(IvI->second);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000616 }
Anna Zaks9802f9f2012-09-26 18:55:16 +0000617}
618
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000619void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCPropertyRefExpr(
Anna Zaks9802f9f2012-09-26 18:55:16 +0000620 const ObjCPropertyRefExpr *PA) {
621
622 if (PA->isExplicitProperty()) {
623 const ObjCPropertyDecl *PD = PA->getExplicitProperty();
624 if (PD) {
625 PD = cast<ObjCPropertyDecl>(PD->getCanonicalDecl());
Anna Zaks0353aad2012-09-29 00:20:38 +0000626 PropToIvarMapTy::const_iterator IvI = PropertyToIvarMap.find(PD);
627 if (IvI != PropertyToIvarMap.end())
628 markInvalidated(IvI->second);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000629 return;
630 }
631 }
632
633 if (PA->isImplicitProperty()) {
634 const ObjCMethodDecl *MD = PA->getImplicitPropertySetter();
635 if (MD) {
636 MD = cast<ObjCMethodDecl>(MD->getCanonicalDecl());
Anna Zaks0353aad2012-09-29 00:20:38 +0000637 MethToIvarMapTy::const_iterator IvI =PropertyGetterToIvarMap.find(MD);
638 if (IvI != PropertyGetterToIvarMap.end())
639 markInvalidated(IvI->second);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000640 return;
641 }
642 }
Anna Zaks0353aad2012-09-29 00:20:38 +0000643}
644
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000645bool IvarInvalidationCheckerImpl::MethodCrawler::isZero(const Expr *E) const {
Anna Zaks0353aad2012-09-29 00:20:38 +0000646 E = peel(E);
Anna Zaks0353aad2012-09-29 00:20:38 +0000647
Anna Zaksbfacf172012-10-01 20:33:58 +0000648 return (E->isNullPointerConstant(Ctx, Expr::NPC_ValueDependentIsNotNull)
649 != Expr::NPCK_NotNull);
Anna Zaks0353aad2012-09-29 00:20:38 +0000650}
651
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000652void IvarInvalidationCheckerImpl::MethodCrawler::check(const Expr *E) {
Anna Zaks0353aad2012-09-29 00:20:38 +0000653 E = peel(E);
654
Anna Zaks0353aad2012-09-29 00:20:38 +0000655 if (const ObjCIvarRefExpr *IvarRef = dyn_cast<ObjCIvarRefExpr>(E)) {
656 checkObjCIvarRefExpr(IvarRef);
657 return;
658 }
659
660 if (const ObjCPropertyRefExpr *PropRef = dyn_cast<ObjCPropertyRefExpr>(E)) {
661 checkObjCPropertyRefExpr(PropRef);
662 return;
663 }
664
665 if (const ObjCMessageExpr *MsgExpr = dyn_cast<ObjCMessageExpr>(E)) {
666 checkObjCMessageExpr(MsgExpr);
667 return;
668 }
669}
670
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000671void IvarInvalidationCheckerImpl::MethodCrawler::VisitBinaryOperator(
Anna Zaks0353aad2012-09-29 00:20:38 +0000672 const BinaryOperator *BO) {
Anna Zaksbfacf172012-10-01 20:33:58 +0000673 VisitStmt(BO);
674
Anna Zaksa96a9ef2013-01-10 23:34:16 +0000675 // Do we assign/compare against zero? If yes, check the variable we are
676 // assigning to.
677 BinaryOperatorKind Opcode = BO->getOpcode();
678 if (Opcode != BO_Assign &&
679 Opcode != BO_EQ &&
680 Opcode != BO_NE)
Anna Zaks0353aad2012-09-29 00:20:38 +0000681 return;
682
Anna Zaksa96a9ef2013-01-10 23:34:16 +0000683 if (isZero(BO->getRHS())) {
684 check(BO->getLHS());
685 return;
686 }
Anna Zaks0353aad2012-09-29 00:20:38 +0000687
Anna Zaksa96a9ef2013-01-10 23:34:16 +0000688 if (Opcode != BO_Assign && isZero(BO->getLHS())) {
689 check(BO->getRHS());
690 return;
691 }
Anna Zaks0353aad2012-09-29 00:20:38 +0000692}
693
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000694void IvarInvalidationCheckerImpl::MethodCrawler::VisitObjCMessageExpr(
Anna Zaks640123d2013-01-10 22:44:16 +0000695 const ObjCMessageExpr *ME) {
Anna Zaks0353aad2012-09-29 00:20:38 +0000696 const ObjCMethodDecl *MD = ME->getMethodDecl();
697 const Expr *Receiver = ME->getInstanceReceiver();
698
699 // Stop if we are calling '[self invalidate]'.
Anna Zaksa5096f62013-02-08 23:55:43 +0000700 if (Receiver && isInvalidationMethod(MD, /*LookForPartial*/ false))
Anna Zaks97c7ce32012-10-01 20:34:04 +0000701 if (Receiver->isObjCSelfExpr()) {
702 CalledAnotherInvalidationMethod = true;
703 return;
Anna Zaks0353aad2012-09-29 00:20:38 +0000704 }
705
706 // Check if we call a setter and set the property to 'nil'.
707 if (MD && (ME->getNumArgs() == 1) && isZero(ME->getArg(0))) {
708 MD = cast<ObjCMethodDecl>(MD->getCanonicalDecl());
709 MethToIvarMapTy::const_iterator IvI = PropertySetterToIvarMap.find(MD);
710 if (IvI != PropertySetterToIvarMap.end()) {
711 markInvalidated(IvI->second);
712 return;
713 }
714 }
715
716 // Check if we call the 'invalidation' routine on the ivar.
717 if (Receiver) {
718 InvalidationMethod = MD;
719 check(Receiver->IgnoreParenCasts());
Craig Topper0dbb7832014-05-27 02:45:47 +0000720 InvalidationMethod = nullptr;
Anna Zaks0353aad2012-09-29 00:20:38 +0000721 }
722
723 VisitStmt(ME);
Anna Zaks9802f9f2012-09-26 18:55:16 +0000724}
725}
726
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000727// Register the checkers.
728namespace {
729
730class IvarInvalidationChecker :
731 public Checker<check::ASTDecl<ObjCImplementationDecl> > {
732public:
733 ChecksFilter Filter;
734public:
735 void checkASTDecl(const ObjCImplementationDecl *D, AnalysisManager& Mgr,
736 BugReporter &BR) const {
737 IvarInvalidationCheckerImpl Walker(Mgr, BR, Filter);
738 Walker.visit(D);
739 }
740};
Anna Zaks9802f9f2012-09-26 18:55:16 +0000741}
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000742
Alexander Kornienko4aca9b12014-02-11 21:49:21 +0000743#define REGISTER_CHECKER(name) \
744 void ento::register##name(CheckerManager &mgr) { \
745 IvarInvalidationChecker *checker = \
746 mgr.registerChecker<IvarInvalidationChecker>(); \
747 checker->Filter.check_##name = true; \
748 checker->Filter.checkName_##name = mgr.getCurrentCheckName(); \
749 }
Anna Zaks91a5fdf2013-02-08 23:55:47 +0000750
751REGISTER_CHECKER(InstanceVariableInvalidation)
752REGISTER_CHECKER(MissingInvalidationMethod)
753