1 //=- IvarInvalidationChecker.cpp - -*- C++ -------------------------------*-==//
2 //
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
16 // 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).
22 //
23 // 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 //
29 //===----------------------------------------------------------------------===//
30
31 #include "ClangSACheckers.h"
32 #include "clang/AST/Attr.h"
33 #include "clang/AST/DeclObjC.h"
34 #include "clang/AST/StmtVisitor.h"
35 #include "clang/StaticAnalyzer/Core/BugReporter/BugReporter.h"
36 #include "clang/StaticAnalyzer/Core/Checker.h"
37 #include "clang/StaticAnalyzer/Core/PathSensitive/AnalysisManager.h"
38 #include "llvm/ADT/DenseMap.h"
39 #include "llvm/ADT/SetVector.h"
40 #include "llvm/ADT/SmallString.h"
41
42 using namespace clang;
43 using namespace ento;
44
45 namespace {
46
47 struct 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 CheckName checkName_MissingInvalidationMethod;
54 CheckName checkName_InstanceVariableInvalidation;
55 };
56
57 class IvarInvalidationCheckerImpl {
58
59 typedef llvm::SmallSetVector<const ObjCMethodDecl*, 2> MethodSet;
60 typedef llvm::DenseMap<const ObjCMethodDecl*,
61 const ObjCIvarDecl*> MethToIvarMapTy;
62 typedef llvm::DenseMap<const ObjCPropertyDecl*,
63 const ObjCIvarDecl*> PropToIvarMapTy;
64 typedef llvm::DenseMap<const ObjCIvarDecl*,
65 const ObjCPropertyDecl*> IvarToPropMapTy;
66
67
68 struct InvalidationInfo {
69 /// Has the ivar been invalidated?
70 bool IsInvalidated;
71
72 /// The methods which can be used to invalidate the ivar.
73 MethodSet InvalidationMethods;
74
InvalidationInfo__anon197013990111::IvarInvalidationCheckerImpl::InvalidationInfo75 InvalidationInfo() : IsInvalidated(false) {}
addInvalidationMethod__anon197013990111::IvarInvalidationCheckerImpl::InvalidationInfo76 void addInvalidationMethod(const ObjCMethodDecl *MD) {
77 InvalidationMethods.insert(MD);
78 }
79
needsInvalidation__anon197013990111::IvarInvalidationCheckerImpl::InvalidationInfo80 bool needsInvalidation() const {
81 return !InvalidationMethods.empty();
82 }
83
hasMethod__anon197013990111::IvarInvalidationCheckerImpl::InvalidationInfo84 bool hasMethod(const ObjCMethodDecl *MD) {
85 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 }
96 };
97
98 typedef llvm::DenseMap<const ObjCIvarDecl*, InvalidationInfo> IvarSet;
99
100 /// 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> {
103 /// The set of Ivars which need to be invalidated.
104 IvarSet &IVars;
105
106 /// Flag is set as the result of a message send to another
107 /// invalidation method.
108 bool &CalledAnotherInvalidationMethod;
109
110 /// 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
122 ASTContext &Ctx;
123
124 /// Peel off parens, casts, OpaqueValueExpr, and PseudoObjectExpr.
125 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);
147
148 public:
MethodCrawler(IvarSet & InIVars,bool & InCalledAnotherInvalidationMethod,const MethToIvarMapTy & InPropertySetterToIvarMap,const MethToIvarMapTy & InPropertyGetterToIvarMap,const PropToIvarMapTy & InPropertyToIvarMap,ASTContext & InCtx)149 MethodCrawler(IvarSet &InIVars,
150 bool &InCalledAnotherInvalidationMethod,
151 const MethToIvarMapTy &InPropertySetterToIvarMap,
152 const MethToIvarMapTy &InPropertyGetterToIvarMap,
153 const PropToIvarMapTy &InPropertyToIvarMap,
154 ASTContext &InCtx)
155 : IVars(InIVars),
156 CalledAnotherInvalidationMethod(InCalledAnotherInvalidationMethod),
157 PropertySetterToIvarMap(InPropertySetterToIvarMap),
158 PropertyGetterToIvarMap(InPropertyGetterToIvarMap),
159 PropertyToIvarMap(InPropertyToIvarMap),
160 InvalidationMethod(nullptr),
161 Ctx(InCtx) {}
162
VisitStmt(const Stmt * S)163 void VisitStmt(const Stmt *S) { VisitChildren(S); }
164
165 void VisitBinaryOperator(const BinaryOperator *BO);
166
167 void VisitObjCMessageExpr(const ObjCMessageExpr *ME);
168
VisitChildren(const Stmt * S)169 void VisitChildren(const Stmt *S) {
170 for (const Stmt *Child : S->children()) {
171 if (Child)
172 this->Visit(Child);
173 if (CalledAnotherInvalidationMethod)
174 return;
175 }
176 }
177 };
178
179 /// Check if the any of the methods inside the interface are annotated with
180 /// the invalidation annotation, update the IvarInfo accordingly.
181 /// \param LookForPartial is set when we are searching for partial
182 /// invalidators.
183 static void containsInvalidationMethod(const ObjCContainerDecl *D,
184 InvalidationInfo &Out,
185 bool LookForPartial);
186
187 /// Check if ivar should be tracked and add to TrackedIvars if positive.
188 /// Returns true if ivar should be tracked.
189 static bool trackIvar(const ObjCIvarDecl *Iv, IvarSet &TrackedIvars,
190 const ObjCIvarDecl **FirstIvarDecl);
191
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,
198 IvarSet &TrackedIvars,
199 const ObjCIvarDecl **FirstIvarDecl);
200
201 /// 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,
204 const IvarToPropMapTy &IvarToPopertyMap);
205
206 void reportNoInvalidationMethod(CheckName CheckName,
207 const ObjCIvarDecl *FirstIvarDecl,
208 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;
214
215 AnalysisManager& Mgr;
216 BugReporter &BR;
217 /// Filter on the checks performed.
218 const ChecksFilter &Filter;
219
220 public:
IvarInvalidationCheckerImpl(AnalysisManager & InMgr,BugReporter & InBR,const ChecksFilter & InFilter)221 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;
227 };
228
isInvalidationMethod(const ObjCMethodDecl * M,bool LookForPartial)229 static bool isInvalidationMethod(const ObjCMethodDecl *M, bool LookForPartial) {
230 for (const auto *Ann : M->specific_attrs<AnnotateAttr>()) {
231 if (!LookForPartial &&
232 Ann->getAnnotation() == "objc_instance_variable_invalidator")
233 return true;
234 if (LookForPartial &&
235 Ann->getAnnotation() == "objc_instance_variable_invalidator_partial")
236 return true;
237 }
238 return false;
239 }
240
containsInvalidationMethod(const ObjCContainerDecl * D,InvalidationInfo & OutInfo,bool Partial)241 void IvarInvalidationCheckerImpl::containsInvalidationMethod(
242 const ObjCContainerDecl *D, InvalidationInfo &OutInfo, bool Partial) {
243
244 if (!D)
245 return;
246
247 assert(!isa<ObjCImplementationDecl>(D));
248 // TODO: Cache the results.
249
250 // Check all methods.
251 for (const auto *MDI : D->methods())
252 if (isInvalidationMethod(MDI, Partial))
253 OutInfo.addInvalidationMethod(
254 cast<ObjCMethodDecl>(MDI->getCanonicalDecl()));
255
256 // If interface, check all parent protocols and super.
257 if (const ObjCInterfaceDecl *InterfD = dyn_cast<ObjCInterfaceDecl>(D)) {
258
259 // Visit all protocols.
260 for (const auto *I : InterfD->protocols())
261 containsInvalidationMethod(I->getDefinition(), OutInfo, Partial);
262
263 // Visit all categories in case the invalidation method is declared in
264 // a category.
265 for (const auto *Ext : InterfD->visible_extensions())
266 containsInvalidationMethod(Ext, OutInfo, Partial);
267
268 containsInvalidationMethod(InterfD->getSuperClass(), OutInfo, Partial);
269 return;
270 }
271
272 // If protocol, check all parent protocols.
273 if (const ObjCProtocolDecl *ProtD = dyn_cast<ObjCProtocolDecl>(D)) {
274 for (const auto *I : ProtD->protocols()) {
275 containsInvalidationMethod(I->getDefinition(), OutInfo, Partial);
276 }
277 return;
278 }
279
280 return;
281 }
282
trackIvar(const ObjCIvarDecl * Iv,IvarSet & TrackedIvars,const ObjCIvarDecl ** FirstIvarDecl)283 bool IvarInvalidationCheckerImpl::trackIvar(const ObjCIvarDecl *Iv,
284 IvarSet &TrackedIvars,
285 const ObjCIvarDecl **FirstIvarDecl) {
286 QualType IvQTy = Iv->getType();
287 const ObjCObjectPointerType *IvTy = IvQTy->getAs<ObjCObjectPointerType>();
288 if (!IvTy)
289 return false;
290 const ObjCInterfaceDecl *IvInterf = IvTy->getInterfaceDecl();
291
292 InvalidationInfo Info;
293 containsInvalidationMethod(IvInterf, Info, /*LookForPartial*/ false);
294 if (Info.needsInvalidation()) {
295 const ObjCIvarDecl *I = cast<ObjCIvarDecl>(Iv->getCanonicalDecl());
296 TrackedIvars[I] = Info;
297 if (!*FirstIvarDecl)
298 *FirstIvarDecl = I;
299 return true;
300 }
301 return false;
302 }
303
findPropertyBackingIvar(const ObjCPropertyDecl * Prop,const ObjCInterfaceDecl * InterfaceD,IvarSet & TrackedIvars,const ObjCIvarDecl ** FirstIvarDecl)304 const ObjCIvarDecl *IvarInvalidationCheckerImpl::findPropertyBackingIvar(
305 const ObjCPropertyDecl *Prop,
306 const ObjCInterfaceDecl *InterfaceD,
307 IvarSet &TrackedIvars,
308 const ObjCIvarDecl **FirstIvarDecl) {
309 const ObjCIvarDecl *IvarD = nullptr;
310
311 // Lookup for the synthesized case.
312 IvarD = Prop->getPropertyIvarDecl();
313 // We only track the ivars/properties that are defined in the current
314 // class (not the parent).
315 if (IvarD && IvarD->getContainingInterface() == InterfaceD) {
316 if (TrackedIvars.count(IvarD)) {
317 return IvarD;
318 }
319 // If the ivar is synthesized we still want to track it.
320 if (trackIvar(IvarD, TrackedIvars, FirstIvarDecl))
321 return IvarD;
322 }
323
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)
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.
346 return nullptr;
347 }
348
printIvar(llvm::raw_svector_ostream & os,const ObjCIvarDecl * IvarDecl,const IvarToPropMapTy & IvarToPopertyMap)349 void IvarInvalidationCheckerImpl::printIvar(llvm::raw_svector_ostream &os,
350 const ObjCIvarDecl *IvarDecl,
351 const IvarToPropMapTy &IvarToPopertyMap) {
352 if (IvarDecl->getSynthesize()) {
353 const ObjCPropertyDecl *PD = IvarToPopertyMap.lookup(IvarDecl);
354 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.
363 void IvarInvalidationCheckerImpl::
visit(const ObjCImplementationDecl * ImplD) const364 visit(const ObjCImplementationDecl *ImplD) const {
365 // Collect all ivars that need cleanup.
366 IvarSet Ivars;
367 // 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.
370 const ObjCIvarDecl *FirstIvarDecl = nullptr;
371 const ObjCInterfaceDecl *InterfaceD = ImplD->getClassInterface();
372
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())
377 trackIvar(Iv, Ivars, &FirstIvarDecl);
378
379 // Construct Property/Property Accessor to Ivar maps to assist checking if an
380 // ivar which is backing a property has been reset.
381 MethToIvarMapTy PropSetterToIvarMap;
382 MethToIvarMapTy PropGetterToIvarMap;
383 PropToIvarMapTy PropertyToIvarMap;
384 IvarToPropMapTy IvarToPopertyMap;
385
386 ObjCInterfaceDecl::PropertyMap PropMap;
387 ObjCInterfaceDecl::PropertyDeclOrder PropOrder;
388 InterfaceD->collectPropertiesToImplement(PropMap, PropOrder);
389
390 for (ObjCInterfaceDecl::PropertyMap::iterator
391 I = PropMap.begin(), E = PropMap.end(); I != E; ++I) {
392 const ObjCPropertyDecl *PD = I->second;
393
394 const ObjCIvarDecl *ID = findPropertyBackingIvar(PD, InterfaceD, Ivars,
395 &FirstIvarDecl);
396 if (!ID)
397 continue;
398
399 // Store the mappings.
400 PD = cast<ObjCPropertyDecl>(PD->getCanonicalDecl());
401 PropertyToIvarMap[PD] = ID;
402 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());
408 PropSetterToIvarMap[SetterD] = ID;
409 }
410
411 const ObjCMethodDecl *GetterD = PD->getGetterMethodDecl();
412 if (GetterD) {
413 GetterD = cast<ObjCMethodDecl>(GetterD->getCanonicalDecl());
414 PropGetterToIvarMap[GetterD] = ID;
415 }
416 }
417
418 // If no ivars need invalidation, there is nothing to check here.
419 if (Ivars.empty())
420 return;
421
422 // 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.
428 bool AtImplementationContainsAtLeastOnePartialInvalidationMethod = false;
429 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()) {
438 AtImplementationContainsAtLeastOnePartialInvalidationMethod = true;
439
440 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
460 // Find all invalidation methods in this @interface declaration and parents.
461 InvalidationInfo Info;
462 containsInvalidationMethod(InterfaceD, Info, /*LookForPartial*/ false);
463
464 // Report an error in case none of the invalidation methods are declared.
465 if (!Info.needsInvalidation() && !PartialInfo.needsInvalidation()) {
466 if (Filter.check_MissingInvalidationMethod)
467 reportNoInvalidationMethod(Filter.checkName_MissingInvalidationMethod,
468 FirstIvarDecl, IvarToPopertyMap, InterfaceD,
469 /*MissingDeclaration*/ true);
470 // If there are no invalidation methods, there is no ivar validation work
471 // to be done.
472 return;
473 }
474
475 // Only check if Ivars are invalidated when InstanceVariableInvalidation
476 // has been requested.
477 if (!Filter.check_InstanceVariableInvalidation)
478 return;
479
480 // 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.
508 for (IvarSet::const_iterator
509 I = IvarsI.begin(), E = IvarsI.end(); I != E; ++I)
510 reportIvarNeedsInvalidation(I->first, IvarToPopertyMap, D);
511 }
512 }
513
514 // Report an error in case none of the invalidation methods are implemented.
515 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)
521 reportIvarNeedsInvalidation(I->first, IvarToPopertyMap, nullptr);
522 } else {
523 // Otherwise, no invalidation methods were implemented.
524 reportNoInvalidationMethod(Filter.checkName_InstanceVariableInvalidation,
525 FirstIvarDecl, IvarToPopertyMap, InterfaceD,
526 /*MissingDeclaration*/ false);
527 }
528 }
529 }
530
reportNoInvalidationMethod(CheckName CheckName,const ObjCIvarDecl * FirstIvarDecl,const IvarToPropMapTy & IvarToPopertyMap,const ObjCInterfaceDecl * InterfaceD,bool MissingDeclaration) const531 void IvarInvalidationCheckerImpl::reportNoInvalidationMethod(
532 CheckName CheckName, const ObjCIvarDecl *FirstIvarDecl,
533 const IvarToPropMapTy &IvarToPopertyMap,
534 const ObjCInterfaceDecl *InterfaceD, bool MissingDeclaration) const {
535 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
549 BR.EmitBasicReport(FirstIvarDecl, CheckName, "Incomplete invalidation",
550 categories::CoreFoundationObjectiveC, os.str(),
551 IvarDecLocation);
552 }
553
554 void IvarInvalidationCheckerImpl::
reportIvarNeedsInvalidation(const ObjCIvarDecl * IvarD,const IvarToPropMapTy & IvarToPopertyMap,const ObjCMethodDecl * MethodD) const555 reportIvarNeedsInvalidation(const ObjCIvarDecl *IvarD,
556 const IvarToPropMapTy &IvarToPopertyMap,
557 const ObjCMethodDecl *MethodD) const {
558 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";
562 if (MethodD) {
563 PathDiagnosticLocation MethodDecLocation =
564 PathDiagnosticLocation::createEnd(MethodD->getBody(),
565 BR.getSourceManager(),
566 Mgr.getAnalysisDeclContext(MethodD));
567 BR.EmitBasicReport(MethodD, Filter.checkName_InstanceVariableInvalidation,
568 "Incomplete invalidation",
569 categories::CoreFoundationObjectiveC, os.str(),
570 MethodDecLocation);
571 } else {
572 BR.EmitBasicReport(
573 IvarD, Filter.checkName_InstanceVariableInvalidation,
574 "Incomplete invalidation", categories::CoreFoundationObjectiveC,
575 os.str(),
576 PathDiagnosticLocation::createBegin(IvarD, BR.getSourceManager()));
577 }
578 }
579
markInvalidated(const ObjCIvarDecl * Iv)580 void IvarInvalidationCheckerImpl::MethodCrawler::markInvalidated(
581 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'.
587 if (!InvalidationMethod ||
588 (InvalidationMethod && I->second.hasMethod(InvalidationMethod)))
589 IVars.erase(I);
590 }
591 }
592
peel(const Expr * E) const593 const Expr *IvarInvalidationCheckerImpl::MethodCrawler::peel(const Expr *E) const {
594 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 }
601
checkObjCIvarRefExpr(const ObjCIvarRefExpr * IvarRef)602 void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCIvarRefExpr(
603 const ObjCIvarRefExpr *IvarRef) {
604 if (const Decl *D = IvarRef->getDecl())
605 markInvalidated(cast<ObjCIvarDecl>(D->getCanonicalDecl()));
606 }
607
checkObjCMessageExpr(const ObjCMessageExpr * ME)608 void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCMessageExpr(
609 const ObjCMessageExpr *ME) {
610 const ObjCMethodDecl *MD = ME->getMethodDecl();
611 if (MD) {
612 MD = cast<ObjCMethodDecl>(MD->getCanonicalDecl());
613 MethToIvarMapTy::const_iterator IvI = PropertyGetterToIvarMap.find(MD);
614 if (IvI != PropertyGetterToIvarMap.end())
615 markInvalidated(IvI->second);
616 }
617 }
618
checkObjCPropertyRefExpr(const ObjCPropertyRefExpr * PA)619 void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCPropertyRefExpr(
620 const ObjCPropertyRefExpr *PA) {
621
622 if (PA->isExplicitProperty()) {
623 const ObjCPropertyDecl *PD = PA->getExplicitProperty();
624 if (PD) {
625 PD = cast<ObjCPropertyDecl>(PD->getCanonicalDecl());
626 PropToIvarMapTy::const_iterator IvI = PropertyToIvarMap.find(PD);
627 if (IvI != PropertyToIvarMap.end())
628 markInvalidated(IvI->second);
629 return;
630 }
631 }
632
633 if (PA->isImplicitProperty()) {
634 const ObjCMethodDecl *MD = PA->getImplicitPropertySetter();
635 if (MD) {
636 MD = cast<ObjCMethodDecl>(MD->getCanonicalDecl());
637 MethToIvarMapTy::const_iterator IvI =PropertyGetterToIvarMap.find(MD);
638 if (IvI != PropertyGetterToIvarMap.end())
639 markInvalidated(IvI->second);
640 return;
641 }
642 }
643 }
644
isZero(const Expr * E) const645 bool IvarInvalidationCheckerImpl::MethodCrawler::isZero(const Expr *E) const {
646 E = peel(E);
647
648 return (E->isNullPointerConstant(Ctx, Expr::NPC_ValueDependentIsNotNull)
649 != Expr::NPCK_NotNull);
650 }
651
check(const Expr * E)652 void IvarInvalidationCheckerImpl::MethodCrawler::check(const Expr *E) {
653 E = peel(E);
654
655 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
VisitBinaryOperator(const BinaryOperator * BO)671 void IvarInvalidationCheckerImpl::MethodCrawler::VisitBinaryOperator(
672 const BinaryOperator *BO) {
673 VisitStmt(BO);
674
675 // 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)
681 return;
682
683 if (isZero(BO->getRHS())) {
684 check(BO->getLHS());
685 return;
686 }
687
688 if (Opcode != BO_Assign && isZero(BO->getLHS())) {
689 check(BO->getRHS());
690 return;
691 }
692 }
693
VisitObjCMessageExpr(const ObjCMessageExpr * ME)694 void IvarInvalidationCheckerImpl::MethodCrawler::VisitObjCMessageExpr(
695 const ObjCMessageExpr *ME) {
696 const ObjCMethodDecl *MD = ME->getMethodDecl();
697 const Expr *Receiver = ME->getInstanceReceiver();
698
699 // Stop if we are calling '[self invalidate]'.
700 if (Receiver && isInvalidationMethod(MD, /*LookForPartial*/ false))
701 if (Receiver->isObjCSelfExpr()) {
702 CalledAnotherInvalidationMethod = true;
703 return;
704 }
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());
720 InvalidationMethod = nullptr;
721 }
722
723 VisitStmt(ME);
724 }
725 }
726
727 // Register the checkers.
728 namespace {
729
730 class IvarInvalidationChecker :
731 public Checker<check::ASTDecl<ObjCImplementationDecl> > {
732 public:
733 ChecksFilter Filter;
734 public:
checkASTDecl(const ObjCImplementationDecl * D,AnalysisManager & Mgr,BugReporter & BR) const735 void checkASTDecl(const ObjCImplementationDecl *D, AnalysisManager& Mgr,
736 BugReporter &BR) const {
737 IvarInvalidationCheckerImpl Walker(Mgr, BR, Filter);
738 Walker.visit(D);
739 }
740 };
741 }
742
743 #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 }
750
751 REGISTER_CHECKER(InstanceVariableInvalidation)
752 REGISTER_CHECKER(MissingInvalidationMethod)
753
754