1 //==- CheckObjCDealloc.cpp - Check ObjC -dealloc implementation --*- C++ -*-==//
2 //
3 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4 // See https://llvm.org/LICENSE.txt for license information.
5 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6 //
7 //===----------------------------------------------------------------------===//
8 //
9 // This checker analyzes Objective-C -dealloc methods and their callees
10 // to warn about improper releasing of instance variables that back synthesized
11 // properties. It warns about missing releases in the following cases:
12 // - When a class has a synthesized instance variable for a 'retain' or 'copy'
13 // property and lacks a -dealloc method in its implementation.
14 // - When a class has a synthesized instance variable for a 'retain'/'copy'
15 // property but the ivar is not released in -dealloc by either -release
16 // or by nilling out the property.
17 //
18 // It warns about extra releases in -dealloc (but not in callees) when a
19 // synthesized instance variable is released in the following cases:
20 // - When the property is 'assign' and is not 'readonly'.
21 // - When the property is 'weak'.
22 //
23 // This checker only warns for instance variables synthesized to back
24 // properties. Handling the more general case would require inferring whether
25 // an instance variable is stored retained or not. For synthesized properties,
26 // this is specified in the property declaration itself.
27 //
28 //===----------------------------------------------------------------------===//
29
30 #include "clang/StaticAnalyzer/Checkers/BuiltinCheckerRegistration.h"
31 #include "clang/Analysis/PathDiagnostic.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/Basic/TargetInfo.h"
38 #include "clang/StaticAnalyzer/Core/BugReporter/BugReporter.h"
39 #include "clang/StaticAnalyzer/Core/BugReporter/BugType.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 #include <optional>
49
50 using namespace clang;
51 using namespace ento;
52
53 /// Indicates whether an instance variable is required to be released in
54 /// -dealloc.
55 enum class ReleaseRequirement {
56 /// The instance variable must be released, either by calling
57 /// -release on it directly or by nilling it out with a property setter.
58 MustRelease,
59
60 /// The instance variable must not be directly released with -release.
61 MustNotReleaseDirectly,
62
63 /// The requirement for the instance variable could not be determined.
64 Unknown
65 };
66
67 /// Returns true if the property implementation is synthesized and the
68 /// type of the property is retainable.
isSynthesizedRetainableProperty(const ObjCPropertyImplDecl * I,const ObjCIvarDecl ** ID,const ObjCPropertyDecl ** PD)69 static bool isSynthesizedRetainableProperty(const ObjCPropertyImplDecl *I,
70 const ObjCIvarDecl **ID,
71 const ObjCPropertyDecl **PD) {
72
73 if (I->getPropertyImplementation() != ObjCPropertyImplDecl::Synthesize)
74 return false;
75
76 (*ID) = I->getPropertyIvarDecl();
77 if (!(*ID))
78 return false;
79
80 QualType T = (*ID)->getType();
81 if (!T->isObjCRetainableType())
82 return false;
83
84 (*PD) = I->getPropertyDecl();
85 // Shouldn't be able to synthesize a property that doesn't exist.
86 assert(*PD);
87
88 return true;
89 }
90
91 namespace {
92
93 class ObjCDeallocChecker
94 : public Checker<check::ASTDecl<ObjCImplementationDecl>,
95 check::PreObjCMessage, check::PostObjCMessage,
96 check::PreCall,
97 check::BeginFunction, check::EndFunction,
98 eval::Assume,
99 check::PointerEscape,
100 check::PreStmt<ReturnStmt>> {
101
102 mutable IdentifierInfo *NSObjectII, *SenTestCaseII, *XCTestCaseII,
103 *Block_releaseII, *CIFilterII;
104
105 mutable Selector DeallocSel, ReleaseSel;
106
107 std::unique_ptr<BugType> MissingReleaseBugType;
108 std::unique_ptr<BugType> ExtraReleaseBugType;
109 std::unique_ptr<BugType> MistakenDeallocBugType;
110
111 public:
112 ObjCDeallocChecker();
113
114 void checkASTDecl(const ObjCImplementationDecl *D, AnalysisManager& Mgr,
115 BugReporter &BR) const;
116 void checkBeginFunction(CheckerContext &Ctx) const;
117 void checkPreObjCMessage(const ObjCMethodCall &M, CheckerContext &C) const;
118 void checkPreCall(const CallEvent &Call, CheckerContext &C) const;
119 void checkPostObjCMessage(const ObjCMethodCall &M, CheckerContext &C) const;
120
121 ProgramStateRef evalAssume(ProgramStateRef State, SVal Cond,
122 bool Assumption) const;
123
124 ProgramStateRef checkPointerEscape(ProgramStateRef State,
125 const InvalidatedSymbols &Escaped,
126 const CallEvent *Call,
127 PointerEscapeKind Kind) const;
128 void checkPreStmt(const ReturnStmt *RS, CheckerContext &C) const;
129 void checkEndFunction(const ReturnStmt *RS, CheckerContext &Ctx) const;
130
131 private:
132 void diagnoseMissingReleases(CheckerContext &C) const;
133
134 bool diagnoseExtraRelease(SymbolRef ReleasedValue, const ObjCMethodCall &M,
135 CheckerContext &C) const;
136
137 bool diagnoseMistakenDealloc(SymbolRef DeallocedValue,
138 const ObjCMethodCall &M,
139 CheckerContext &C) const;
140
141 SymbolRef getValueReleasedByNillingOut(const ObjCMethodCall &M,
142 CheckerContext &C) const;
143
144 const ObjCIvarRegion *getIvarRegionForIvarSymbol(SymbolRef IvarSym) const;
145 SymbolRef getInstanceSymbolFromIvarSymbol(SymbolRef IvarSym) const;
146
147 const ObjCPropertyImplDecl*
148 findPropertyOnDeallocatingInstance(SymbolRef IvarSym,
149 CheckerContext &C) const;
150
151 ReleaseRequirement
152 getDeallocReleaseRequirement(const ObjCPropertyImplDecl *PropImpl) const;
153
154 bool isInInstanceDealloc(const CheckerContext &C, SVal &SelfValOut) const;
155 bool isInInstanceDealloc(const CheckerContext &C, const LocationContext *LCtx,
156 SVal &SelfValOut) const;
157 bool instanceDeallocIsOnStack(const CheckerContext &C,
158 SVal &InstanceValOut) const;
159
160 bool isSuperDeallocMessage(const ObjCMethodCall &M) const;
161
162 const ObjCImplDecl *getContainingObjCImpl(const LocationContext *LCtx) const;
163
164 const ObjCPropertyDecl *
165 findShadowedPropertyDecl(const ObjCPropertyImplDecl *PropImpl) const;
166
167 void transitionToReleaseValue(CheckerContext &C, SymbolRef Value) const;
168 ProgramStateRef removeValueRequiringRelease(ProgramStateRef State,
169 SymbolRef InstanceSym,
170 SymbolRef ValueSym) const;
171
172 void initIdentifierInfoAndSelectors(ASTContext &Ctx) const;
173
174 bool classHasSeparateTeardown(const ObjCInterfaceDecl *ID) const;
175
176 bool isReleasedByCIFilterDealloc(const ObjCPropertyImplDecl *PropImpl) const;
177 bool isNibLoadedIvarWithoutRetain(const ObjCPropertyImplDecl *PropImpl) const;
178 };
179 } // End anonymous namespace.
180
181
182 /// Maps from the symbol for a class instance to the set of
183 /// symbols remaining that must be released in -dealloc.
REGISTER_SET_FACTORY_WITH_PROGRAMSTATE(SymbolSet,SymbolRef)184 REGISTER_SET_FACTORY_WITH_PROGRAMSTATE(SymbolSet, SymbolRef)
185 REGISTER_MAP_WITH_PROGRAMSTATE(UnreleasedIvarMap, SymbolRef, SymbolSet)
186
187
188 /// An AST check that diagnose when the class requires a -dealloc method and
189 /// is missing one.
190 void ObjCDeallocChecker::checkASTDecl(const ObjCImplementationDecl *D,
191 AnalysisManager &Mgr,
192 BugReporter &BR) const {
193 assert(Mgr.getLangOpts().getGC() != LangOptions::GCOnly);
194 assert(!Mgr.getLangOpts().ObjCAutoRefCount);
195 initIdentifierInfoAndSelectors(Mgr.getASTContext());
196
197 const ObjCInterfaceDecl *ID = D->getClassInterface();
198 // If the class is known to have a lifecycle with a separate teardown method
199 // then it may not require a -dealloc method.
200 if (classHasSeparateTeardown(ID))
201 return;
202
203 // Does the class contain any synthesized properties that are retainable?
204 // If not, skip the check entirely.
205 const ObjCPropertyImplDecl *PropImplRequiringRelease = nullptr;
206 bool HasOthers = false;
207 for (const auto *I : D->property_impls()) {
208 if (getDeallocReleaseRequirement(I) == ReleaseRequirement::MustRelease) {
209 if (!PropImplRequiringRelease)
210 PropImplRequiringRelease = I;
211 else {
212 HasOthers = true;
213 break;
214 }
215 }
216 }
217
218 if (!PropImplRequiringRelease)
219 return;
220
221 const ObjCMethodDecl *MD = nullptr;
222
223 // Scan the instance methods for "dealloc".
224 for (const auto *I : D->instance_methods()) {
225 if (I->getSelector() == DeallocSel) {
226 MD = I;
227 break;
228 }
229 }
230
231 if (!MD) { // No dealloc found.
232 const char* Name = "Missing -dealloc";
233
234 std::string Buf;
235 llvm::raw_string_ostream OS(Buf);
236 OS << "'" << *D << "' lacks a 'dealloc' instance method but "
237 << "must release '" << *PropImplRequiringRelease->getPropertyIvarDecl()
238 << "'";
239
240 if (HasOthers)
241 OS << " and others";
242 PathDiagnosticLocation DLoc =
243 PathDiagnosticLocation::createBegin(D, BR.getSourceManager());
244
245 BR.EmitBasicReport(D, this, Name, categories::CoreFoundationObjectiveC,
246 OS.str(), DLoc);
247 return;
248 }
249 }
250
251 /// If this is the beginning of -dealloc, mark the values initially stored in
252 /// instance variables that must be released by the end of -dealloc
253 /// as unreleased in the state.
checkBeginFunction(CheckerContext & C) const254 void ObjCDeallocChecker::checkBeginFunction(
255 CheckerContext &C) const {
256 initIdentifierInfoAndSelectors(C.getASTContext());
257
258 // Only do this if the current method is -dealloc.
259 SVal SelfVal;
260 if (!isInInstanceDealloc(C, SelfVal))
261 return;
262
263 SymbolRef SelfSymbol = SelfVal.getAsSymbol();
264
265 const LocationContext *LCtx = C.getLocationContext();
266 ProgramStateRef InitialState = C.getState();
267
268 ProgramStateRef State = InitialState;
269
270 SymbolSet::Factory &F = State->getStateManager().get_context<SymbolSet>();
271
272 // Symbols that must be released by the end of the -dealloc;
273 SymbolSet RequiredReleases = F.getEmptySet();
274
275 // If we're an inlined -dealloc, we should add our symbols to the existing
276 // set from our subclass.
277 if (const SymbolSet *CurrSet = State->get<UnreleasedIvarMap>(SelfSymbol))
278 RequiredReleases = *CurrSet;
279
280 for (auto *PropImpl : getContainingObjCImpl(LCtx)->property_impls()) {
281 ReleaseRequirement Requirement = getDeallocReleaseRequirement(PropImpl);
282 if (Requirement != ReleaseRequirement::MustRelease)
283 continue;
284
285 SVal LVal = State->getLValue(PropImpl->getPropertyIvarDecl(), SelfVal);
286 std::optional<Loc> LValLoc = LVal.getAs<Loc>();
287 if (!LValLoc)
288 continue;
289
290 SVal InitialVal = State->getSVal(*LValLoc);
291 SymbolRef Symbol = InitialVal.getAsSymbol();
292 if (!Symbol || !isa<SymbolRegionValue>(Symbol))
293 continue;
294
295 // Mark the value as requiring a release.
296 RequiredReleases = F.add(RequiredReleases, Symbol);
297 }
298
299 if (!RequiredReleases.isEmpty()) {
300 State = State->set<UnreleasedIvarMap>(SelfSymbol, RequiredReleases);
301 }
302
303 if (State != InitialState) {
304 C.addTransition(State);
305 }
306 }
307
308 /// Given a symbol for an ivar, return the ivar region it was loaded from.
309 /// Returns nullptr if the instance symbol cannot be found.
310 const ObjCIvarRegion *
getIvarRegionForIvarSymbol(SymbolRef IvarSym) const311 ObjCDeallocChecker::getIvarRegionForIvarSymbol(SymbolRef IvarSym) const {
312 return dyn_cast_or_null<ObjCIvarRegion>(IvarSym->getOriginRegion());
313 }
314
315 /// Given a symbol for an ivar, return a symbol for the instance containing
316 /// the ivar. Returns nullptr if the instance symbol cannot be found.
317 SymbolRef
getInstanceSymbolFromIvarSymbol(SymbolRef IvarSym) const318 ObjCDeallocChecker::getInstanceSymbolFromIvarSymbol(SymbolRef IvarSym) const {
319
320 const ObjCIvarRegion *IvarRegion = getIvarRegionForIvarSymbol(IvarSym);
321 if (!IvarRegion)
322 return nullptr;
323
324 return IvarRegion->getSymbolicBase()->getSymbol();
325 }
326
327 /// If we are in -dealloc or -dealloc is on the stack, handle the call if it is
328 /// a release or a nilling-out property setter.
checkPreObjCMessage(const ObjCMethodCall & M,CheckerContext & C) const329 void ObjCDeallocChecker::checkPreObjCMessage(
330 const ObjCMethodCall &M, CheckerContext &C) const {
331 // Only run if -dealloc is on the stack.
332 SVal DeallocedInstance;
333 if (!instanceDeallocIsOnStack(C, DeallocedInstance))
334 return;
335
336 SymbolRef ReleasedValue = nullptr;
337
338 if (M.getSelector() == ReleaseSel) {
339 ReleasedValue = M.getReceiverSVal().getAsSymbol();
340 } else if (M.getSelector() == DeallocSel && !M.isReceiverSelfOrSuper()) {
341 if (diagnoseMistakenDealloc(M.getReceiverSVal().getAsSymbol(), M, C))
342 return;
343 }
344
345 if (ReleasedValue) {
346 // An instance variable symbol was released with -release:
347 // [_property release];
348 if (diagnoseExtraRelease(ReleasedValue,M, C))
349 return;
350 } else {
351 // An instance variable symbol was released nilling out its property:
352 // self.property = nil;
353 ReleasedValue = getValueReleasedByNillingOut(M, C);
354 }
355
356 if (!ReleasedValue)
357 return;
358
359 transitionToReleaseValue(C, ReleasedValue);
360 }
361
362 /// If we are in -dealloc or -dealloc is on the stack, handle the call if it is
363 /// call to Block_release().
checkPreCall(const CallEvent & Call,CheckerContext & C) const364 void ObjCDeallocChecker::checkPreCall(const CallEvent &Call,
365 CheckerContext &C) const {
366 const IdentifierInfo *II = Call.getCalleeIdentifier();
367 if (II != Block_releaseII)
368 return;
369
370 if (Call.getNumArgs() != 1)
371 return;
372
373 SymbolRef ReleasedValue = Call.getArgSVal(0).getAsSymbol();
374 if (!ReleasedValue)
375 return;
376
377 transitionToReleaseValue(C, ReleasedValue);
378 }
379 /// If the message was a call to '[super dealloc]', diagnose any missing
380 /// releases.
checkPostObjCMessage(const ObjCMethodCall & M,CheckerContext & C) const381 void ObjCDeallocChecker::checkPostObjCMessage(
382 const ObjCMethodCall &M, CheckerContext &C) const {
383 // We perform this check post-message so that if the super -dealloc
384 // calls a helper method and that this class overrides, any ivars released in
385 // the helper method will be recorded before checking.
386 if (isSuperDeallocMessage(M))
387 diagnoseMissingReleases(C);
388 }
389
390 /// Check for missing releases even when -dealloc does not call
391 /// '[super dealloc]'.
checkEndFunction(const ReturnStmt * RS,CheckerContext & C) const392 void ObjCDeallocChecker::checkEndFunction(
393 const ReturnStmt *RS, CheckerContext &C) const {
394 diagnoseMissingReleases(C);
395 }
396
397 /// Check for missing releases on early return.
checkPreStmt(const ReturnStmt * RS,CheckerContext & C) const398 void ObjCDeallocChecker::checkPreStmt(
399 const ReturnStmt *RS, CheckerContext &C) const {
400 diagnoseMissingReleases(C);
401 }
402
403 /// When a symbol is assumed to be nil, remove it from the set of symbols
404 /// require to be nil.
evalAssume(ProgramStateRef State,SVal Cond,bool Assumption) const405 ProgramStateRef ObjCDeallocChecker::evalAssume(ProgramStateRef State, SVal Cond,
406 bool Assumption) const {
407 if (State->get<UnreleasedIvarMap>().isEmpty())
408 return State;
409
410 auto *CondBSE = dyn_cast_or_null<BinarySymExpr>(Cond.getAsSymbol());
411 if (!CondBSE)
412 return State;
413
414 BinaryOperator::Opcode OpCode = CondBSE->getOpcode();
415 if (Assumption) {
416 if (OpCode != BO_EQ)
417 return State;
418 } else {
419 if (OpCode != BO_NE)
420 return State;
421 }
422
423 SymbolRef NullSymbol = nullptr;
424 if (auto *SIE = dyn_cast<SymIntExpr>(CondBSE)) {
425 const llvm::APInt &RHS = SIE->getRHS();
426 if (RHS != 0)
427 return State;
428 NullSymbol = SIE->getLHS();
429 } else if (auto *SIE = dyn_cast<IntSymExpr>(CondBSE)) {
430 const llvm::APInt &LHS = SIE->getLHS();
431 if (LHS != 0)
432 return State;
433 NullSymbol = SIE->getRHS();
434 } else {
435 return State;
436 }
437
438 SymbolRef InstanceSymbol = getInstanceSymbolFromIvarSymbol(NullSymbol);
439 if (!InstanceSymbol)
440 return State;
441
442 State = removeValueRequiringRelease(State, InstanceSymbol, NullSymbol);
443
444 return State;
445 }
446
447 /// If a symbol escapes conservatively assume unseen code released it.
checkPointerEscape(ProgramStateRef State,const InvalidatedSymbols & Escaped,const CallEvent * Call,PointerEscapeKind Kind) const448 ProgramStateRef ObjCDeallocChecker::checkPointerEscape(
449 ProgramStateRef State, const InvalidatedSymbols &Escaped,
450 const CallEvent *Call, PointerEscapeKind Kind) const {
451
452 if (State->get<UnreleasedIvarMap>().isEmpty())
453 return State;
454
455 // Don't treat calls to '[super dealloc]' as escaping for the purposes
456 // of this checker. Because the checker diagnoses missing releases in the
457 // post-message handler for '[super dealloc], escaping here would cause
458 // the checker to never warn.
459 auto *OMC = dyn_cast_or_null<ObjCMethodCall>(Call);
460 if (OMC && isSuperDeallocMessage(*OMC))
461 return State;
462
463 for (const auto &Sym : Escaped) {
464 if (!Call || (Call && !Call->isInSystemHeader())) {
465 // If Sym is a symbol for an object with instance variables that
466 // must be released, remove these obligations when the object escapes
467 // unless via a call to a system function. System functions are
468 // very unlikely to release instance variables on objects passed to them,
469 // and are frequently called on 'self' in -dealloc (e.g., to remove
470 // observers) -- we want to avoid false negatives from escaping on
471 // them.
472 State = State->remove<UnreleasedIvarMap>(Sym);
473 }
474
475
476 SymbolRef InstanceSymbol = getInstanceSymbolFromIvarSymbol(Sym);
477 if (!InstanceSymbol)
478 continue;
479
480 State = removeValueRequiringRelease(State, InstanceSymbol, Sym);
481 }
482
483 return State;
484 }
485
486 /// Report any unreleased instance variables for the current instance being
487 /// dealloced.
diagnoseMissingReleases(CheckerContext & C) const488 void ObjCDeallocChecker::diagnoseMissingReleases(CheckerContext &C) const {
489 ProgramStateRef State = C.getState();
490
491 SVal SelfVal;
492 if (!isInInstanceDealloc(C, SelfVal))
493 return;
494
495 const MemRegion *SelfRegion = SelfVal.castAs<loc::MemRegionVal>().getRegion();
496 const LocationContext *LCtx = C.getLocationContext();
497
498 ExplodedNode *ErrNode = nullptr;
499
500 SymbolRef SelfSym = SelfVal.getAsSymbol();
501 if (!SelfSym)
502 return;
503
504 const SymbolSet *OldUnreleased = State->get<UnreleasedIvarMap>(SelfSym);
505 if (!OldUnreleased)
506 return;
507
508 SymbolSet NewUnreleased = *OldUnreleased;
509 SymbolSet::Factory &F = State->getStateManager().get_context<SymbolSet>();
510
511 ProgramStateRef InitialState = State;
512
513 for (auto *IvarSymbol : *OldUnreleased) {
514 const TypedValueRegion *TVR =
515 cast<SymbolRegionValue>(IvarSymbol)->getRegion();
516 const ObjCIvarRegion *IvarRegion = cast<ObjCIvarRegion>(TVR);
517
518 // Don't warn if the ivar is not for this instance.
519 if (SelfRegion != IvarRegion->getSuperRegion())
520 continue;
521
522 const ObjCIvarDecl *IvarDecl = IvarRegion->getDecl();
523 // Prevent an inlined call to -dealloc in a super class from warning
524 // about the values the subclass's -dealloc should release.
525 if (IvarDecl->getContainingInterface() !=
526 cast<ObjCMethodDecl>(LCtx->getDecl())->getClassInterface())
527 continue;
528
529 // Prevents diagnosing multiple times for the same instance variable
530 // at, for example, both a return and at the end of the function.
531 NewUnreleased = F.remove(NewUnreleased, IvarSymbol);
532
533 if (State->getStateManager()
534 .getConstraintManager()
535 .isNull(State, IvarSymbol)
536 .isConstrainedTrue()) {
537 continue;
538 }
539
540 // A missing release manifests as a leak, so treat as a non-fatal error.
541 if (!ErrNode)
542 ErrNode = C.generateNonFatalErrorNode();
543 // If we've already reached this node on another path, return without
544 // diagnosing.
545 if (!ErrNode)
546 return;
547
548 std::string Buf;
549 llvm::raw_string_ostream OS(Buf);
550
551 const ObjCInterfaceDecl *Interface = IvarDecl->getContainingInterface();
552 // If the class is known to have a lifecycle with teardown that is
553 // separate from -dealloc, do not warn about missing releases. We
554 // suppress here (rather than not tracking for instance variables in
555 // such classes) because these classes are rare.
556 if (classHasSeparateTeardown(Interface))
557 return;
558
559 ObjCImplDecl *ImplDecl = Interface->getImplementation();
560
561 const ObjCPropertyImplDecl *PropImpl =
562 ImplDecl->FindPropertyImplIvarDecl(IvarDecl->getIdentifier());
563
564 const ObjCPropertyDecl *PropDecl = PropImpl->getPropertyDecl();
565
566 assert(PropDecl->getSetterKind() == ObjCPropertyDecl::Copy ||
567 PropDecl->getSetterKind() == ObjCPropertyDecl::Retain);
568
569 OS << "The '" << *IvarDecl << "' ivar in '" << *ImplDecl
570 << "' was ";
571
572 if (PropDecl->getSetterKind() == ObjCPropertyDecl::Retain)
573 OS << "retained";
574 else
575 OS << "copied";
576
577 OS << " by a synthesized property but not released"
578 " before '[super dealloc]'";
579
580 auto BR = std::make_unique<PathSensitiveBugReport>(*MissingReleaseBugType,
581 OS.str(), ErrNode);
582 C.emitReport(std::move(BR));
583 }
584
585 if (NewUnreleased.isEmpty()) {
586 State = State->remove<UnreleasedIvarMap>(SelfSym);
587 } else {
588 State = State->set<UnreleasedIvarMap>(SelfSym, NewUnreleased);
589 }
590
591 if (ErrNode) {
592 C.addTransition(State, ErrNode);
593 } else if (State != InitialState) {
594 C.addTransition(State);
595 }
596
597 // Make sure that after checking in the top-most frame the list of
598 // tracked ivars is empty. This is intended to detect accidental leaks in
599 // the UnreleasedIvarMap program state.
600 assert(!LCtx->inTopFrame() || State->get<UnreleasedIvarMap>().isEmpty());
601 }
602
603 /// Given a symbol, determine whether the symbol refers to an ivar on
604 /// the top-most deallocating instance. If so, find the property for that
605 /// ivar, if one exists. Otherwise return null.
606 const ObjCPropertyImplDecl *
findPropertyOnDeallocatingInstance(SymbolRef IvarSym,CheckerContext & C) const607 ObjCDeallocChecker::findPropertyOnDeallocatingInstance(
608 SymbolRef IvarSym, CheckerContext &C) const {
609 SVal DeallocedInstance;
610 if (!isInInstanceDealloc(C, DeallocedInstance))
611 return nullptr;
612
613 // Try to get the region from which the ivar value was loaded.
614 auto *IvarRegion = getIvarRegionForIvarSymbol(IvarSym);
615 if (!IvarRegion)
616 return nullptr;
617
618 // Don't try to find the property if the ivar was not loaded from the
619 // given instance.
620 if (DeallocedInstance.castAs<loc::MemRegionVal>().getRegion() !=
621 IvarRegion->getSuperRegion())
622 return nullptr;
623
624 const LocationContext *LCtx = C.getLocationContext();
625 const ObjCIvarDecl *IvarDecl = IvarRegion->getDecl();
626
627 const ObjCImplDecl *Container = getContainingObjCImpl(LCtx);
628 const ObjCPropertyImplDecl *PropImpl =
629 Container->FindPropertyImplIvarDecl(IvarDecl->getIdentifier());
630 return PropImpl;
631 }
632
633 /// Emits a warning if the current context is -dealloc and ReleasedValue
634 /// must not be directly released in a -dealloc. Returns true if a diagnostic
635 /// was emitted.
diagnoseExtraRelease(SymbolRef ReleasedValue,const ObjCMethodCall & M,CheckerContext & C) const636 bool ObjCDeallocChecker::diagnoseExtraRelease(SymbolRef ReleasedValue,
637 const ObjCMethodCall &M,
638 CheckerContext &C) const {
639 // Try to get the region from which the released value was loaded.
640 // Note that, unlike diagnosing for missing releases, here we don't track
641 // values that must not be released in the state. This is because even if
642 // these values escape, it is still an error under the rules of MRR to
643 // release them in -dealloc.
644 const ObjCPropertyImplDecl *PropImpl =
645 findPropertyOnDeallocatingInstance(ReleasedValue, C);
646
647 if (!PropImpl)
648 return false;
649
650 // If the ivar belongs to a property that must not be released directly
651 // in dealloc, emit a warning.
652 if (getDeallocReleaseRequirement(PropImpl) !=
653 ReleaseRequirement::MustNotReleaseDirectly) {
654 return false;
655 }
656
657 // If the property is readwrite but it shadows a read-only property in its
658 // external interface, treat the property a read-only. If the outside
659 // world cannot write to a property then the internal implementation is free
660 // to make its own convention about whether the value is stored retained
661 // or not. We look up the shadow here rather than in
662 // getDeallocReleaseRequirement() because doing so can be expensive.
663 const ObjCPropertyDecl *PropDecl = findShadowedPropertyDecl(PropImpl);
664 if (PropDecl) {
665 if (PropDecl->isReadOnly())
666 return false;
667 } else {
668 PropDecl = PropImpl->getPropertyDecl();
669 }
670
671 ExplodedNode *ErrNode = C.generateNonFatalErrorNode();
672 if (!ErrNode)
673 return false;
674
675 std::string Buf;
676 llvm::raw_string_ostream OS(Buf);
677
678 assert(PropDecl->getSetterKind() == ObjCPropertyDecl::Weak ||
679 (PropDecl->getSetterKind() == ObjCPropertyDecl::Assign &&
680 !PropDecl->isReadOnly()) ||
681 isReleasedByCIFilterDealloc(PropImpl)
682 );
683
684 const ObjCImplDecl *Container = getContainingObjCImpl(C.getLocationContext());
685 OS << "The '" << *PropImpl->getPropertyIvarDecl()
686 << "' ivar in '" << *Container;
687
688
689 if (isReleasedByCIFilterDealloc(PropImpl)) {
690 OS << "' will be released by '-[CIFilter dealloc]' but also released here";
691 } else {
692 OS << "' was synthesized for ";
693
694 if (PropDecl->getSetterKind() == ObjCPropertyDecl::Weak)
695 OS << "a weak";
696 else
697 OS << "an assign, readwrite";
698
699 OS << " property but was released in 'dealloc'";
700 }
701
702 auto BR = std::make_unique<PathSensitiveBugReport>(*ExtraReleaseBugType,
703 OS.str(), ErrNode);
704 BR->addRange(M.getOriginExpr()->getSourceRange());
705
706 C.emitReport(std::move(BR));
707
708 return true;
709 }
710
711 /// Emits a warning if the current context is -dealloc and DeallocedValue
712 /// must not be directly dealloced in a -dealloc. Returns true if a diagnostic
713 /// was emitted.
diagnoseMistakenDealloc(SymbolRef DeallocedValue,const ObjCMethodCall & M,CheckerContext & C) const714 bool ObjCDeallocChecker::diagnoseMistakenDealloc(SymbolRef DeallocedValue,
715 const ObjCMethodCall &M,
716 CheckerContext &C) const {
717 // TODO: Apart from unknown/undefined receivers, this may happen when
718 // dealloc is called as a class method. Should we warn?
719 if (!DeallocedValue)
720 return false;
721
722 // Find the property backing the instance variable that M
723 // is dealloc'ing.
724 const ObjCPropertyImplDecl *PropImpl =
725 findPropertyOnDeallocatingInstance(DeallocedValue, C);
726 if (!PropImpl)
727 return false;
728
729 if (getDeallocReleaseRequirement(PropImpl) !=
730 ReleaseRequirement::MustRelease) {
731 return false;
732 }
733
734 ExplodedNode *ErrNode = C.generateErrorNode();
735 if (!ErrNode)
736 return false;
737
738 std::string Buf;
739 llvm::raw_string_ostream OS(Buf);
740
741 OS << "'" << *PropImpl->getPropertyIvarDecl()
742 << "' should be released rather than deallocated";
743
744 auto BR = std::make_unique<PathSensitiveBugReport>(*MistakenDeallocBugType,
745 OS.str(), ErrNode);
746 BR->addRange(M.getOriginExpr()->getSourceRange());
747
748 C.emitReport(std::move(BR));
749
750 return true;
751 }
752
ObjCDeallocChecker()753 ObjCDeallocChecker::ObjCDeallocChecker()
754 : NSObjectII(nullptr), SenTestCaseII(nullptr), XCTestCaseII(nullptr),
755 CIFilterII(nullptr) {
756
757 MissingReleaseBugType.reset(
758 new BugType(this, "Missing ivar release (leak)",
759 categories::MemoryRefCount));
760
761 ExtraReleaseBugType.reset(
762 new BugType(this, "Extra ivar release",
763 categories::MemoryRefCount));
764
765 MistakenDeallocBugType.reset(
766 new BugType(this, "Mistaken dealloc",
767 categories::MemoryRefCount));
768 }
769
initIdentifierInfoAndSelectors(ASTContext & Ctx) const770 void ObjCDeallocChecker::initIdentifierInfoAndSelectors(
771 ASTContext &Ctx) const {
772 if (NSObjectII)
773 return;
774
775 NSObjectII = &Ctx.Idents.get("NSObject");
776 SenTestCaseII = &Ctx.Idents.get("SenTestCase");
777 XCTestCaseII = &Ctx.Idents.get("XCTestCase");
778 Block_releaseII = &Ctx.Idents.get("_Block_release");
779 CIFilterII = &Ctx.Idents.get("CIFilter");
780
781 IdentifierInfo *DeallocII = &Ctx.Idents.get("dealloc");
782 IdentifierInfo *ReleaseII = &Ctx.Idents.get("release");
783 DeallocSel = Ctx.Selectors.getSelector(0, &DeallocII);
784 ReleaseSel = Ctx.Selectors.getSelector(0, &ReleaseII);
785 }
786
787 /// Returns true if M is a call to '[super dealloc]'.
isSuperDeallocMessage(const ObjCMethodCall & M) const788 bool ObjCDeallocChecker::isSuperDeallocMessage(
789 const ObjCMethodCall &M) const {
790 if (M.getOriginExpr()->getReceiverKind() != ObjCMessageExpr::SuperInstance)
791 return false;
792
793 return M.getSelector() == DeallocSel;
794 }
795
796 /// Returns the ObjCImplDecl containing the method declaration in LCtx.
797 const ObjCImplDecl *
getContainingObjCImpl(const LocationContext * LCtx) const798 ObjCDeallocChecker::getContainingObjCImpl(const LocationContext *LCtx) const {
799 auto *MD = cast<ObjCMethodDecl>(LCtx->getDecl());
800 return cast<ObjCImplDecl>(MD->getDeclContext());
801 }
802
803 /// Returns the property that shadowed by PropImpl if one exists and
804 /// nullptr otherwise.
findShadowedPropertyDecl(const ObjCPropertyImplDecl * PropImpl) const805 const ObjCPropertyDecl *ObjCDeallocChecker::findShadowedPropertyDecl(
806 const ObjCPropertyImplDecl *PropImpl) const {
807 const ObjCPropertyDecl *PropDecl = PropImpl->getPropertyDecl();
808
809 // Only readwrite properties can shadow.
810 if (PropDecl->isReadOnly())
811 return nullptr;
812
813 auto *CatDecl = dyn_cast<ObjCCategoryDecl>(PropDecl->getDeclContext());
814
815 // Only class extensions can contain shadowing properties.
816 if (!CatDecl || !CatDecl->IsClassExtension())
817 return nullptr;
818
819 IdentifierInfo *ID = PropDecl->getIdentifier();
820 DeclContext::lookup_result R = CatDecl->getClassInterface()->lookup(ID);
821 for (DeclContext::lookup_iterator I = R.begin(), E = R.end(); I != E; ++I) {
822 auto *ShadowedPropDecl = dyn_cast<ObjCPropertyDecl>(*I);
823 if (!ShadowedPropDecl)
824 continue;
825
826 if (ShadowedPropDecl->isInstanceProperty()) {
827 assert(ShadowedPropDecl->isReadOnly());
828 return ShadowedPropDecl;
829 }
830 }
831
832 return nullptr;
833 }
834
835 /// Add a transition noting the release of the given value.
transitionToReleaseValue(CheckerContext & C,SymbolRef Value) const836 void ObjCDeallocChecker::transitionToReleaseValue(CheckerContext &C,
837 SymbolRef Value) const {
838 assert(Value);
839 SymbolRef InstanceSym = getInstanceSymbolFromIvarSymbol(Value);
840 if (!InstanceSym)
841 return;
842 ProgramStateRef InitialState = C.getState();
843
844 ProgramStateRef ReleasedState =
845 removeValueRequiringRelease(InitialState, InstanceSym, Value);
846
847 if (ReleasedState != InitialState) {
848 C.addTransition(ReleasedState);
849 }
850 }
851
852 /// Remove the Value requiring a release from the tracked set for
853 /// Instance and return the resultant state.
removeValueRequiringRelease(ProgramStateRef State,SymbolRef Instance,SymbolRef Value) const854 ProgramStateRef ObjCDeallocChecker::removeValueRequiringRelease(
855 ProgramStateRef State, SymbolRef Instance, SymbolRef Value) const {
856 assert(Instance);
857 assert(Value);
858 const ObjCIvarRegion *RemovedRegion = getIvarRegionForIvarSymbol(Value);
859 if (!RemovedRegion)
860 return State;
861
862 const SymbolSet *Unreleased = State->get<UnreleasedIvarMap>(Instance);
863 if (!Unreleased)
864 return State;
865
866 // Mark the value as no longer requiring a release.
867 SymbolSet::Factory &F = State->getStateManager().get_context<SymbolSet>();
868 SymbolSet NewUnreleased = *Unreleased;
869 for (auto &Sym : *Unreleased) {
870 const ObjCIvarRegion *UnreleasedRegion = getIvarRegionForIvarSymbol(Sym);
871 assert(UnreleasedRegion);
872 if (RemovedRegion->getDecl() == UnreleasedRegion->getDecl()) {
873 NewUnreleased = F.remove(NewUnreleased, Sym);
874 }
875 }
876
877 if (NewUnreleased.isEmpty()) {
878 return State->remove<UnreleasedIvarMap>(Instance);
879 }
880
881 return State->set<UnreleasedIvarMap>(Instance, NewUnreleased);
882 }
883
884 /// Determines whether the instance variable for \p PropImpl must or must not be
885 /// released in -dealloc or whether it cannot be determined.
getDeallocReleaseRequirement(const ObjCPropertyImplDecl * PropImpl) const886 ReleaseRequirement ObjCDeallocChecker::getDeallocReleaseRequirement(
887 const ObjCPropertyImplDecl *PropImpl) const {
888 const ObjCIvarDecl *IvarDecl;
889 const ObjCPropertyDecl *PropDecl;
890 if (!isSynthesizedRetainableProperty(PropImpl, &IvarDecl, &PropDecl))
891 return ReleaseRequirement::Unknown;
892
893 ObjCPropertyDecl::SetterKind SK = PropDecl->getSetterKind();
894
895 switch (SK) {
896 // Retain and copy setters retain/copy their values before storing and so
897 // the value in their instance variables must be released in -dealloc.
898 case ObjCPropertyDecl::Retain:
899 case ObjCPropertyDecl::Copy:
900 if (isReleasedByCIFilterDealloc(PropImpl))
901 return ReleaseRequirement::MustNotReleaseDirectly;
902
903 if (isNibLoadedIvarWithoutRetain(PropImpl))
904 return ReleaseRequirement::Unknown;
905
906 return ReleaseRequirement::MustRelease;
907
908 case ObjCPropertyDecl::Weak:
909 return ReleaseRequirement::MustNotReleaseDirectly;
910
911 case ObjCPropertyDecl::Assign:
912 // It is common for the ivars for read-only assign properties to
913 // always be stored retained, so their release requirement cannot be
914 // be determined.
915 if (PropDecl->isReadOnly())
916 return ReleaseRequirement::Unknown;
917
918 return ReleaseRequirement::MustNotReleaseDirectly;
919 }
920 llvm_unreachable("Unrecognized setter kind");
921 }
922
923 /// Returns the released value if M is a call a setter that releases
924 /// and nils out its underlying instance variable.
925 SymbolRef
getValueReleasedByNillingOut(const ObjCMethodCall & M,CheckerContext & C) const926 ObjCDeallocChecker::getValueReleasedByNillingOut(const ObjCMethodCall &M,
927 CheckerContext &C) const {
928 SVal ReceiverVal = M.getReceiverSVal();
929 if (!ReceiverVal.isValid())
930 return nullptr;
931
932 if (M.getNumArgs() == 0)
933 return nullptr;
934
935 if (!M.getArgExpr(0)->getType()->isObjCRetainableType())
936 return nullptr;
937
938 // Is the first argument nil?
939 SVal Arg = M.getArgSVal(0);
940 ProgramStateRef notNilState, nilState;
941 std::tie(notNilState, nilState) =
942 M.getState()->assume(Arg.castAs<DefinedOrUnknownSVal>());
943 if (!(nilState && !notNilState))
944 return nullptr;
945
946 const ObjCPropertyDecl *Prop = M.getAccessedProperty();
947 if (!Prop)
948 return nullptr;
949
950 ObjCIvarDecl *PropIvarDecl = Prop->getPropertyIvarDecl();
951 if (!PropIvarDecl)
952 return nullptr;
953
954 ProgramStateRef State = C.getState();
955
956 SVal LVal = State->getLValue(PropIvarDecl, ReceiverVal);
957 std::optional<Loc> LValLoc = LVal.getAs<Loc>();
958 if (!LValLoc)
959 return nullptr;
960
961 SVal CurrentValInIvar = State->getSVal(*LValLoc);
962 return CurrentValInIvar.getAsSymbol();
963 }
964
965 /// Returns true if the current context is a call to -dealloc and false
966 /// otherwise. If true, it also sets SelfValOut to the value of
967 /// 'self'.
isInInstanceDealloc(const CheckerContext & C,SVal & SelfValOut) const968 bool ObjCDeallocChecker::isInInstanceDealloc(const CheckerContext &C,
969 SVal &SelfValOut) const {
970 return isInInstanceDealloc(C, C.getLocationContext(), SelfValOut);
971 }
972
973 /// Returns true if LCtx is a call to -dealloc and false
974 /// otherwise. If true, it also sets SelfValOut to the value of
975 /// 'self'.
isInInstanceDealloc(const CheckerContext & C,const LocationContext * LCtx,SVal & SelfValOut) const976 bool ObjCDeallocChecker::isInInstanceDealloc(const CheckerContext &C,
977 const LocationContext *LCtx,
978 SVal &SelfValOut) const {
979 auto *MD = dyn_cast<ObjCMethodDecl>(LCtx->getDecl());
980 if (!MD || !MD->isInstanceMethod() || MD->getSelector() != DeallocSel)
981 return false;
982
983 const ImplicitParamDecl *SelfDecl = LCtx->getSelfDecl();
984 assert(SelfDecl && "No self in -dealloc?");
985
986 ProgramStateRef State = C.getState();
987 SelfValOut = State->getSVal(State->getRegion(SelfDecl, LCtx));
988 return true;
989 }
990
991 /// Returns true if there is a call to -dealloc anywhere on the stack and false
992 /// otherwise. If true, it also sets InstanceValOut to the value of
993 /// 'self' in the frame for -dealloc.
instanceDeallocIsOnStack(const CheckerContext & C,SVal & InstanceValOut) const994 bool ObjCDeallocChecker::instanceDeallocIsOnStack(const CheckerContext &C,
995 SVal &InstanceValOut) const {
996 const LocationContext *LCtx = C.getLocationContext();
997
998 while (LCtx) {
999 if (isInInstanceDealloc(C, LCtx, InstanceValOut))
1000 return true;
1001
1002 LCtx = LCtx->getParent();
1003 }
1004
1005 return false;
1006 }
1007
1008 /// Returns true if the ID is a class in which is known to have
1009 /// a separate teardown lifecycle. In this case, -dealloc warnings
1010 /// about missing releases should be suppressed.
classHasSeparateTeardown(const ObjCInterfaceDecl * ID) const1011 bool ObjCDeallocChecker::classHasSeparateTeardown(
1012 const ObjCInterfaceDecl *ID) const {
1013 // Suppress if the class is not a subclass of NSObject.
1014 for ( ; ID ; ID = ID->getSuperClass()) {
1015 IdentifierInfo *II = ID->getIdentifier();
1016
1017 if (II == NSObjectII)
1018 return false;
1019
1020 // FIXME: For now, ignore classes that subclass SenTestCase and XCTestCase,
1021 // as these don't need to implement -dealloc. They implement tear down in
1022 // another way, which we should try and catch later.
1023 // http://llvm.org/bugs/show_bug.cgi?id=3187
1024 if (II == XCTestCaseII || II == SenTestCaseII)
1025 return true;
1026 }
1027
1028 return true;
1029 }
1030
1031 /// The -dealloc method in CIFilter highly unusual in that is will release
1032 /// instance variables belonging to its *subclasses* if the variable name
1033 /// starts with "input" or backs a property whose name starts with "input".
1034 /// Subclasses should not release these ivars in their own -dealloc method --
1035 /// doing so could result in an over release.
1036 ///
1037 /// This method returns true if the property will be released by
1038 /// -[CIFilter dealloc].
isReleasedByCIFilterDealloc(const ObjCPropertyImplDecl * PropImpl) const1039 bool ObjCDeallocChecker::isReleasedByCIFilterDealloc(
1040 const ObjCPropertyImplDecl *PropImpl) const {
1041 assert(PropImpl->getPropertyIvarDecl());
1042 StringRef PropName = PropImpl->getPropertyDecl()->getName();
1043 StringRef IvarName = PropImpl->getPropertyIvarDecl()->getName();
1044
1045 const char *ReleasePrefix = "input";
1046 if (!(PropName.startswith(ReleasePrefix) ||
1047 IvarName.startswith(ReleasePrefix))) {
1048 return false;
1049 }
1050
1051 const ObjCInterfaceDecl *ID =
1052 PropImpl->getPropertyIvarDecl()->getContainingInterface();
1053 for ( ; ID ; ID = ID->getSuperClass()) {
1054 IdentifierInfo *II = ID->getIdentifier();
1055 if (II == CIFilterII)
1056 return true;
1057 }
1058
1059 return false;
1060 }
1061
1062 /// Returns whether the ivar backing the property is an IBOutlet that
1063 /// has its value set by nib loading code without retaining the value.
1064 ///
1065 /// On macOS, if there is no setter, the nib-loading code sets the ivar
1066 /// directly, without retaining the value,
1067 ///
1068 /// On iOS and its derivatives, the nib-loading code will call
1069 /// -setValue:forKey:, which retains the value before directly setting the ivar.
isNibLoadedIvarWithoutRetain(const ObjCPropertyImplDecl * PropImpl) const1070 bool ObjCDeallocChecker::isNibLoadedIvarWithoutRetain(
1071 const ObjCPropertyImplDecl *PropImpl) const {
1072 const ObjCIvarDecl *IvarDecl = PropImpl->getPropertyIvarDecl();
1073 if (!IvarDecl->hasAttr<IBOutletAttr>())
1074 return false;
1075
1076 const llvm::Triple &Target =
1077 IvarDecl->getASTContext().getTargetInfo().getTriple();
1078
1079 if (!Target.isMacOSX())
1080 return false;
1081
1082 if (PropImpl->getPropertyDecl()->getSetterMethodDecl())
1083 return false;
1084
1085 return true;
1086 }
1087
registerObjCDeallocChecker(CheckerManager & Mgr)1088 void ento::registerObjCDeallocChecker(CheckerManager &Mgr) {
1089 Mgr.registerChecker<ObjCDeallocChecker>();
1090 }
1091
shouldRegisterObjCDeallocChecker(const CheckerManager & mgr)1092 bool ento::shouldRegisterObjCDeallocChecker(const CheckerManager &mgr) {
1093 // These checker only makes sense under MRR.
1094 const LangOptions &LO = mgr.getLangOpts();
1095 return LO.getGC() != LangOptions::GCOnly && !LO.ObjCAutoRefCount;
1096 }
1097