xref: /llvm-project/clang/lib/StaticAnalyzer/Checkers/CheckObjCDealloc.cpp (revision ec6f61cc2e10f3625d15a10587209eb25e6b0c76)
1 //==- CheckObjCDealloc.cpp - Check ObjC -dealloc implementation --*- 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 analyzes Objective-C -dealloc methods and their callees
11 //  to warn about improper releasing of instance variables that back synthesized
12 // properties. It warns about missing releases in the following cases:
13 //  - When a class has a synthesized instance variable for a 'retain' or 'copy'
14 //    property and lacks a -dealloc method in its implementation.
15 //  - When a class has a synthesized instance variable for a 'retain'/'copy'
16 //   property but the ivar is not released in -dealloc by either -release
17 //   or by nilling out the property.
18 //
19 //  It warns about extra releases in -dealloc (but not in callees) when a
20 //  synthesized instance variable is released in the following cases:
21 //  - When the property is 'assign' and is not 'readonly'.
22 //  - When the property is 'weak'.
23 //
24 //  This checker only warns for instance variables synthesized to back
25 //  properties. Handling the more general case would require inferring whether
26 //  an instance variable is stored retained or not. For synthesized properties,
27 //  this is specified in the property declaration itself.
28 //
29 //===----------------------------------------------------------------------===//
30 
31 #include "ClangSACheckers.h"
32 #include "clang/AST/Attr.h"
33 #include "clang/AST/DeclObjC.h"
34 #include "clang/AST/Expr.h"
35 #include "clang/AST/ExprObjC.h"
36 #include "clang/Basic/LangOptions.h"
37 #include "clang/StaticAnalyzer/Core/BugReporter/BugReporter.h"
38 #include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
39 #include "clang/StaticAnalyzer/Core/BugReporter/PathDiagnostic.h"
40 #include "clang/StaticAnalyzer/Core/Checker.h"
41 #include "clang/StaticAnalyzer/Core/PathSensitive/AnalysisManager.h"
42 #include "clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h"
43 #include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
44 #include "clang/StaticAnalyzer/Core/PathSensitive/ProgramState.h"
45 #include "clang/StaticAnalyzer/Core/PathSensitive/ProgramStateTrait.h"
46 #include "clang/StaticAnalyzer/Core/PathSensitive/SymbolManager.h"
47 #include "llvm/Support/raw_ostream.h"
48 
49 using namespace clang;
50 using namespace ento;
51 
52 /// Indicates whether an instance variable is required to be released in
53 /// -dealloc.
54 enum class ReleaseRequirement {
55   /// The instance variable must be released, either by calling
56   /// -release on it directly or by nilling it out with a property setter.
57   MustRelease,
58 
59   /// The instance variable must not be directly released with -release.
60   MustNotReleaseDirectly,
61 
62   /// The requirement for the instance variable could not be determined.
63   Unknown
64 };
65 
66 /// Returns true if the property implementation is synthesized and the
67 /// type of the property is retainable.
68 static bool isSynthesizedRetainableProperty(const ObjCPropertyImplDecl *I,
69                                             const ObjCIvarDecl **ID,
70                                             const ObjCPropertyDecl **PD) {
71 
72   if (I->getPropertyImplementation() != ObjCPropertyImplDecl::Synthesize)
73     return false;
74 
75   (*ID) = I->getPropertyIvarDecl();
76   if (!(*ID))
77     return false;
78 
79   QualType T = (*ID)->getType();
80   if (!T->isObjCRetainableType())
81     return false;
82 
83   (*PD) = I->getPropertyDecl();
84   // Shouldn't be able to synthesize a property that doesn't exist.
85   assert(*PD);
86 
87   return true;
88 }
89 
90 namespace {
91 
92 class ObjCDeallocChecker
93     : public Checker<check::ASTDecl<ObjCImplementationDecl>,
94                      check::PreObjCMessage, check::PostObjCMessage,
95                      check::BeginFunction, check::EndFunction,
96                      check::PointerEscape,
97                      check::PreStmt<ReturnStmt>> {
98 
99   mutable IdentifierInfo *NSObjectII, *SenTestCaseII;
100   mutable Selector DeallocSel, ReleaseSel;
101 
102   std::unique_ptr<BugType> MissingReleaseBugType;
103   std::unique_ptr<BugType> ExtraReleaseBugType;
104 
105 public:
106   ObjCDeallocChecker();
107 
108   void checkASTDecl(const ObjCImplementationDecl *D, AnalysisManager& Mgr,
109                     BugReporter &BR) const;
110   void checkBeginFunction(CheckerContext &Ctx) const;
111   void checkPreObjCMessage(const ObjCMethodCall &M, CheckerContext &C) const;
112   void checkPostObjCMessage(const ObjCMethodCall &M, CheckerContext &C) const;
113 
114   ProgramStateRef checkPointerEscape(ProgramStateRef State,
115                                      const InvalidatedSymbols &Escaped,
116                                      const CallEvent *Call,
117                                      PointerEscapeKind Kind) const;
118   void checkPreStmt(const ReturnStmt *RS, CheckerContext &C) const;
119   void checkEndFunction(CheckerContext &Ctx) const;
120 
121 private:
122   void diagnoseMissingReleases(CheckerContext &C) const;
123 
124   bool diagnoseExtraRelease(SymbolRef ReleasedValue, const ObjCMethodCall &M,
125                             CheckerContext &C) const;
126 
127   SymbolRef getValueExplicitlyReleased(const ObjCMethodCall &M,
128                                       CheckerContext &C) const;
129   SymbolRef getValueReleasedByNillingOut(const ObjCMethodCall &M,
130                                          CheckerContext &C) const;
131 
132   SymbolRef getInstanceSymbolFromIvarSymbol(SymbolRef IvarSym) const;
133 
134   ReleaseRequirement
135   getDeallocReleaseRequirement(const ObjCPropertyImplDecl *PropImpl) const;
136 
137   bool isInInstanceDealloc(const CheckerContext &C, SVal &SelfValOut) const;
138   bool isInInstanceDealloc(const CheckerContext &C, const LocationContext *LCtx,
139                            SVal &SelfValOut) const;
140   bool instanceDeallocIsOnStack(const CheckerContext &C,
141                                 SVal &InstanceValOut) const;
142 
143   bool isSuperDeallocMessage(const ObjCMethodCall &M) const;
144 
145   const ObjCImplDecl *getContainingObjCImpl(const LocationContext *LCtx) const;
146 
147   const ObjCPropertyDecl *
148   findShadowedPropertyDecl(const ObjCPropertyImplDecl *PropImpl) const;
149 
150   ProgramStateRef removeValueRequiringRelease(ProgramStateRef State,
151                                               SymbolRef InstanceSym,
152                                               SymbolRef ValueSym) const;
153 
154   void initIdentifierInfoAndSelectors(ASTContext &Ctx) const;
155 
156   bool classHasSeparateTeardown(const ObjCInterfaceDecl *ID) const;
157 };
158 } // End anonymous namespace.
159 
160 typedef llvm::ImmutableSet<SymbolRef> SymbolSet;
161 
162 /// Maps from the symbol for a class instance to the set of
163 /// symbols remaining that must be released in -dealloc.
164 REGISTER_MAP_WITH_PROGRAMSTATE(UnreleasedIvarMap, SymbolRef, SymbolSet)
165 
166 namespace clang {
167 namespace ento {
168 template<> struct ProgramStateTrait<SymbolSet>
169 :  public ProgramStatePartialTrait<SymbolSet> {
170   static void *GDMIndex() { static int index = 0; return &index; }
171 };
172 }
173 }
174 
175 /// An AST check that diagnose when the class requires a -dealloc method and
176 /// is missing one.
177 void ObjCDeallocChecker::checkASTDecl(const ObjCImplementationDecl *D,
178                                       AnalysisManager &Mgr,
179                                       BugReporter &BR) const {
180   assert(Mgr.getLangOpts().getGC() != LangOptions::GCOnly);
181   assert(!Mgr.getLangOpts().ObjCAutoRefCount);
182   initIdentifierInfoAndSelectors(Mgr.getASTContext());
183 
184   const ObjCInterfaceDecl *ID = D->getClassInterface();
185 
186   // Does the class contain any synthesized properties that are retainable?
187   // If not, skip the check entirely.
188   bool containsRetainedSynthesizedProperty = false;
189   for (const auto *I : D->property_impls()) {
190     if (getDeallocReleaseRequirement(I) == ReleaseRequirement::MustRelease) {
191       containsRetainedSynthesizedProperty = true;
192       break;
193     }
194   }
195 
196   if (!containsRetainedSynthesizedProperty)
197     return;
198 
199   // If the class is known to have a lifecycle with a separate teardown method
200   // then it may not require a -dealloc method.
201   if (classHasSeparateTeardown(ID))
202     return;
203 
204   const ObjCMethodDecl *MD = nullptr;
205 
206   // Scan the instance methods for "dealloc".
207   for (const auto *I : D->instance_methods()) {
208     if (I->getSelector() == DeallocSel) {
209       MD = I;
210       break;
211     }
212   }
213 
214   if (!MD) { // No dealloc found.
215     const char* Name = "Missing -dealloc";
216 
217     std::string Buf;
218     llvm::raw_string_ostream OS(Buf);
219     OS << "Objective-C class '" << *D << "' lacks a 'dealloc' instance method";
220 
221     PathDiagnosticLocation DLoc =
222         PathDiagnosticLocation::createBegin(D, BR.getSourceManager());
223 
224     BR.EmitBasicReport(D, this, Name, categories::CoreFoundationObjectiveC,
225                        OS.str(), DLoc);
226     return;
227   }
228 }
229 
230 /// If this is the beginning of -dealloc, mark the values initially stored in
231 /// instance variables that must be released by the end of -dealloc
232 /// as unreleased in the state.
233 void ObjCDeallocChecker::checkBeginFunction(
234     CheckerContext &C) const {
235   initIdentifierInfoAndSelectors(C.getASTContext());
236 
237   // Only do this if the current method is -dealloc.
238   SVal SelfVal;
239   if (!isInInstanceDealloc(C, SelfVal))
240     return;
241 
242   SymbolRef SelfSymbol = SelfVal.getAsSymbol();
243 
244   const LocationContext *LCtx = C.getLocationContext();
245   ProgramStateRef InitialState = C.getState();
246 
247   ProgramStateRef State = InitialState;
248 
249   SymbolSet::Factory &F = State->getStateManager().get_context<SymbolSet>();
250 
251   // Symbols that must be released by the end of the -dealloc;
252   SymbolSet RequiredReleases = F.getEmptySet();
253 
254   // If we're an inlined -dealloc, we should add our symbols to the existing
255   // set from our subclass.
256   if (const SymbolSet *CurrSet = State->get<UnreleasedIvarMap>(SelfSymbol))
257     RequiredReleases = *CurrSet;
258 
259   for (auto *PropImpl : getContainingObjCImpl(LCtx)->property_impls()) {
260     ReleaseRequirement Requirement = getDeallocReleaseRequirement(PropImpl);
261     if (Requirement != ReleaseRequirement::MustRelease)
262       continue;
263 
264     SVal LVal = State->getLValue(PropImpl->getPropertyIvarDecl(), SelfVal);
265     Optional<Loc> LValLoc = LVal.getAs<Loc>();
266     if (!LValLoc)
267       continue;
268 
269     SVal InitialVal = State->getSVal(LValLoc.getValue());
270     SymbolRef Symbol = InitialVal.getAsSymbol();
271     if (!Symbol || !isa<SymbolRegionValue>(Symbol))
272       continue;
273 
274     // Mark the value as requiring a release.
275     RequiredReleases = F.add(RequiredReleases, Symbol);
276   }
277 
278   if (!RequiredReleases.isEmpty()) {
279     State = State->set<UnreleasedIvarMap>(SelfSymbol, RequiredReleases);
280   }
281 
282   if (State != InitialState) {
283     C.addTransition(State);
284   }
285 }
286 
287 /// Given a symbol for an ivar, return a symbol for the instance containing
288 /// the ivar. Returns nullptr if the instance symbol cannot be found.
289 SymbolRef
290 ObjCDeallocChecker::getInstanceSymbolFromIvarSymbol(SymbolRef IvarSym) const {
291   if (auto *SRV = dyn_cast<SymbolRegionValue>(IvarSym)) {
292     const TypedValueRegion *TVR = SRV->getRegion();
293     const ObjCIvarRegion *IvarRegion = dyn_cast_or_null<ObjCIvarRegion>(TVR);
294     if (!IvarRegion)
295       return nullptr;
296 
297     return IvarRegion->getSymbolicBase()->getSymbol();
298   }
299 
300   return nullptr;
301 }
302 
303 /// If we are in -dealloc or -dealloc is on the stack, handle the call if it is
304 /// a release or a nilling-out property setter.
305 void ObjCDeallocChecker::checkPreObjCMessage(
306     const ObjCMethodCall &M, CheckerContext &C) const {
307   // Only run if -dealloc is on the stack.
308   SVal DeallocedInstance;
309   if (!instanceDeallocIsOnStack(C, DeallocedInstance))
310     return;
311 
312   SymbolRef ReleasedValue = getValueExplicitlyReleased(M, C);
313 
314   if (ReleasedValue) {
315     // An instance variable symbol was released with -release:
316     //    [_property release];
317     if (diagnoseExtraRelease(ReleasedValue,M, C))
318       return;
319   } else {
320     // An instance variable symbol was released nilling out its property:
321     //    self.property = nil;
322     ReleasedValue = getValueReleasedByNillingOut(M, C);
323   }
324 
325   if (!ReleasedValue)
326     return;
327 
328   SymbolRef InstanceSym = getInstanceSymbolFromIvarSymbol(ReleasedValue);
329   if (!InstanceSym)
330     return;
331   ProgramStateRef InitialState = C.getState();
332 
333   ProgramStateRef ReleasedState =
334       removeValueRequiringRelease(InitialState, InstanceSym, ReleasedValue);
335 
336   if (ReleasedState != InitialState) {
337     C.addTransition(ReleasedState);
338   }
339 }
340 
341 /// If the message was a call to '[super dealloc]', diagnose any missing
342 /// releases.
343 void ObjCDeallocChecker::checkPostObjCMessage(
344     const ObjCMethodCall &M, CheckerContext &C) const {
345   // We perform this check post-message so that if the super -dealloc
346   // calls a helper method and that this class overrides, any ivars released in
347   // the helper method will be recorded before checking.
348   if (isSuperDeallocMessage(M))
349     diagnoseMissingReleases(C);
350 }
351 
352 /// Check for missing releases even when -dealloc does not call
353 /// '[super dealloc]'.
354 void ObjCDeallocChecker::checkEndFunction(
355     CheckerContext &C) const {
356   diagnoseMissingReleases(C);
357 }
358 
359 /// Check for missing releases on early return.
360 void ObjCDeallocChecker::checkPreStmt(
361     const ReturnStmt *RS, CheckerContext &C) const {
362   diagnoseMissingReleases(C);
363 }
364 
365 /// If a symbol escapes conservatively assume unseen code released it.
366 ProgramStateRef ObjCDeallocChecker::checkPointerEscape(
367     ProgramStateRef State, const InvalidatedSymbols &Escaped,
368     const CallEvent *Call, PointerEscapeKind Kind) const {
369 
370   // Don't treat calls to '[super dealloc]' as escaping for the purposes
371   // of this checker. Because the checker diagnoses missing releases in the
372   // post-message handler for '[super dealloc], escaping here would cause
373   // the checker to never warn.
374   auto *OMC = dyn_cast_or_null<ObjCMethodCall>(Call);
375   if (OMC && isSuperDeallocMessage(*OMC))
376     return State;
377 
378   for (const auto &Sym : Escaped) {
379     State = State->remove<UnreleasedIvarMap>(Sym);
380 
381     SymbolRef InstanceSymbol = getInstanceSymbolFromIvarSymbol(Sym);
382     if (!InstanceSymbol)
383       continue;
384 
385     State = removeValueRequiringRelease(State, InstanceSymbol, Sym);
386   }
387 
388   return State;
389 }
390 
391 /// Report any unreleased instance variables for the current instance being
392 /// dealloced.
393 void ObjCDeallocChecker::diagnoseMissingReleases(CheckerContext &C) const {
394   ProgramStateRef State = C.getState();
395 
396   SVal SelfVal;
397   if (!isInInstanceDealloc(C, SelfVal))
398     return;
399 
400   const MemRegion *SelfRegion = SelfVal.castAs<loc::MemRegionVal>().getRegion();
401   const LocationContext *LCtx = C.getLocationContext();
402 
403   ExplodedNode *ErrNode = nullptr;
404 
405   SymbolRef SelfSym = SelfVal.getAsSymbol();
406   if (!SelfSym)
407     return;
408 
409   const SymbolSet *OldUnreleased = State->get<UnreleasedIvarMap>(SelfSym);
410   if (!OldUnreleased)
411     return;
412 
413   SymbolSet NewUnreleased = *OldUnreleased;
414   SymbolSet::Factory &F = State->getStateManager().get_context<SymbolSet>();
415 
416   ProgramStateRef InitialState = State;
417 
418   for (auto *IvarSymbol : *OldUnreleased) {
419     const TypedValueRegion *TVR =
420         cast<SymbolRegionValue>(IvarSymbol)->getRegion();
421     const ObjCIvarRegion *IvarRegion = cast<ObjCIvarRegion>(TVR);
422 
423     // Don't warn if the ivar is not for this instance.
424     if (SelfRegion != IvarRegion->getSuperRegion())
425       continue;
426 
427     // Prevent an inlined call to -dealloc in a super class from warning
428     // about the values the subclass's -dealloc should release.
429     if (IvarRegion->getDecl()->getContainingInterface() !=
430         cast<ObjCMethodDecl>(LCtx->getDecl())->getClassInterface())
431       continue;
432 
433     // Prevents diagnosing multiple times for the same instance variable
434     // at, for example, both a return and at the end of of the function.
435     NewUnreleased = F.remove(NewUnreleased, IvarSymbol);
436 
437     if (State->getStateManager()
438             .getConstraintManager()
439             .isNull(State, IvarSymbol)
440             .isConstrainedTrue()) {
441       continue;
442     }
443 
444     // A missing release manifests as a leak, so treat as a non-fatal error.
445     if (!ErrNode)
446       ErrNode = C.generateNonFatalErrorNode();
447     // If we've already reached this node on another path, return without
448     // diagnosing.
449     if (!ErrNode)
450       return;
451 
452     std::string Buf;
453     llvm::raw_string_ostream OS(Buf);
454 
455     const ObjCIvarDecl *IvarDecl = IvarRegion->getDecl();
456     const ObjCInterfaceDecl *Interface = IvarDecl->getContainingInterface();
457     // If the class is known to have a lifecycle with teardown that is
458     // separate from -dealloc, do not warn about missing releases. We
459     // suppress here (rather than not tracking for instance variables in
460     // such classes) because these classes are rare.
461     if (classHasSeparateTeardown(Interface))
462       return;
463 
464     ObjCImplDecl *ImplDecl = Interface->getImplementation();
465 
466     const ObjCPropertyImplDecl *PropImpl =
467         ImplDecl->FindPropertyImplIvarDecl(IvarDecl->getIdentifier());
468 
469     const ObjCPropertyDecl *PropDecl = PropImpl->getPropertyDecl();
470 
471     assert(PropDecl->getSetterKind() == ObjCPropertyDecl::Copy ||
472            PropDecl->getSetterKind() == ObjCPropertyDecl::Retain);
473 
474     OS << "The '" << *IvarDecl << "' ivar in '" << *ImplDecl
475        << "' was ";
476 
477     if (PropDecl->getSetterKind() == ObjCPropertyDecl::Retain)
478       OS << "retained";
479     else
480       OS << "copied";
481 
482     OS << " by a synthesized property but not released"
483           " before '[super dealloc]'";
484 
485     std::unique_ptr<BugReport> BR(
486         new BugReport(*MissingReleaseBugType, OS.str(), ErrNode));
487 
488     C.emitReport(std::move(BR));
489   }
490 
491   if (NewUnreleased.isEmpty()) {
492     State = State->remove<UnreleasedIvarMap>(SelfSym);
493   } else {
494     State = State->set<UnreleasedIvarMap>(SelfSym, NewUnreleased);
495   }
496 
497   if (ErrNode) {
498     C.addTransition(State, ErrNode);
499   } else if (State != InitialState) {
500     C.addTransition(State);
501   }
502 
503   // Make sure that after checking in the top-most frame the list of
504   // tracked ivars is empty. This is intended to detect accidental leaks in
505   // the UnreleasedIvarMap program state.
506   assert(!LCtx->inTopFrame() || State->get<UnreleasedIvarMap>().isEmpty());
507 }
508 
509 /// Emits a warning if the current context is -dealloc and ReleasedValue
510 /// must not be directly released in a -dealloc. Returns true if a diagnostic
511 /// was emitted.
512 bool ObjCDeallocChecker::diagnoseExtraRelease(SymbolRef ReleasedValue,
513                                               const ObjCMethodCall &M,
514                                               CheckerContext &C) const {
515   SVal DeallocedInstance;
516   if (!isInInstanceDealloc(C, DeallocedInstance))
517     return false;
518 
519   // Try to get the region from which the the released value was loaded.
520   // Note that, unlike diagnosing for missing releases, here we don't track
521   // values that must not be released in the state. This is because even if
522   // these values escape, it is still an error under the rules of MRR to
523   // release them in -dealloc.
524   const MemRegion *RegionLoadedFrom = nullptr;
525   if (auto *DerivedSym = dyn_cast<SymbolDerived>(ReleasedValue))
526     RegionLoadedFrom = DerivedSym->getRegion();
527   else if (auto *RegionSym = dyn_cast<SymbolRegionValue>(ReleasedValue))
528     RegionLoadedFrom = RegionSym->getRegion();
529   else
530     return false;
531 
532   auto *ReleasedIvar = dyn_cast<ObjCIvarRegion>(RegionLoadedFrom);
533   if (!ReleasedIvar)
534     return false;
535 
536   if (DeallocedInstance.castAs<loc::MemRegionVal>().getRegion() !=
537       ReleasedIvar->getSuperRegion())
538     return false;
539 
540   const LocationContext *LCtx = C.getLocationContext();
541   const ObjCIvarDecl *ReleasedIvarDecl = ReleasedIvar->getDecl();
542 
543   // If the ivar belongs to a property that must not be released directly
544   // in dealloc, emit a warning.
545   const ObjCImplDecl *Container = getContainingObjCImpl(LCtx);
546   const ObjCPropertyImplDecl *PropImpl =
547       Container->FindPropertyImplIvarDecl(ReleasedIvarDecl->getIdentifier());
548   if (!PropImpl)
549     return false;
550 
551   if (getDeallocReleaseRequirement(PropImpl) !=
552       ReleaseRequirement::MustNotReleaseDirectly) {
553     return false;
554   }
555 
556   // If the property is readwrite but it shadows a read-only property in its
557   // external interface, treat the property a read-only. If the outside
558   // world cannot write to a property then the internal implementation is free
559   // to make its own convention about whether the value is stored retained
560   // or not. We look up the shadow here rather than in
561   // getDeallocReleaseRequirement() because doing so can be expensive.
562   const ObjCPropertyDecl *PropDecl = findShadowedPropertyDecl(PropImpl);
563   if (PropDecl) {
564     if (PropDecl->isReadOnly())
565       return false;
566   } else {
567     PropDecl = PropImpl->getPropertyDecl();
568   }
569 
570   ExplodedNode *ErrNode = C.generateNonFatalErrorNode();
571   if (!ErrNode)
572     return false;
573 
574   std::string Buf;
575   llvm::raw_string_ostream OS(Buf);
576 
577   assert(PropDecl->getSetterKind() == ObjCPropertyDecl::Weak ||
578          (PropDecl->getSetterKind() == ObjCPropertyDecl::Assign &&
579           !PropDecl->isReadOnly()));
580 
581   OS << "The '" << *PropImpl->getPropertyIvarDecl()
582      << "' ivar in '" << *Container
583      << "' was synthesized for ";
584 
585   if (PropDecl->getSetterKind() == ObjCPropertyDecl::Weak)
586     OS << "a weak";
587   else
588     OS << "an assign, readwrite";
589 
590   OS <<  " property but was released in 'dealloc'";
591 
592   std::unique_ptr<BugReport> BR(
593       new BugReport(*ExtraReleaseBugType, OS.str(), ErrNode));
594   BR->addRange(M.getOriginExpr()->getSourceRange());
595 
596   C.emitReport(std::move(BR));
597 
598   return true;
599 }
600 
601 
602 ObjCDeallocChecker::
603     ObjCDeallocChecker()
604     : NSObjectII(nullptr), SenTestCaseII(nullptr) {
605 
606   MissingReleaseBugType.reset(
607       new BugType(this, "Missing ivar release (leak)",
608                   categories::MemoryCoreFoundationObjectiveC));
609 
610   ExtraReleaseBugType.reset(
611       new BugType(this, "Extra ivar release",
612                   categories::MemoryCoreFoundationObjectiveC));
613 }
614 
615 void ObjCDeallocChecker::initIdentifierInfoAndSelectors(
616     ASTContext &Ctx) const {
617   if (NSObjectII)
618     return;
619 
620   NSObjectII = &Ctx.Idents.get("NSObject");
621   SenTestCaseII = &Ctx.Idents.get("SenTestCase");
622 
623   IdentifierInfo *DeallocII = &Ctx.Idents.get("dealloc");
624   IdentifierInfo *ReleaseII = &Ctx.Idents.get("release");
625   DeallocSel = Ctx.Selectors.getSelector(0, &DeallocII);
626   ReleaseSel = Ctx.Selectors.getSelector(0, &ReleaseII);
627 }
628 
629 /// Returns true if M is a call to '[super dealloc]'.
630 bool ObjCDeallocChecker::isSuperDeallocMessage(
631     const ObjCMethodCall &M) const {
632   if (M.getOriginExpr()->getReceiverKind() != ObjCMessageExpr::SuperInstance)
633     return false;
634 
635   return M.getSelector() == DeallocSel;
636 }
637 
638 /// Returns the ObjCImplDecl containing the method declaration in LCtx.
639 const ObjCImplDecl *
640 ObjCDeallocChecker::getContainingObjCImpl(const LocationContext *LCtx) const {
641   auto *MD = cast<ObjCMethodDecl>(LCtx->getDecl());
642   return cast<ObjCImplDecl>(MD->getDeclContext());
643 }
644 
645 /// Returns the property that shadowed by PropImpl if one exists and
646 /// nullptr otherwise.
647 const ObjCPropertyDecl *ObjCDeallocChecker::findShadowedPropertyDecl(
648     const ObjCPropertyImplDecl *PropImpl) const {
649   const ObjCPropertyDecl *PropDecl = PropImpl->getPropertyDecl();
650 
651   // Only readwrite properties can shadow.
652   if (PropDecl->isReadOnly())
653     return nullptr;
654 
655   auto *CatDecl = dyn_cast<ObjCCategoryDecl>(PropDecl->getDeclContext());
656 
657   // Only class extensions can contain shadowing properties.
658   if (!CatDecl || !CatDecl->IsClassExtension())
659     return nullptr;
660 
661   IdentifierInfo *ID = PropDecl->getIdentifier();
662   DeclContext::lookup_result R = CatDecl->getClassInterface()->lookup(ID);
663   for (DeclContext::lookup_iterator I = R.begin(), E = R.end(); I != E; ++I) {
664     auto *ShadowedPropDecl = dyn_cast<ObjCPropertyDecl>(*I);
665     if (!ShadowedPropDecl)
666       continue;
667 
668     if (ShadowedPropDecl->isInstanceProperty()) {
669       assert(ShadowedPropDecl->isReadOnly());
670       return ShadowedPropDecl;
671     }
672   }
673 
674   return nullptr;
675 }
676 
677 /// Remove the Value requiring a release from the tracked set for
678 /// Instance and return the resultant state.
679 ProgramStateRef ObjCDeallocChecker::removeValueRequiringRelease(
680     ProgramStateRef State, SymbolRef Instance, SymbolRef Value) const {
681   assert(Instance);
682   assert(Value);
683 
684   const SymbolSet *Unreleased = State->get<UnreleasedIvarMap>(Instance);
685   if (!Unreleased)
686     return State;
687 
688   // Mark the value as no longer requiring a release.
689   SymbolSet::Factory &F = State->getStateManager().get_context<SymbolSet>();
690   SymbolSet NewUnreleased = F.remove(*Unreleased, Value);
691 
692   if (NewUnreleased.isEmpty()) {
693     return State->remove<UnreleasedIvarMap>(Instance);
694   }
695 
696   return State->set<UnreleasedIvarMap>(Instance, NewUnreleased);
697 }
698 
699 /// Determines whether the instance variable for \p PropImpl must or must not be
700 /// released in -dealloc or whether it cannot be determined.
701 ReleaseRequirement ObjCDeallocChecker::getDeallocReleaseRequirement(
702     const ObjCPropertyImplDecl *PropImpl) const {
703   const ObjCIvarDecl *IvarDecl;
704   const ObjCPropertyDecl *PropDecl;
705   if (!isSynthesizedRetainableProperty(PropImpl, &IvarDecl, &PropDecl))
706     return ReleaseRequirement::Unknown;
707 
708   ObjCPropertyDecl::SetterKind SK = PropDecl->getSetterKind();
709 
710   switch (SK) {
711   // Retain and copy setters retain/copy their values before storing and so
712   // the value in their instance variables must be released in -dealloc.
713   case ObjCPropertyDecl::Retain:
714   case ObjCPropertyDecl::Copy:
715     return ReleaseRequirement::MustRelease;
716 
717   case ObjCPropertyDecl::Weak:
718     return ReleaseRequirement::MustNotReleaseDirectly;
719 
720   case ObjCPropertyDecl::Assign:
721     // It is common for the ivars for read-only assign properties to
722     // always be stored retained, so their release requirement cannot be
723     // be determined.
724     if (PropDecl->isReadOnly())
725       return ReleaseRequirement::Unknown;
726 
727     return ReleaseRequirement::MustNotReleaseDirectly;
728   }
729   llvm_unreachable("Unrecognized setter kind");
730 }
731 
732 /// Returns the released value if M is a call to -release. Returns
733 /// nullptr otherwise.
734 SymbolRef
735 ObjCDeallocChecker::getValueExplicitlyReleased(const ObjCMethodCall &M,
736                                               CheckerContext &C) const {
737   if (M.getSelector() != ReleaseSel)
738     return nullptr;
739 
740   return M.getReceiverSVal().getAsSymbol();
741 }
742 
743 /// Returns the released value if M is a call a setter that releases
744 /// and nils out its underlying instance variable.
745 SymbolRef
746 ObjCDeallocChecker::getValueReleasedByNillingOut(const ObjCMethodCall &M,
747                                                  CheckerContext &C) const {
748   SVal ReceiverVal = M.getReceiverSVal();
749   if (!ReceiverVal.isValid())
750     return nullptr;
751 
752   // Is the first argument nil?
753   if (M.getNumArgs() == 0)
754     return nullptr;
755   SVal Arg = M.getArgSVal(0);
756   ProgramStateRef notNilState, nilState;
757   std::tie(notNilState, nilState) =
758       M.getState()->assume(Arg.castAs<DefinedOrUnknownSVal>());
759   if (!(nilState && !notNilState))
760     return nullptr;
761 
762   const ObjCPropertyDecl *Prop = M.getAccessedProperty();
763   if (!Prop)
764     return nullptr;
765 
766   ObjCIvarDecl *PropIvarDecl = Prop->getPropertyIvarDecl();
767   if (!PropIvarDecl)
768     return nullptr;
769 
770   ProgramStateRef State = C.getState();
771 
772   SVal LVal = State->getLValue(PropIvarDecl, ReceiverVal);
773   Optional<Loc> LValLoc = LVal.getAs<Loc>();
774   if (!LValLoc)
775     return nullptr;
776 
777   SVal CurrentValInIvar = State->getSVal(LValLoc.getValue());
778   return CurrentValInIvar.getAsSymbol();
779 }
780 
781 /// Returns true if the current context is a call to -dealloc and false
782 /// otherwise. If true, it also sets SelfValOut to the value of
783 /// 'self'.
784 bool ObjCDeallocChecker::isInInstanceDealloc(const CheckerContext &C,
785                                              SVal &SelfValOut) const {
786   return isInInstanceDealloc(C, C.getLocationContext(), SelfValOut);
787 }
788 
789 /// Returns true if LCtx is a call to -dealloc and false
790 /// otherwise. If true, it also sets SelfValOut to the value of
791 /// 'self'.
792 bool ObjCDeallocChecker::isInInstanceDealloc(const CheckerContext &C,
793                                              const LocationContext *LCtx,
794                                              SVal &SelfValOut) const {
795   auto *MD = dyn_cast<ObjCMethodDecl>(LCtx->getDecl());
796   if (!MD || !MD->isInstanceMethod() || MD->getSelector() != DeallocSel)
797     return false;
798 
799   const ImplicitParamDecl *SelfDecl = LCtx->getSelfDecl();
800   assert(SelfDecl && "No self in -dealloc?");
801 
802   ProgramStateRef State = C.getState();
803   SelfValOut = State->getSVal(State->getRegion(SelfDecl, LCtx));
804   return true;
805 }
806 
807 /// Returns true if there is a call to -dealloc anywhere on the stack and false
808 /// otherwise. If true, it also sets InstanceValOut to the value of
809 /// 'self' in the frame for -dealloc.
810 bool ObjCDeallocChecker::instanceDeallocIsOnStack(const CheckerContext &C,
811                                                   SVal &InstanceValOut) const {
812   const LocationContext *LCtx = C.getLocationContext();
813 
814   while (LCtx) {
815     if (isInInstanceDealloc(C, LCtx, InstanceValOut))
816       return true;
817 
818     LCtx = LCtx->getParent();
819   }
820 
821   return false;
822 }
823 
824 /// Returns true if the ID is a class in which which is known to have
825 /// a separate teardown lifecycle. In this case, -dealloc warnings
826 /// about missing releases should be suppressed.
827 bool ObjCDeallocChecker::classHasSeparateTeardown(
828     const ObjCInterfaceDecl *ID) const {
829   // Suppress if the class is not a subclass of NSObject.
830   for ( ; ID ; ID = ID->getSuperClass()) {
831     IdentifierInfo *II = ID->getIdentifier();
832 
833     if (II == NSObjectII)
834       return false;
835 
836     // FIXME: For now, ignore classes that subclass SenTestCase, as these don't
837     // need to implement -dealloc.  They implement tear down in another way,
838     // which we should try and catch later.
839     //  http://llvm.org/bugs/show_bug.cgi?id=3187
840     if (II == SenTestCaseII)
841       return true;
842   }
843 
844   return true;
845 }
846 
847 void ento::registerObjCDeallocChecker(CheckerManager &Mgr) {
848   const LangOptions &LangOpts = Mgr.getLangOpts();
849   // These checker only makes sense under MRR.
850   if (LangOpts.getGC() == LangOptions::GCOnly || LangOpts.ObjCAutoRefCount)
851     return;
852 
853   Mgr.registerChecker<ObjCDeallocChecker>();
854 }
855