xref: /aosp_15_r20/external/clang/lib/StaticAnalyzer/Checkers/ObjCSuperDeallocChecker.cpp (revision 67e74705e28f6214e480b399dd47ea732279e315)
1*67e74705SXin Li //===- ObjCSuperDeallocChecker.cpp - Check correct use of [super dealloc] -===//
2*67e74705SXin Li //
3*67e74705SXin Li //                     The LLVM Compiler Infrastructure
4*67e74705SXin Li //
5*67e74705SXin Li // This file is distributed under the University of Illinois Open Source
6*67e74705SXin Li // License. See LICENSE.TXT for details.
7*67e74705SXin Li //
8*67e74705SXin Li //===----------------------------------------------------------------------===//
9*67e74705SXin Li //
10*67e74705SXin Li // This defines ObjCSuperDeallocChecker, a builtin check that warns when
11*67e74705SXin Li // self is used after a call to [super dealloc] in MRR mode.
12*67e74705SXin Li //
13*67e74705SXin Li //===----------------------------------------------------------------------===//
14*67e74705SXin Li 
15*67e74705SXin Li #include "ClangSACheckers.h"
16*67e74705SXin Li #include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
17*67e74705SXin Li #include "clang/StaticAnalyzer/Core/Checker.h"
18*67e74705SXin Li #include "clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h"
19*67e74705SXin Li #include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
20*67e74705SXin Li #include "clang/StaticAnalyzer/Core/PathSensitive/ProgramState.h"
21*67e74705SXin Li #include "clang/StaticAnalyzer/Core/PathSensitive/SymbolManager.h"
22*67e74705SXin Li 
23*67e74705SXin Li using namespace clang;
24*67e74705SXin Li using namespace ento;
25*67e74705SXin Li 
26*67e74705SXin Li namespace {
27*67e74705SXin Li class ObjCSuperDeallocChecker
28*67e74705SXin Li     : public Checker<check::PostObjCMessage, check::PreObjCMessage,
29*67e74705SXin Li                      check::PreCall, check::Location> {
30*67e74705SXin Li 
31*67e74705SXin Li   mutable IdentifierInfo *IIdealloc, *IINSObject;
32*67e74705SXin Li   mutable Selector SELdealloc;
33*67e74705SXin Li 
34*67e74705SXin Li   std::unique_ptr<BugType> DoubleSuperDeallocBugType;
35*67e74705SXin Li 
36*67e74705SXin Li   void initIdentifierInfoAndSelectors(ASTContext &Ctx) const;
37*67e74705SXin Li 
38*67e74705SXin Li   bool isSuperDeallocMessage(const ObjCMethodCall &M) const;
39*67e74705SXin Li 
40*67e74705SXin Li public:
41*67e74705SXin Li   ObjCSuperDeallocChecker();
42*67e74705SXin Li   void checkPostObjCMessage(const ObjCMethodCall &M, CheckerContext &C) const;
43*67e74705SXin Li   void checkPreObjCMessage(const ObjCMethodCall &M, CheckerContext &C) const;
44*67e74705SXin Li 
45*67e74705SXin Li   void checkPreCall(const CallEvent &Call, CheckerContext &C) const;
46*67e74705SXin Li 
47*67e74705SXin Li   void checkLocation(SVal l, bool isLoad, const Stmt *S,
48*67e74705SXin Li                      CheckerContext &C) const;
49*67e74705SXin Li 
50*67e74705SXin Li private:
51*67e74705SXin Li 
52*67e74705SXin Li   void diagnoseCallArguments(const CallEvent &CE, CheckerContext &C) const;
53*67e74705SXin Li 
54*67e74705SXin Li   void reportUseAfterDealloc(SymbolRef Sym, StringRef Desc, const Stmt *S,
55*67e74705SXin Li                              CheckerContext &C) const;
56*67e74705SXin Li };
57*67e74705SXin Li 
58*67e74705SXin Li } // End anonymous namespace.
59*67e74705SXin Li 
60*67e74705SXin Li // Remember whether [super dealloc] has previously been called on the
61*67e74705SXin Li // SymbolRef for the receiver.
62*67e74705SXin Li REGISTER_SET_WITH_PROGRAMSTATE(CalledSuperDealloc, SymbolRef)
63*67e74705SXin Li 
64*67e74705SXin Li namespace {
65*67e74705SXin Li class SuperDeallocBRVisitor final
66*67e74705SXin Li     : public BugReporterVisitorImpl<SuperDeallocBRVisitor> {
67*67e74705SXin Li 
68*67e74705SXin Li   SymbolRef ReceiverSymbol;
69*67e74705SXin Li   bool Satisfied;
70*67e74705SXin Li 
71*67e74705SXin Li public:
SuperDeallocBRVisitor(SymbolRef ReceiverSymbol)72*67e74705SXin Li   SuperDeallocBRVisitor(SymbolRef ReceiverSymbol)
73*67e74705SXin Li       : ReceiverSymbol(ReceiverSymbol),
74*67e74705SXin Li         Satisfied(false) {}
75*67e74705SXin Li 
76*67e74705SXin Li   PathDiagnosticPiece *VisitNode(const ExplodedNode *Succ,
77*67e74705SXin Li                                  const ExplodedNode *Pred,
78*67e74705SXin Li                                  BugReporterContext &BRC,
79*67e74705SXin Li                                  BugReport &BR) override;
80*67e74705SXin Li 
Profile(llvm::FoldingSetNodeID & ID) const81*67e74705SXin Li   void Profile(llvm::FoldingSetNodeID &ID) const override {
82*67e74705SXin Li     ID.Add(ReceiverSymbol);
83*67e74705SXin Li   }
84*67e74705SXin Li };
85*67e74705SXin Li } // End anonymous namespace.
86*67e74705SXin Li 
checkPreObjCMessage(const ObjCMethodCall & M,CheckerContext & C) const87*67e74705SXin Li void ObjCSuperDeallocChecker::checkPreObjCMessage(const ObjCMethodCall &M,
88*67e74705SXin Li                                                   CheckerContext &C) const {
89*67e74705SXin Li 
90*67e74705SXin Li   ProgramStateRef State = C.getState();
91*67e74705SXin Li   SymbolRef ReceiverSymbol = M.getReceiverSVal().getAsSymbol();
92*67e74705SXin Li   if (!ReceiverSymbol) {
93*67e74705SXin Li     diagnoseCallArguments(M, C);
94*67e74705SXin Li     return;
95*67e74705SXin Li   }
96*67e74705SXin Li 
97*67e74705SXin Li   bool AlreadyCalled = State->contains<CalledSuperDealloc>(ReceiverSymbol);
98*67e74705SXin Li   if (!AlreadyCalled)
99*67e74705SXin Li     return;
100*67e74705SXin Li 
101*67e74705SXin Li   StringRef Desc;
102*67e74705SXin Li 
103*67e74705SXin Li   if (isSuperDeallocMessage(M)) {
104*67e74705SXin Li     Desc = "[super dealloc] should not be called multiple times";
105*67e74705SXin Li   } else {
106*67e74705SXin Li     Desc = StringRef();
107*67e74705SXin Li   }
108*67e74705SXin Li 
109*67e74705SXin Li   reportUseAfterDealloc(ReceiverSymbol, Desc, M.getOriginExpr(), C);
110*67e74705SXin Li 
111*67e74705SXin Li   return;
112*67e74705SXin Li }
113*67e74705SXin Li 
checkPreCall(const CallEvent & Call,CheckerContext & C) const114*67e74705SXin Li void ObjCSuperDeallocChecker::checkPreCall(const CallEvent &Call,
115*67e74705SXin Li                                            CheckerContext &C) const {
116*67e74705SXin Li   diagnoseCallArguments(Call, C);
117*67e74705SXin Li }
118*67e74705SXin Li 
checkPostObjCMessage(const ObjCMethodCall & M,CheckerContext & C) const119*67e74705SXin Li void ObjCSuperDeallocChecker::checkPostObjCMessage(const ObjCMethodCall &M,
120*67e74705SXin Li                                                    CheckerContext &C) const {
121*67e74705SXin Li   // Check for [super dealloc] method call.
122*67e74705SXin Li   if (!isSuperDeallocMessage(M))
123*67e74705SXin Li     return;
124*67e74705SXin Li 
125*67e74705SXin Li   ProgramStateRef State = C.getState();
126*67e74705SXin Li   SymbolRef ReceiverSymbol = M.getSelfSVal().getAsSymbol();
127*67e74705SXin Li   assert(ReceiverSymbol && "No receiver symbol at call to [super dealloc]?");
128*67e74705SXin Li 
129*67e74705SXin Li   // We add this transition in checkPostObjCMessage to avoid warning when
130*67e74705SXin Li   // we inline a call to [super dealloc] where the inlined call itself
131*67e74705SXin Li   // calls [super dealloc].
132*67e74705SXin Li   State = State->add<CalledSuperDealloc>(ReceiverSymbol);
133*67e74705SXin Li   C.addTransition(State);
134*67e74705SXin Li }
135*67e74705SXin Li 
checkLocation(SVal L,bool IsLoad,const Stmt * S,CheckerContext & C) const136*67e74705SXin Li void ObjCSuperDeallocChecker::checkLocation(SVal L, bool IsLoad, const Stmt *S,
137*67e74705SXin Li                                   CheckerContext &C) const {
138*67e74705SXin Li   SymbolRef BaseSym = L.getLocSymbolInBase();
139*67e74705SXin Li   if (!BaseSym)
140*67e74705SXin Li     return;
141*67e74705SXin Li 
142*67e74705SXin Li   ProgramStateRef State = C.getState();
143*67e74705SXin Li 
144*67e74705SXin Li   if (!State->contains<CalledSuperDealloc>(BaseSym))
145*67e74705SXin Li     return;
146*67e74705SXin Li 
147*67e74705SXin Li   const MemRegion *R = L.getAsRegion();
148*67e74705SXin Li   if (!R)
149*67e74705SXin Li     return;
150*67e74705SXin Li 
151*67e74705SXin Li   // Climb the super regions to find the base symbol while recording
152*67e74705SXin Li   // the second-to-last region for error reporting.
153*67e74705SXin Li   const MemRegion *PriorSubRegion = nullptr;
154*67e74705SXin Li   while (const SubRegion *SR = dyn_cast<SubRegion>(R)) {
155*67e74705SXin Li     if (const SymbolicRegion *SymR = dyn_cast<SymbolicRegion>(SR)) {
156*67e74705SXin Li       BaseSym = SymR->getSymbol();
157*67e74705SXin Li       break;
158*67e74705SXin Li     } else {
159*67e74705SXin Li       R = SR->getSuperRegion();
160*67e74705SXin Li       PriorSubRegion = SR;
161*67e74705SXin Li     }
162*67e74705SXin Li   }
163*67e74705SXin Li 
164*67e74705SXin Li   StringRef Desc = StringRef();
165*67e74705SXin Li   auto *IvarRegion = dyn_cast_or_null<ObjCIvarRegion>(PriorSubRegion);
166*67e74705SXin Li 
167*67e74705SXin Li   std::string Buf;
168*67e74705SXin Li   llvm::raw_string_ostream OS(Buf);
169*67e74705SXin Li   if (IvarRegion) {
170*67e74705SXin Li     OS << "Use of instance variable '" << *IvarRegion->getDecl() <<
171*67e74705SXin Li           "' after 'self' has been deallocated";
172*67e74705SXin Li     Desc = OS.str();
173*67e74705SXin Li   }
174*67e74705SXin Li 
175*67e74705SXin Li   reportUseAfterDealloc(BaseSym, Desc, S, C);
176*67e74705SXin Li }
177*67e74705SXin Li 
178*67e74705SXin Li /// Report a use-after-dealloc on Sym. If not empty,
179*67e74705SXin Li /// Desc will be used to describe the error; otherwise,
180*67e74705SXin Li /// a default warning will be used.
reportUseAfterDealloc(SymbolRef Sym,StringRef Desc,const Stmt * S,CheckerContext & C) const181*67e74705SXin Li void ObjCSuperDeallocChecker::reportUseAfterDealloc(SymbolRef Sym,
182*67e74705SXin Li                                                     StringRef Desc,
183*67e74705SXin Li                                                     const Stmt *S,
184*67e74705SXin Li                                                     CheckerContext &C) const {
185*67e74705SXin Li   // We have a use of self after free.
186*67e74705SXin Li   // This likely causes a crash, so stop exploring the
187*67e74705SXin Li   // path by generating a sink.
188*67e74705SXin Li   ExplodedNode *ErrNode = C.generateErrorNode();
189*67e74705SXin Li   // If we've already reached this node on another path, return.
190*67e74705SXin Li   if (!ErrNode)
191*67e74705SXin Li     return;
192*67e74705SXin Li 
193*67e74705SXin Li   if (Desc.empty())
194*67e74705SXin Li     Desc = "use of 'self' after it has been deallocated";
195*67e74705SXin Li 
196*67e74705SXin Li   // Generate the report.
197*67e74705SXin Li   std::unique_ptr<BugReport> BR(
198*67e74705SXin Li       new BugReport(*DoubleSuperDeallocBugType, Desc, ErrNode));
199*67e74705SXin Li   BR->addRange(S->getSourceRange());
200*67e74705SXin Li   BR->addVisitor(llvm::make_unique<SuperDeallocBRVisitor>(Sym));
201*67e74705SXin Li   C.emitReport(std::move(BR));
202*67e74705SXin Li }
203*67e74705SXin Li 
204*67e74705SXin Li /// Diagnose if any of the arguments to CE have already been
205*67e74705SXin Li /// dealloc'd.
diagnoseCallArguments(const CallEvent & CE,CheckerContext & C) const206*67e74705SXin Li void ObjCSuperDeallocChecker::diagnoseCallArguments(const CallEvent &CE,
207*67e74705SXin Li                                                     CheckerContext &C) const {
208*67e74705SXin Li   ProgramStateRef State = C.getState();
209*67e74705SXin Li   unsigned ArgCount = CE.getNumArgs();
210*67e74705SXin Li   for (unsigned I = 0; I < ArgCount; I++) {
211*67e74705SXin Li     SymbolRef Sym = CE.getArgSVal(I).getAsSymbol();
212*67e74705SXin Li     if (!Sym)
213*67e74705SXin Li       continue;
214*67e74705SXin Li 
215*67e74705SXin Li     if (State->contains<CalledSuperDealloc>(Sym)) {
216*67e74705SXin Li       reportUseAfterDealloc(Sym, StringRef(), CE.getArgExpr(I), C);
217*67e74705SXin Li       return;
218*67e74705SXin Li     }
219*67e74705SXin Li   }
220*67e74705SXin Li }
221*67e74705SXin Li 
ObjCSuperDeallocChecker()222*67e74705SXin Li ObjCSuperDeallocChecker::ObjCSuperDeallocChecker()
223*67e74705SXin Li     : IIdealloc(nullptr), IINSObject(nullptr) {
224*67e74705SXin Li 
225*67e74705SXin Li   DoubleSuperDeallocBugType.reset(
226*67e74705SXin Li       new BugType(this, "[super dealloc] should not be called more than once",
227*67e74705SXin Li                   categories::CoreFoundationObjectiveC));
228*67e74705SXin Li }
229*67e74705SXin Li 
230*67e74705SXin Li void
initIdentifierInfoAndSelectors(ASTContext & Ctx) const231*67e74705SXin Li ObjCSuperDeallocChecker::initIdentifierInfoAndSelectors(ASTContext &Ctx) const {
232*67e74705SXin Li   if (IIdealloc)
233*67e74705SXin Li     return;
234*67e74705SXin Li 
235*67e74705SXin Li   IIdealloc = &Ctx.Idents.get("dealloc");
236*67e74705SXin Li   IINSObject = &Ctx.Idents.get("NSObject");
237*67e74705SXin Li 
238*67e74705SXin Li   SELdealloc = Ctx.Selectors.getSelector(0, &IIdealloc);
239*67e74705SXin Li }
240*67e74705SXin Li 
241*67e74705SXin Li bool
isSuperDeallocMessage(const ObjCMethodCall & M) const242*67e74705SXin Li ObjCSuperDeallocChecker::isSuperDeallocMessage(const ObjCMethodCall &M) const {
243*67e74705SXin Li   if (M.getOriginExpr()->getReceiverKind() != ObjCMessageExpr::SuperInstance)
244*67e74705SXin Li     return false;
245*67e74705SXin Li 
246*67e74705SXin Li   ASTContext &Ctx = M.getState()->getStateManager().getContext();
247*67e74705SXin Li   initIdentifierInfoAndSelectors(Ctx);
248*67e74705SXin Li 
249*67e74705SXin Li   return M.getSelector() == SELdealloc;
250*67e74705SXin Li }
251*67e74705SXin Li 
VisitNode(const ExplodedNode * Succ,const ExplodedNode * Pred,BugReporterContext & BRC,BugReport & BR)252*67e74705SXin Li PathDiagnosticPiece *SuperDeallocBRVisitor::VisitNode(const ExplodedNode *Succ,
253*67e74705SXin Li                                                       const ExplodedNode *Pred,
254*67e74705SXin Li                                                       BugReporterContext &BRC,
255*67e74705SXin Li                                                       BugReport &BR) {
256*67e74705SXin Li   if (Satisfied)
257*67e74705SXin Li     return nullptr;
258*67e74705SXin Li 
259*67e74705SXin Li   ProgramStateRef State = Succ->getState();
260*67e74705SXin Li 
261*67e74705SXin Li   bool CalledNow =
262*67e74705SXin Li       Succ->getState()->contains<CalledSuperDealloc>(ReceiverSymbol);
263*67e74705SXin Li   bool CalledBefore =
264*67e74705SXin Li       Pred->getState()->contains<CalledSuperDealloc>(ReceiverSymbol);
265*67e74705SXin Li 
266*67e74705SXin Li   // Is Succ the node on which the analyzer noted that [super dealloc] was
267*67e74705SXin Li   // called on ReceiverSymbol?
268*67e74705SXin Li   if (CalledNow && !CalledBefore) {
269*67e74705SXin Li     Satisfied = true;
270*67e74705SXin Li 
271*67e74705SXin Li     ProgramPoint P = Succ->getLocation();
272*67e74705SXin Li     PathDiagnosticLocation L =
273*67e74705SXin Li         PathDiagnosticLocation::create(P, BRC.getSourceManager());
274*67e74705SXin Li 
275*67e74705SXin Li     if (!L.isValid() || !L.asLocation().isValid())
276*67e74705SXin Li       return nullptr;
277*67e74705SXin Li 
278*67e74705SXin Li     return new PathDiagnosticEventPiece(
279*67e74705SXin Li         L, "[super dealloc] called here");
280*67e74705SXin Li   }
281*67e74705SXin Li 
282*67e74705SXin Li   return nullptr;
283*67e74705SXin Li }
284*67e74705SXin Li 
285*67e74705SXin Li //===----------------------------------------------------------------------===//
286*67e74705SXin Li // Checker Registration.
287*67e74705SXin Li //===----------------------------------------------------------------------===//
288*67e74705SXin Li 
registerObjCSuperDeallocChecker(CheckerManager & Mgr)289*67e74705SXin Li void ento::registerObjCSuperDeallocChecker(CheckerManager &Mgr) {
290*67e74705SXin Li   const LangOptions &LangOpts = Mgr.getLangOpts();
291*67e74705SXin Li   if (LangOpts.getGC() == LangOptions::GCOnly || LangOpts.ObjCAutoRefCount)
292*67e74705SXin Li     return;
293*67e74705SXin Li   Mgr.registerChecker<ObjCSuperDeallocChecker>();
294*67e74705SXin Li }
295