• Home
  • Line#
  • Scopes#
  • Navigate#
  • Raw
  • Download
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 struct ChecksFilter {
47   /// Check for missing invalidation method declarations.
48   DefaultBool check_MissingInvalidationMethod;
49   /// Check that all ivars are invalidated.
50   DefaultBool check_InstanceVariableInvalidation;
51 
52   CheckName checkName_MissingInvalidationMethod;
53   CheckName checkName_InstanceVariableInvalidation;
54 };
55 
56 class IvarInvalidationCheckerImpl {
57   typedef llvm::SmallSetVector<const ObjCMethodDecl*, 2> MethodSet;
58   typedef llvm::DenseMap<const ObjCMethodDecl*,
59                          const ObjCIvarDecl*> MethToIvarMapTy;
60   typedef llvm::DenseMap<const ObjCPropertyDecl*,
61                          const ObjCIvarDecl*> PropToIvarMapTy;
62   typedef llvm::DenseMap<const ObjCIvarDecl*,
63                          const ObjCPropertyDecl*> IvarToPropMapTy;
64 
65   struct InvalidationInfo {
66     /// Has the ivar been invalidated?
67     bool IsInvalidated;
68 
69     /// The methods which can be used to invalidate the ivar.
70     MethodSet InvalidationMethods;
71 
InvalidationInfo__anon9851fed80111::IvarInvalidationCheckerImpl::InvalidationInfo72     InvalidationInfo() : IsInvalidated(false) {}
addInvalidationMethod__anon9851fed80111::IvarInvalidationCheckerImpl::InvalidationInfo73     void addInvalidationMethod(const ObjCMethodDecl *MD) {
74       InvalidationMethods.insert(MD);
75     }
76 
needsInvalidation__anon9851fed80111::IvarInvalidationCheckerImpl::InvalidationInfo77     bool needsInvalidation() const {
78       return !InvalidationMethods.empty();
79     }
80 
hasMethod__anon9851fed80111::IvarInvalidationCheckerImpl::InvalidationInfo81     bool hasMethod(const ObjCMethodDecl *MD) {
82       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     }
93   };
94 
95   typedef llvm::DenseMap<const ObjCIvarDecl*, InvalidationInfo> IvarSet;
96 
97   /// 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> {
100     /// The set of Ivars which need to be invalidated.
101     IvarSet &IVars;
102 
103     /// Flag is set as the result of a message send to another
104     /// invalidation method.
105     bool &CalledAnotherInvalidationMethod;
106 
107     /// 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 
119     ASTContext &Ctx;
120 
121     /// Peel off parens, casts, OpaqueValueExpr, and PseudoObjectExpr.
122     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);
144 
145   public:
MethodCrawler(IvarSet & InIVars,bool & InCalledAnotherInvalidationMethod,const MethToIvarMapTy & InPropertySetterToIvarMap,const MethToIvarMapTy & InPropertyGetterToIvarMap,const PropToIvarMapTy & InPropertyToIvarMap,ASTContext & InCtx)146     MethodCrawler(IvarSet &InIVars,
147                   bool &InCalledAnotherInvalidationMethod,
148                   const MethToIvarMapTy &InPropertySetterToIvarMap,
149                   const MethToIvarMapTy &InPropertyGetterToIvarMap,
150                   const PropToIvarMapTy &InPropertyToIvarMap,
151                   ASTContext &InCtx)
152     : IVars(InIVars),
153       CalledAnotherInvalidationMethod(InCalledAnotherInvalidationMethod),
154       PropertySetterToIvarMap(InPropertySetterToIvarMap),
155       PropertyGetterToIvarMap(InPropertyGetterToIvarMap),
156       PropertyToIvarMap(InPropertyToIvarMap),
157       InvalidationMethod(nullptr),
158       Ctx(InCtx) {}
159 
VisitStmt(const Stmt * S)160     void VisitStmt(const Stmt *S) { VisitChildren(S); }
161 
162     void VisitBinaryOperator(const BinaryOperator *BO);
163 
164     void VisitObjCMessageExpr(const ObjCMessageExpr *ME);
165 
VisitChildren(const Stmt * S)166     void VisitChildren(const Stmt *S) {
167       for (const auto *Child : S->children()) {
168         if (Child)
169           this->Visit(Child);
170         if (CalledAnotherInvalidationMethod)
171           return;
172       }
173     }
174   };
175 
176   /// Check if the any of the methods inside the interface are annotated with
177   /// the invalidation annotation, update the IvarInfo accordingly.
178   /// \param LookForPartial is set when we are searching for partial
179   ///        invalidators.
180   static void containsInvalidationMethod(const ObjCContainerDecl *D,
181                                          InvalidationInfo &Out,
182                                          bool LookForPartial);
183 
184   /// Check if ivar should be tracked and add to TrackedIvars if positive.
185   /// Returns true if ivar should be tracked.
186   static bool trackIvar(const ObjCIvarDecl *Iv, IvarSet &TrackedIvars,
187                         const ObjCIvarDecl **FirstIvarDecl);
188 
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,
195       IvarSet &TrackedIvars,
196       const ObjCIvarDecl **FirstIvarDecl);
197 
198   /// 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,
201                         const IvarToPropMapTy &IvarToPopertyMap);
202 
203   void reportNoInvalidationMethod(CheckName CheckName,
204                                   const ObjCIvarDecl *FirstIvarDecl,
205                                   const IvarToPropMapTy &IvarToPopertyMap,
206                                   const ObjCInterfaceDecl *InterfaceD,
207                                   bool MissingDeclaration) const;
208 
209   void reportIvarNeedsInvalidation(const ObjCIvarDecl *IvarD,
210                                    const IvarToPropMapTy &IvarToPopertyMap,
211                                    const ObjCMethodDecl *MethodD) const;
212 
213   AnalysisManager& Mgr;
214   BugReporter &BR;
215   /// Filter on the checks performed.
216   const ChecksFilter &Filter;
217 
218 public:
IvarInvalidationCheckerImpl(AnalysisManager & InMgr,BugReporter & InBR,const ChecksFilter & InFilter)219   IvarInvalidationCheckerImpl(AnalysisManager& InMgr,
220                               BugReporter &InBR,
221                               const ChecksFilter &InFilter) :
222     Mgr (InMgr), BR(InBR), Filter(InFilter) {}
223 
224   void visit(const ObjCImplementationDecl *D) const;
225 };
226 
isInvalidationMethod(const ObjCMethodDecl * M,bool LookForPartial)227 static bool isInvalidationMethod(const ObjCMethodDecl *M, bool LookForPartial) {
228   for (const auto *Ann : M->specific_attrs<AnnotateAttr>()) {
229     if (!LookForPartial &&
230         Ann->getAnnotation() == "objc_instance_variable_invalidator")
231       return true;
232     if (LookForPartial &&
233         Ann->getAnnotation() == "objc_instance_variable_invalidator_partial")
234       return true;
235   }
236   return false;
237 }
238 
containsInvalidationMethod(const ObjCContainerDecl * D,InvalidationInfo & OutInfo,bool Partial)239 void IvarInvalidationCheckerImpl::containsInvalidationMethod(
240     const ObjCContainerDecl *D, InvalidationInfo &OutInfo, bool Partial) {
241 
242   if (!D)
243     return;
244 
245   assert(!isa<ObjCImplementationDecl>(D));
246   // TODO: Cache the results.
247 
248   // Check all methods.
249   for (const auto *MDI : D->methods())
250     if (isInvalidationMethod(MDI, Partial))
251       OutInfo.addInvalidationMethod(
252           cast<ObjCMethodDecl>(MDI->getCanonicalDecl()));
253 
254   // If interface, check all parent protocols and super.
255   if (const ObjCInterfaceDecl *InterfD = dyn_cast<ObjCInterfaceDecl>(D)) {
256 
257     // Visit all protocols.
258     for (const auto *I : InterfD->protocols())
259       containsInvalidationMethod(I->getDefinition(), OutInfo, Partial);
260 
261     // Visit all categories in case the invalidation method is declared in
262     // a category.
263     for (const auto *Ext : InterfD->visible_extensions())
264       containsInvalidationMethod(Ext, OutInfo, Partial);
265 
266     containsInvalidationMethod(InterfD->getSuperClass(), OutInfo, Partial);
267     return;
268   }
269 
270   // If protocol, check all parent protocols.
271   if (const ObjCProtocolDecl *ProtD = dyn_cast<ObjCProtocolDecl>(D)) {
272     for (const auto *I : ProtD->protocols()) {
273       containsInvalidationMethod(I->getDefinition(), OutInfo, Partial);
274     }
275     return;
276   }
277 }
278 
trackIvar(const ObjCIvarDecl * Iv,IvarSet & TrackedIvars,const ObjCIvarDecl ** FirstIvarDecl)279 bool IvarInvalidationCheckerImpl::trackIvar(const ObjCIvarDecl *Iv,
280                                         IvarSet &TrackedIvars,
281                                         const ObjCIvarDecl **FirstIvarDecl) {
282   QualType IvQTy = Iv->getType();
283   const ObjCObjectPointerType *IvTy = IvQTy->getAs<ObjCObjectPointerType>();
284   if (!IvTy)
285     return false;
286   const ObjCInterfaceDecl *IvInterf = IvTy->getInterfaceDecl();
287 
288   InvalidationInfo Info;
289   containsInvalidationMethod(IvInterf, Info, /*LookForPartial*/ false);
290   if (Info.needsInvalidation()) {
291     const ObjCIvarDecl *I = cast<ObjCIvarDecl>(Iv->getCanonicalDecl());
292     TrackedIvars[I] = Info;
293     if (!*FirstIvarDecl)
294       *FirstIvarDecl = I;
295     return true;
296   }
297   return false;
298 }
299 
findPropertyBackingIvar(const ObjCPropertyDecl * Prop,const ObjCInterfaceDecl * InterfaceD,IvarSet & TrackedIvars,const ObjCIvarDecl ** FirstIvarDecl)300 const ObjCIvarDecl *IvarInvalidationCheckerImpl::findPropertyBackingIvar(
301                         const ObjCPropertyDecl *Prop,
302                         const ObjCInterfaceDecl *InterfaceD,
303                         IvarSet &TrackedIvars,
304                         const ObjCIvarDecl **FirstIvarDecl) {
305   const ObjCIvarDecl *IvarD = nullptr;
306 
307   // Lookup for the synthesized case.
308   IvarD = Prop->getPropertyIvarDecl();
309   // We only track the ivars/properties that are defined in the current
310   // class (not the parent).
311   if (IvarD && IvarD->getContainingInterface() == InterfaceD) {
312     if (TrackedIvars.count(IvarD)) {
313       return IvarD;
314     }
315     // If the ivar is synthesized we still want to track it.
316     if (trackIvar(IvarD, TrackedIvars, FirstIvarDecl))
317       return IvarD;
318   }
319 
320   // Lookup IVars named "_PropName"or "PropName" among the tracked Ivars.
321   StringRef PropName = Prop->getIdentifier()->getName();
322   for (IvarSet::const_iterator I = TrackedIvars.begin(),
323                                E = TrackedIvars.end(); I != E; ++I) {
324     const ObjCIvarDecl *Iv = I->first;
325     StringRef IvarName = Iv->getName();
326 
327     if (IvarName == PropName)
328       return Iv;
329 
330     SmallString<128> PropNameWithUnderscore;
331     {
332       llvm::raw_svector_ostream os(PropNameWithUnderscore);
333       os << '_' << PropName;
334     }
335     if (IvarName == PropNameWithUnderscore)
336       return Iv;
337   }
338 
339   // Note, this is a possible source of false positives. We could look at the
340   // getter implementation to find the ivar when its name is not derived from
341   // the property name.
342   return nullptr;
343 }
344 
printIvar(llvm::raw_svector_ostream & os,const ObjCIvarDecl * IvarDecl,const IvarToPropMapTy & IvarToPopertyMap)345 void IvarInvalidationCheckerImpl::printIvar(llvm::raw_svector_ostream &os,
346                                       const ObjCIvarDecl *IvarDecl,
347                                       const IvarToPropMapTy &IvarToPopertyMap) {
348   if (IvarDecl->getSynthesize()) {
349     const ObjCPropertyDecl *PD = IvarToPopertyMap.lookup(IvarDecl);
350     assert(PD &&"Do we synthesize ivars for something other than properties?");
351     os << "Property "<< PD->getName() << " ";
352   } else {
353     os << "Instance variable "<< IvarDecl->getName() << " ";
354   }
355 }
356 
357 // Check that the invalidatable interfaces with ivars/properties implement the
358 // invalidation methods.
359 void IvarInvalidationCheckerImpl::
visit(const ObjCImplementationDecl * ImplD) const360 visit(const ObjCImplementationDecl *ImplD) const {
361   // Collect all ivars that need cleanup.
362   IvarSet Ivars;
363   // Record the first Ivar needing invalidation; used in reporting when only
364   // one ivar is sufficient. Cannot grab the first on the Ivars set to ensure
365   // deterministic output.
366   const ObjCIvarDecl *FirstIvarDecl = nullptr;
367   const ObjCInterfaceDecl *InterfaceD = ImplD->getClassInterface();
368 
369   // Collect ivars declared in this class, its extensions and its implementation
370   ObjCInterfaceDecl *IDecl = const_cast<ObjCInterfaceDecl *>(InterfaceD);
371   for (const ObjCIvarDecl *Iv = IDecl->all_declared_ivar_begin(); Iv;
372        Iv= Iv->getNextIvar())
373     trackIvar(Iv, Ivars, &FirstIvarDecl);
374 
375   // Construct Property/Property Accessor to Ivar maps to assist checking if an
376   // ivar which is backing a property has been reset.
377   MethToIvarMapTy PropSetterToIvarMap;
378   MethToIvarMapTy PropGetterToIvarMap;
379   PropToIvarMapTy PropertyToIvarMap;
380   IvarToPropMapTy IvarToPopertyMap;
381 
382   ObjCInterfaceDecl::PropertyMap PropMap;
383   ObjCInterfaceDecl::PropertyDeclOrder PropOrder;
384   InterfaceD->collectPropertiesToImplement(PropMap, PropOrder);
385 
386   for (ObjCInterfaceDecl::PropertyMap::iterator
387       I = PropMap.begin(), E = PropMap.end(); I != E; ++I) {
388     const ObjCPropertyDecl *PD = I->second;
389     if (PD->isClassProperty())
390       continue;
391 
392     const ObjCIvarDecl *ID = findPropertyBackingIvar(PD, InterfaceD, Ivars,
393                                                      &FirstIvarDecl);
394     if (!ID)
395       continue;
396 
397     // Store the mappings.
398     PD = cast<ObjCPropertyDecl>(PD->getCanonicalDecl());
399     PropertyToIvarMap[PD] = ID;
400     IvarToPopertyMap[ID] = PD;
401 
402     // Find the setter and the getter.
403     const ObjCMethodDecl *SetterD = PD->getSetterMethodDecl();
404     if (SetterD) {
405       SetterD = cast<ObjCMethodDecl>(SetterD->getCanonicalDecl());
406       PropSetterToIvarMap[SetterD] = ID;
407     }
408 
409     const ObjCMethodDecl *GetterD = PD->getGetterMethodDecl();
410     if (GetterD) {
411       GetterD = cast<ObjCMethodDecl>(GetterD->getCanonicalDecl());
412       PropGetterToIvarMap[GetterD] = ID;
413     }
414   }
415 
416   // If no ivars need invalidation, there is nothing to check here.
417   if (Ivars.empty())
418     return;
419 
420   // Find all partial invalidation methods.
421   InvalidationInfo PartialInfo;
422   containsInvalidationMethod(InterfaceD, PartialInfo, /*LookForPartial*/ true);
423 
424   // Remove ivars invalidated by the partial invalidation methods. They do not
425   // need to be invalidated in the regular invalidation methods.
426   bool AtImplementationContainsAtLeastOnePartialInvalidationMethod = false;
427   for (MethodSet::iterator
428       I = PartialInfo.InvalidationMethods.begin(),
429       E = PartialInfo.InvalidationMethods.end(); I != E; ++I) {
430     const ObjCMethodDecl *InterfD = *I;
431 
432     // Get the corresponding method in the @implementation.
433     const ObjCMethodDecl *D = ImplD->getMethod(InterfD->getSelector(),
434                                                InterfD->isInstanceMethod());
435     if (D && D->hasBody()) {
436       AtImplementationContainsAtLeastOnePartialInvalidationMethod = true;
437 
438       bool CalledAnotherInvalidationMethod = false;
439       // The MethodCrowler is going to remove the invalidated ivars.
440       MethodCrawler(Ivars,
441                     CalledAnotherInvalidationMethod,
442                     PropSetterToIvarMap,
443                     PropGetterToIvarMap,
444                     PropertyToIvarMap,
445                     BR.getContext()).VisitStmt(D->getBody());
446       // If another invalidation method was called, trust that full invalidation
447       // has occurred.
448       if (CalledAnotherInvalidationMethod)
449         Ivars.clear();
450     }
451   }
452 
453   // If all ivars have been invalidated by partial invalidators, there is
454   // nothing to check here.
455   if (Ivars.empty())
456     return;
457 
458   // Find all invalidation methods in this @interface declaration and parents.
459   InvalidationInfo Info;
460   containsInvalidationMethod(InterfaceD, Info, /*LookForPartial*/ false);
461 
462   // Report an error in case none of the invalidation methods are declared.
463   if (!Info.needsInvalidation() && !PartialInfo.needsInvalidation()) {
464     if (Filter.check_MissingInvalidationMethod)
465       reportNoInvalidationMethod(Filter.checkName_MissingInvalidationMethod,
466                                  FirstIvarDecl, IvarToPopertyMap, InterfaceD,
467                                  /*MissingDeclaration*/ true);
468     // If there are no invalidation methods, there is no ivar validation work
469     // to be done.
470     return;
471   }
472 
473   // Only check if Ivars are invalidated when InstanceVariableInvalidation
474   // has been requested.
475   if (!Filter.check_InstanceVariableInvalidation)
476     return;
477 
478   // Check that all ivars are invalidated by the invalidation methods.
479   bool AtImplementationContainsAtLeastOneInvalidationMethod = false;
480   for (MethodSet::iterator I = Info.InvalidationMethods.begin(),
481                            E = Info.InvalidationMethods.end(); I != E; ++I) {
482     const ObjCMethodDecl *InterfD = *I;
483 
484     // Get the corresponding method in the @implementation.
485     const ObjCMethodDecl *D = ImplD->getMethod(InterfD->getSelector(),
486                                                InterfD->isInstanceMethod());
487     if (D && D->hasBody()) {
488       AtImplementationContainsAtLeastOneInvalidationMethod = true;
489 
490       // Get a copy of ivars needing invalidation.
491       IvarSet IvarsI = Ivars;
492 
493       bool CalledAnotherInvalidationMethod = false;
494       MethodCrawler(IvarsI,
495                     CalledAnotherInvalidationMethod,
496                     PropSetterToIvarMap,
497                     PropGetterToIvarMap,
498                     PropertyToIvarMap,
499                     BR.getContext()).VisitStmt(D->getBody());
500       // If another invalidation method was called, trust that full invalidation
501       // has occurred.
502       if (CalledAnotherInvalidationMethod)
503         continue;
504 
505       // Warn on the ivars that were not invalidated by the method.
506       for (IvarSet::const_iterator
507           I = IvarsI.begin(), E = IvarsI.end(); I != E; ++I)
508         reportIvarNeedsInvalidation(I->first, IvarToPopertyMap, D);
509     }
510   }
511 
512   // Report an error in case none of the invalidation methods are implemented.
513   if (!AtImplementationContainsAtLeastOneInvalidationMethod) {
514     if (AtImplementationContainsAtLeastOnePartialInvalidationMethod) {
515       // Warn on the ivars that were not invalidated by the prrtial
516       // invalidation methods.
517       for (IvarSet::const_iterator
518            I = Ivars.begin(), E = Ivars.end(); I != E; ++I)
519         reportIvarNeedsInvalidation(I->first, IvarToPopertyMap, nullptr);
520     } else {
521       // Otherwise, no invalidation methods were implemented.
522       reportNoInvalidationMethod(Filter.checkName_InstanceVariableInvalidation,
523                                  FirstIvarDecl, IvarToPopertyMap, InterfaceD,
524                                  /*MissingDeclaration*/ false);
525     }
526   }
527 }
528 
reportNoInvalidationMethod(CheckName CheckName,const ObjCIvarDecl * FirstIvarDecl,const IvarToPropMapTy & IvarToPopertyMap,const ObjCInterfaceDecl * InterfaceD,bool MissingDeclaration) const529 void IvarInvalidationCheckerImpl::reportNoInvalidationMethod(
530     CheckName CheckName, const ObjCIvarDecl *FirstIvarDecl,
531     const IvarToPropMapTy &IvarToPopertyMap,
532     const ObjCInterfaceDecl *InterfaceD, bool MissingDeclaration) const {
533   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, CheckName, "Incomplete invalidation",
548                      categories::CoreFoundationObjectiveC, os.str(),
549                      IvarDecLocation);
550 }
551 
552 void IvarInvalidationCheckerImpl::
reportIvarNeedsInvalidation(const ObjCIvarDecl * IvarD,const IvarToPropMapTy & IvarToPopertyMap,const ObjCMethodDecl * MethodD) const553 reportIvarNeedsInvalidation(const ObjCIvarDecl *IvarD,
554                             const IvarToPropMapTy &IvarToPopertyMap,
555                             const ObjCMethodDecl *MethodD) const {
556   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   if (MethodD) {
561     PathDiagnosticLocation MethodDecLocation =
562                            PathDiagnosticLocation::createEnd(MethodD->getBody(),
563                            BR.getSourceManager(),
564                            Mgr.getAnalysisDeclContext(MethodD));
565     BR.EmitBasicReport(MethodD, Filter.checkName_InstanceVariableInvalidation,
566                        "Incomplete invalidation",
567                        categories::CoreFoundationObjectiveC, os.str(),
568                        MethodDecLocation);
569   } else {
570     BR.EmitBasicReport(
571         IvarD, Filter.checkName_InstanceVariableInvalidation,
572         "Incomplete invalidation", categories::CoreFoundationObjectiveC,
573         os.str(),
574         PathDiagnosticLocation::createBegin(IvarD, BR.getSourceManager()));
575   }
576 }
577 
markInvalidated(const ObjCIvarDecl * Iv)578 void IvarInvalidationCheckerImpl::MethodCrawler::markInvalidated(
579     const ObjCIvarDecl *Iv) {
580   IvarSet::iterator I = IVars.find(Iv);
581   if (I != IVars.end()) {
582     // If InvalidationMethod is present, we are processing the message send and
583     // should ensure we are invalidating with the appropriate method,
584     // otherwise, we are processing setting to 'nil'.
585     if (!InvalidationMethod || I->second.hasMethod(InvalidationMethod))
586       IVars.erase(I);
587   }
588 }
589 
peel(const Expr * E) const590 const Expr *IvarInvalidationCheckerImpl::MethodCrawler::peel(const Expr *E) const {
591   E = E->IgnoreParenCasts();
592   if (const PseudoObjectExpr *POE = dyn_cast<PseudoObjectExpr>(E))
593     E = POE->getSyntacticForm()->IgnoreParenCasts();
594   if (const OpaqueValueExpr *OVE = dyn_cast<OpaqueValueExpr>(E))
595     E = OVE->getSourceExpr()->IgnoreParenCasts();
596   return E;
597 }
598 
checkObjCIvarRefExpr(const ObjCIvarRefExpr * IvarRef)599 void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCIvarRefExpr(
600     const ObjCIvarRefExpr *IvarRef) {
601   if (const Decl *D = IvarRef->getDecl())
602     markInvalidated(cast<ObjCIvarDecl>(D->getCanonicalDecl()));
603 }
604 
checkObjCMessageExpr(const ObjCMessageExpr * ME)605 void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCMessageExpr(
606     const ObjCMessageExpr *ME) {
607   const ObjCMethodDecl *MD = ME->getMethodDecl();
608   if (MD) {
609     MD = cast<ObjCMethodDecl>(MD->getCanonicalDecl());
610     MethToIvarMapTy::const_iterator IvI = PropertyGetterToIvarMap.find(MD);
611     if (IvI != PropertyGetterToIvarMap.end())
612       markInvalidated(IvI->second);
613   }
614 }
615 
checkObjCPropertyRefExpr(const ObjCPropertyRefExpr * PA)616 void IvarInvalidationCheckerImpl::MethodCrawler::checkObjCPropertyRefExpr(
617     const ObjCPropertyRefExpr *PA) {
618 
619   if (PA->isExplicitProperty()) {
620     const ObjCPropertyDecl *PD = PA->getExplicitProperty();
621     if (PD) {
622       PD = cast<ObjCPropertyDecl>(PD->getCanonicalDecl());
623       PropToIvarMapTy::const_iterator IvI = PropertyToIvarMap.find(PD);
624       if (IvI != PropertyToIvarMap.end())
625         markInvalidated(IvI->second);
626       return;
627     }
628   }
629 
630   if (PA->isImplicitProperty()) {
631     const ObjCMethodDecl *MD = PA->getImplicitPropertySetter();
632     if (MD) {
633       MD = cast<ObjCMethodDecl>(MD->getCanonicalDecl());
634       MethToIvarMapTy::const_iterator IvI =PropertyGetterToIvarMap.find(MD);
635       if (IvI != PropertyGetterToIvarMap.end())
636         markInvalidated(IvI->second);
637       return;
638     }
639   }
640 }
641 
isZero(const Expr * E) const642 bool IvarInvalidationCheckerImpl::MethodCrawler::isZero(const Expr *E) const {
643   E = peel(E);
644 
645   return (E->isNullPointerConstant(Ctx, Expr::NPC_ValueDependentIsNotNull)
646            != Expr::NPCK_NotNull);
647 }
648 
check(const Expr * E)649 void IvarInvalidationCheckerImpl::MethodCrawler::check(const Expr *E) {
650   E = peel(E);
651 
652   if (const ObjCIvarRefExpr *IvarRef = dyn_cast<ObjCIvarRefExpr>(E)) {
653     checkObjCIvarRefExpr(IvarRef);
654     return;
655   }
656 
657   if (const ObjCPropertyRefExpr *PropRef = dyn_cast<ObjCPropertyRefExpr>(E)) {
658     checkObjCPropertyRefExpr(PropRef);
659     return;
660   }
661 
662   if (const ObjCMessageExpr *MsgExpr = dyn_cast<ObjCMessageExpr>(E)) {
663     checkObjCMessageExpr(MsgExpr);
664     return;
665   }
666 }
667 
VisitBinaryOperator(const BinaryOperator * BO)668 void IvarInvalidationCheckerImpl::MethodCrawler::VisitBinaryOperator(
669     const BinaryOperator *BO) {
670   VisitStmt(BO);
671 
672   // Do we assign/compare against zero? If yes, check the variable we are
673   // assigning to.
674   BinaryOperatorKind Opcode = BO->getOpcode();
675   if (Opcode != BO_Assign &&
676       Opcode != BO_EQ &&
677       Opcode != BO_NE)
678     return;
679 
680   if (isZero(BO->getRHS())) {
681       check(BO->getLHS());
682       return;
683   }
684 
685   if (Opcode != BO_Assign && isZero(BO->getLHS())) {
686     check(BO->getRHS());
687     return;
688   }
689 }
690 
VisitObjCMessageExpr(const ObjCMessageExpr * ME)691 void IvarInvalidationCheckerImpl::MethodCrawler::VisitObjCMessageExpr(
692   const ObjCMessageExpr *ME) {
693   const ObjCMethodDecl *MD = ME->getMethodDecl();
694   const Expr *Receiver = ME->getInstanceReceiver();
695 
696   // Stop if we are calling '[self invalidate]'.
697   if (Receiver && isInvalidationMethod(MD, /*LookForPartial*/ false))
698     if (Receiver->isObjCSelfExpr()) {
699       CalledAnotherInvalidationMethod = true;
700       return;
701     }
702 
703   // Check if we call a setter and set the property to 'nil'.
704   if (MD && (ME->getNumArgs() == 1) && isZero(ME->getArg(0))) {
705     MD = cast<ObjCMethodDecl>(MD->getCanonicalDecl());
706     MethToIvarMapTy::const_iterator IvI = PropertySetterToIvarMap.find(MD);
707     if (IvI != PropertySetterToIvarMap.end()) {
708       markInvalidated(IvI->second);
709       return;
710     }
711   }
712 
713   // Check if we call the 'invalidation' routine on the ivar.
714   if (Receiver) {
715     InvalidationMethod = MD;
716     check(Receiver->IgnoreParenCasts());
717     InvalidationMethod = nullptr;
718   }
719 
720   VisitStmt(ME);
721 }
722 } // end anonymous namespace
723 
724 // Register the checkers.
725 namespace {
726 class IvarInvalidationChecker :
727   public Checker<check::ASTDecl<ObjCImplementationDecl> > {
728 public:
729   ChecksFilter Filter;
730 public:
checkASTDecl(const ObjCImplementationDecl * D,AnalysisManager & Mgr,BugReporter & BR) const731   void checkASTDecl(const ObjCImplementationDecl *D, AnalysisManager& Mgr,
732                     BugReporter &BR) const {
733     IvarInvalidationCheckerImpl Walker(Mgr, BR, Filter);
734     Walker.visit(D);
735   }
736 };
737 } // end anonymous namespace
738 
739 #define REGISTER_CHECKER(name)                                                 \
740   void ento::register##name(CheckerManager &mgr) {                             \
741     IvarInvalidationChecker *checker =                                         \
742         mgr.registerChecker<IvarInvalidationChecker>();                        \
743     checker->Filter.check_##name = true;                                       \
744     checker->Filter.checkName_##name = mgr.getCurrentCheckName();              \
745   }
746 
747 REGISTER_CHECKER(InstanceVariableInvalidation)
748 REGISTER_CHECKER(MissingInvalidationMethod)
749