1a9ce612fSTed Kremenek //=-- ExprEngineObjC.cpp - ExprEngine support for Objective-C ---*- C++ -*-===//
2a9ce612fSTed Kremenek //
32946cd70SChandler Carruth // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
42946cd70SChandler Carruth // See https://llvm.org/LICENSE.txt for license information.
52946cd70SChandler Carruth // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6a9ce612fSTed Kremenek //
7a9ce612fSTed Kremenek //===----------------------------------------------------------------------===//
8a9ce612fSTed Kremenek //
9a9ce612fSTed Kremenek //  This file defines ExprEngine's support for Objective-C expressions.
10a9ce612fSTed Kremenek //
11a9ce612fSTed Kremenek //===----------------------------------------------------------------------===//
12a9ce612fSTed Kremenek 
1311764ab4SBenjamin Kramer #include "clang/AST/StmtObjC.h"
14a9ce612fSTed Kremenek #include "clang/StaticAnalyzer/Core/CheckerManager.h"
154f7df9beSJordan Rose #include "clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h"
16a9ce612fSTed Kremenek #include "clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h"
17a9ce612fSTed Kremenek 
18a9ce612fSTed Kremenek using namespace clang;
19a9ce612fSTed Kremenek using namespace ento;
20a9ce612fSTed Kremenek 
VisitLvalObjCIvarRefExpr(const ObjCIvarRefExpr * Ex,ExplodedNode * Pred,ExplodedNodeSet & Dst)21a9ce612fSTed Kremenek void ExprEngine::VisitLvalObjCIvarRefExpr(const ObjCIvarRefExpr *Ex,
22a9ce612fSTed Kremenek                                           ExplodedNode *Pred,
23a9ce612fSTed Kremenek                                           ExplodedNodeSet &Dst) {
2449b1e38eSTed Kremenek   ProgramStateRef state = Pred->getState();
25632e3b7eSTed Kremenek   const LocationContext *LCtx = Pred->getLocationContext();
26632e3b7eSTed Kremenek   SVal baseVal = state->getSVal(Ex->getBase(), LCtx);
27a9ce612fSTed Kremenek   SVal location = state->getLValue(Ex->getDecl(), baseVal);
28a9ce612fSTed Kremenek 
29a9ce612fSTed Kremenek   ExplodedNodeSet dstIvar;
3092375adaSJordan Rose   StmtNodeBuilder Bldr(Pred, dstIvar, *currBldrCtx);
3192375adaSJordan Rose   Bldr.generateNode(Ex, Pred, state->BindExpr(Ex, LCtx, location));
32a9ce612fSTed Kremenek 
33a9ce612fSTed Kremenek   // Perform the post-condition check of the ObjCIvarRefExpr and store
34a9ce612fSTed Kremenek   // the created nodes in 'Dst'.
35a9ce612fSTed Kremenek   getCheckerManager().runCheckersForPostStmt(Dst, dstIvar, Ex, *this);
36a9ce612fSTed Kremenek }
37a9ce612fSTed Kremenek 
VisitObjCAtSynchronizedStmt(const ObjCAtSynchronizedStmt * S,ExplodedNode * Pred,ExplodedNodeSet & Dst)38a9ce612fSTed Kremenek void ExprEngine::VisitObjCAtSynchronizedStmt(const ObjCAtSynchronizedStmt *S,
39a9ce612fSTed Kremenek                                              ExplodedNode *Pred,
40a9ce612fSTed Kremenek                                              ExplodedNodeSet &Dst) {
41a9ce612fSTed Kremenek   getCheckerManager().runCheckersForPreStmt(Dst, Pred, S, *this);
42a9ce612fSTed Kremenek }
43a9ce612fSTed Kremenek 
4404b9dc58SGeorge Karpenkov /// Generate a node in \p Bldr for an iteration statement using ObjC
4504b9dc58SGeorge Karpenkov /// for-loop iterator.
populateObjCForDestinationSet(ExplodedNodeSet & dstLocation,SValBuilder & svalBuilder,const ObjCForCollectionStmt * S,const Stmt * elem,SVal elementV,SymbolManager & SymMgr,const NodeBuilderContext * currBldrCtx,StmtNodeBuilder & Bldr,bool hasElements)4604b9dc58SGeorge Karpenkov static void populateObjCForDestinationSet(
4704b9dc58SGeorge Karpenkov     ExplodedNodeSet &dstLocation, SValBuilder &svalBuilder,
4804b9dc58SGeorge Karpenkov     const ObjCForCollectionStmt *S, const Stmt *elem, SVal elementV,
4904b9dc58SGeorge Karpenkov     SymbolManager &SymMgr, const NodeBuilderContext *currBldrCtx,
5004b9dc58SGeorge Karpenkov     StmtNodeBuilder &Bldr, bool hasElements) {
5104b9dc58SGeorge Karpenkov 
5204b9dc58SGeorge Karpenkov   for (ExplodedNode *Pred : dstLocation) {
5304b9dc58SGeorge Karpenkov     ProgramStateRef state = Pred->getState();
5404b9dc58SGeorge Karpenkov     const LocationContext *LCtx = Pred->getLocationContext();
5504b9dc58SGeorge Karpenkov 
56*b9bca883SKristóf Umann     ProgramStateRef nextState =
57*b9bca883SKristóf Umann         ExprEngine::setWhetherHasMoreIteration(state, S, LCtx, hasElements);
5804b9dc58SGeorge Karpenkov 
5904b9dc58SGeorge Karpenkov     if (auto MV = elementV.getAs<loc::MemRegionVal>())
6004b9dc58SGeorge Karpenkov       if (const auto *R = dyn_cast<TypedValueRegion>(MV->getRegion())) {
6104b9dc58SGeorge Karpenkov         // FIXME: The proper thing to do is to really iterate over the
6204b9dc58SGeorge Karpenkov         //  container.  We will do this with dispatch logic to the store.
6304b9dc58SGeorge Karpenkov         //  For now, just 'conjure' up a symbolic value.
6404b9dc58SGeorge Karpenkov         QualType T = R->getValueType();
6504b9dc58SGeorge Karpenkov         assert(Loc::isLocType(T));
6604b9dc58SGeorge Karpenkov 
6704b9dc58SGeorge Karpenkov         SVal V;
6804b9dc58SGeorge Karpenkov         if (hasElements) {
6904b9dc58SGeorge Karpenkov           SymbolRef Sym = SymMgr.conjureSymbol(elem, LCtx, T,
7004b9dc58SGeorge Karpenkov                                                currBldrCtx->blockCount());
7104b9dc58SGeorge Karpenkov           V = svalBuilder.makeLoc(Sym);
7204b9dc58SGeorge Karpenkov         } else {
7304b9dc58SGeorge Karpenkov           V = svalBuilder.makeIntVal(0, T);
7404b9dc58SGeorge Karpenkov         }
7504b9dc58SGeorge Karpenkov 
7604b9dc58SGeorge Karpenkov         nextState = nextState->bindLoc(elementV, V, LCtx);
7704b9dc58SGeorge Karpenkov       }
7804b9dc58SGeorge Karpenkov 
7904b9dc58SGeorge Karpenkov     Bldr.generateNode(S, Pred, nextState);
8004b9dc58SGeorge Karpenkov   }
8104b9dc58SGeorge Karpenkov }
8204b9dc58SGeorge Karpenkov 
VisitObjCForCollectionStmt(const ObjCForCollectionStmt * S,ExplodedNode * Pred,ExplodedNodeSet & Dst)83a9ce612fSTed Kremenek void ExprEngine::VisitObjCForCollectionStmt(const ObjCForCollectionStmt *S,
84a9ce612fSTed Kremenek                                             ExplodedNode *Pred,
85a9ce612fSTed Kremenek                                             ExplodedNodeSet &Dst) {
86a9ce612fSTed Kremenek 
87a9ce612fSTed Kremenek   // ObjCForCollectionStmts are processed in two places.  This method
88a9ce612fSTed Kremenek   // handles the case where an ObjCForCollectionStmt* occurs as one of the
89a9ce612fSTed Kremenek   // statements within a basic block.  This transfer function does two things:
90a9ce612fSTed Kremenek   //
91a9ce612fSTed Kremenek   //  (1) binds the next container value to 'element'.  This creates a new
92a9ce612fSTed Kremenek   //      node in the ExplodedGraph.
93a9ce612fSTed Kremenek   //
94*b9bca883SKristóf Umann   //  (2) note whether the collection has any more elements (or in other words,
95*b9bca883SKristóf Umann   //      whether the loop has more iterations). This will be tested in
96*b9bca883SKristóf Umann   //      processBranch.
97a9ce612fSTed Kremenek   //
98a9ce612fSTed Kremenek   // FIXME: Eventually this logic should actually do dispatches to
99a9ce612fSTed Kremenek   //   'countByEnumeratingWithState:objects:count:' (NSFastEnumeration).
100a9ce612fSTed Kremenek   //   This will require simulating a temporary NSFastEnumerationState, either
101a9ce612fSTed Kremenek   //   through an SVal or through the use of MemRegions.  This value can
102a9ce612fSTed Kremenek   //   be affixed to the ObjCForCollectionStmt* instead of 0/1; when the loop
103a9ce612fSTed Kremenek   //   terminates we reclaim the temporary (it goes out of scope) and we
104a9ce612fSTed Kremenek   //   we can test if the SVal is 0 or if the MemRegion is null (depending
105a9ce612fSTed Kremenek   //   on what approach we take).
106a9ce612fSTed Kremenek   //
107a9ce612fSTed Kremenek   //  For now: simulate (1) by assigning either a symbol or nil if the
108a9ce612fSTed Kremenek   //    container is empty.  Thus this transfer function will by default
109a9ce612fSTed Kremenek   //    result in state splitting.
110a9ce612fSTed Kremenek 
111a9ce612fSTed Kremenek   const Stmt *elem = S->getElement();
11204b9dc58SGeorge Karpenkov   const Stmt *collection = S->getCollection();
11349b1e38eSTed Kremenek   ProgramStateRef state = Pred->getState();
11404b9dc58SGeorge Karpenkov   SVal collectionV = state->getSVal(collection, Pred->getLocationContext());
115a9ce612fSTed Kremenek 
11604b9dc58SGeorge Karpenkov   SVal elementV;
11704b9dc58SGeorge Karpenkov   if (const auto *DS = dyn_cast<DeclStmt>(elem)) {
118a9ce612fSTed Kremenek     const VarDecl *elemD = cast<VarDecl>(DS->getSingleDecl());
1190dbb783cSCraig Topper     assert(elemD->getInit() == nullptr);
120a9ce612fSTed Kremenek     elementV = state->getLValue(elemD, Pred->getLocationContext());
12104b9dc58SGeorge Karpenkov   } else {
122632e3b7eSTed Kremenek     elementV = state->getSVal(elem, Pred->getLocationContext());
123a9ce612fSTed Kremenek   }
124a9ce612fSTed Kremenek 
12504b9dc58SGeorge Karpenkov   bool isContainerNull = state->isNull(collectionV).isConstrainedTrue();
12604b9dc58SGeorge Karpenkov 
127a9ce612fSTed Kremenek   ExplodedNodeSet dstLocation;
128c82d457dSGeorge Karpenkov   evalLocation(dstLocation, S, elem, Pred, state, elementV, false);
129efef7602SJordan Rose 
130efef7602SJordan Rose   ExplodedNodeSet Tmp;
131d94854a4STed Kremenek   StmtNodeBuilder Bldr(Pred, Tmp, *currBldrCtx);
132a9ce612fSTed Kremenek 
13304b9dc58SGeorge Karpenkov   if (!isContainerNull)
13404b9dc58SGeorge Karpenkov     populateObjCForDestinationSet(dstLocation, svalBuilder, S, elem, elementV,
13504b9dc58SGeorge Karpenkov                                   SymMgr, currBldrCtx, Bldr,
13604b9dc58SGeorge Karpenkov                                   /*hasElements=*/true);
137a9ce612fSTed Kremenek 
13804b9dc58SGeorge Karpenkov   populateObjCForDestinationSet(dstLocation, svalBuilder, S, elem, elementV,
13904b9dc58SGeorge Karpenkov                                 SymMgr, currBldrCtx, Bldr,
14004b9dc58SGeorge Karpenkov                                 /*hasElements=*/false);
141efef7602SJordan Rose 
142efef7602SJordan Rose   // Finally, run any custom checkers.
143efef7602SJordan Rose   // FIXME: Eventually all pre- and post-checks should live in VisitStmt.
144efef7602SJordan Rose   getCheckerManager().runCheckersForPostStmt(Dst, Tmp, S, *this);
145a9ce612fSTed Kremenek }
146a9ce612fSTed Kremenek 
VisitObjCMessage(const ObjCMessageExpr * ME,ExplodedNode * Pred,ExplodedNodeSet & Dst)147fcd016e5SJordan Rose void ExprEngine::VisitObjCMessage(const ObjCMessageExpr *ME,
148a9ce612fSTed Kremenek                                   ExplodedNode *Pred,
149a9ce612fSTed Kremenek                                   ExplodedNodeSet &Dst) {
150fcd016e5SJordan Rose   CallEventManager &CEMgr = getStateManager().getCallEventManager();
151fcd016e5SJordan Rose   CallEventRef<ObjCMethodCall> Msg =
152fcd016e5SJordan Rose     CEMgr.getObjCMethodCall(ME, Pred->getState(), Pred->getLocationContext());
153a9ce612fSTed Kremenek 
154ca5ab2b0SDevin Coughlin   // There are three cases for the receiver:
155ca5ab2b0SDevin Coughlin   //   (1) it is definitely nil,
156ca5ab2b0SDevin Coughlin   //   (2) it is definitely non-nil, and
157ca5ab2b0SDevin Coughlin   //   (3) we don't know.
158ca5ab2b0SDevin Coughlin   //
159ca5ab2b0SDevin Coughlin   // If the receiver is definitely nil, we skip the pre/post callbacks and
160ca5ab2b0SDevin Coughlin   // instead call the ObjCMessageNil callbacks and return.
161ca5ab2b0SDevin Coughlin   //
162ca5ab2b0SDevin Coughlin   // If the receiver is definitely non-nil, we call the pre- callbacks,
163ca5ab2b0SDevin Coughlin   // evaluate the call, and call the post- callbacks.
164ca5ab2b0SDevin Coughlin   //
165ca5ab2b0SDevin Coughlin   // If we don't know, we drop the potential nil flow and instead
166ca5ab2b0SDevin Coughlin   // continue from the assumed non-nil state as in (2). This approach
167ca5ab2b0SDevin Coughlin   // intentionally drops coverage in order to prevent false alarms
168ca5ab2b0SDevin Coughlin   // in the following scenario:
169ca5ab2b0SDevin Coughlin   //
170ca5ab2b0SDevin Coughlin   // id result = [o someMethod]
171ca5ab2b0SDevin Coughlin   // if (result) {
172ca5ab2b0SDevin Coughlin   //   if (!o) {
173ca5ab2b0SDevin Coughlin   //     // <-- This program point should be unreachable because if o is nil
174ca5ab2b0SDevin Coughlin   //     // it must the case that result is nil as well.
175ca5ab2b0SDevin Coughlin   //   }
176ca5ab2b0SDevin Coughlin   // }
177ca5ab2b0SDevin Coughlin   //
178ca5ab2b0SDevin Coughlin   // We could avoid dropping coverage by performing an explicit case split
179ca5ab2b0SDevin Coughlin   // on each method call -- but this would get very expensive. An alternative
180ca5ab2b0SDevin Coughlin   // would be to introduce lazy constraints.
181ca5ab2b0SDevin Coughlin   // FIXME: This ignores many potential bugs (<rdar://problem/11733396>).
182ca5ab2b0SDevin Coughlin   // Revisit once we have lazier constraints.
183ca5ab2b0SDevin Coughlin   if (Msg->isInstanceMessage()) {
184ca5ab2b0SDevin Coughlin     SVal recVal = Msg->getReceiverSVal();
185ca5ab2b0SDevin Coughlin     if (!recVal.isUndef()) {
186ca5ab2b0SDevin Coughlin       // Bifurcate the state into nil and non-nil ones.
187ca5ab2b0SDevin Coughlin       DefinedOrUnknownSVal receiverVal =
188ca5ab2b0SDevin Coughlin           recVal.castAs<DefinedOrUnknownSVal>();
189ca5ab2b0SDevin Coughlin       ProgramStateRef State = Pred->getState();
190ca5ab2b0SDevin Coughlin 
191ca5ab2b0SDevin Coughlin       ProgramStateRef notNilState, nilState;
192ca5ab2b0SDevin Coughlin       std::tie(notNilState, nilState) = State->assume(receiverVal);
193ca5ab2b0SDevin Coughlin 
194ca5ab2b0SDevin Coughlin       // Receiver is definitely nil, so run ObjCMessageNil callbacks and return.
195ca5ab2b0SDevin Coughlin       if (nilState && !notNilState) {
1963ccf14ebSArtem Dergachev         ExplodedNodeSet dstNil;
1973ccf14ebSArtem Dergachev         StmtNodeBuilder Bldr(Pred, dstNil, *currBldrCtx);
198676a58c9SDevin Coughlin         bool HasTag = Pred->getLocation().getTag();
199ca5ab2b0SDevin Coughlin         Pred = Bldr.generateNode(ME, Pred, nilState, nullptr,
200ca5ab2b0SDevin Coughlin                                  ProgramPoint::PreStmtKind);
201676a58c9SDevin Coughlin         assert((Pred || HasTag) && "Should have cached out already!");
202676a58c9SDevin Coughlin         (void)HasTag;
203ca5ab2b0SDevin Coughlin         if (!Pred)
204ca5ab2b0SDevin Coughlin           return;
2053ccf14ebSArtem Dergachev 
2063ccf14ebSArtem Dergachev         ExplodedNodeSet dstPostCheckers;
2073ccf14ebSArtem Dergachev         getCheckerManager().runCheckersForObjCMessageNil(dstPostCheckers, Pred,
208ca5ab2b0SDevin Coughlin                                                          *Msg, *this);
2093ccf14ebSArtem Dergachev         for (auto I : dstPostCheckers)
2103ccf14ebSArtem Dergachev           finishArgumentConstruction(Dst, I, *Msg);
211ca5ab2b0SDevin Coughlin         return;
212ca5ab2b0SDevin Coughlin       }
213ca5ab2b0SDevin Coughlin 
214ca5ab2b0SDevin Coughlin       ExplodedNodeSet dstNonNil;
215ca5ab2b0SDevin Coughlin       StmtNodeBuilder Bldr(Pred, dstNonNil, *currBldrCtx);
216ca5ab2b0SDevin Coughlin       // Generate a transition to the non-nil state, dropping any potential
217ca5ab2b0SDevin Coughlin       // nil flow.
218ca5ab2b0SDevin Coughlin       if (notNilState != State) {
219676a58c9SDevin Coughlin         bool HasTag = Pred->getLocation().getTag();
220ca5ab2b0SDevin Coughlin         Pred = Bldr.generateNode(ME, Pred, notNilState);
221676a58c9SDevin Coughlin         assert((Pred || HasTag) && "Should have cached out already!");
222676a58c9SDevin Coughlin         (void)HasTag;
223ca5ab2b0SDevin Coughlin         if (!Pred)
224ca5ab2b0SDevin Coughlin           return;
225ca5ab2b0SDevin Coughlin       }
226ca5ab2b0SDevin Coughlin     }
227ca5ab2b0SDevin Coughlin   }
228ca5ab2b0SDevin Coughlin 
229a9ce612fSTed Kremenek   // Handle the previsits checks.
230a9ce612fSTed Kremenek   ExplodedNodeSet dstPrevisit;
231a9ce612fSTed Kremenek   getCheckerManager().runCheckersForPreObjCMessage(dstPrevisit, Pred,
232fcd016e5SJordan Rose                                                    *Msg, *this);
233afe7c2c2SJordan Rose   ExplodedNodeSet dstGenericPrevisit;
234afe7c2c2SJordan Rose   getCheckerManager().runCheckersForPreCall(dstGenericPrevisit, dstPrevisit,
235fcd016e5SJordan Rose                                             *Msg, *this);
236a9ce612fSTed Kremenek 
237a9ce612fSTed Kremenek   // Proceed with evaluate the message expression.
238a9ce612fSTed Kremenek   ExplodedNodeSet dstEval;
239d94854a4STed Kremenek   StmtNodeBuilder Bldr(dstGenericPrevisit, dstEval, *currBldrCtx);
240a9ce612fSTed Kremenek 
241afe7c2c2SJordan Rose   for (ExplodedNodeSet::iterator DI = dstGenericPrevisit.begin(),
242afe7c2c2SJordan Rose        DE = dstGenericPrevisit.end(); DI != DE; ++DI) {
243a9ce612fSTed Kremenek     ExplodedNode *Pred = *DI;
244fcd016e5SJordan Rose     ProgramStateRef State = Pred->getState();
245fcd016e5SJordan Rose     CallEventRef<ObjCMethodCall> UpdatedMsg = Msg.cloneWithState(State);
246a9ce612fSTed Kremenek 
247fcd016e5SJordan Rose     if (UpdatedMsg->isInstanceMessage()) {
248fcd016e5SJordan Rose       SVal recVal = UpdatedMsg->getReceiverSVal();
249a9ce612fSTed Kremenek       if (!recVal.isUndef()) {
2508b3f9386STed Kremenek         if (ObjCNoRet.isImplicitNoReturn(ME)) {
251e594034fSAnna Zaks           // If we raise an exception, for now treat it as a sink.
252a9ce612fSTed Kremenek           // Eventually we will want to handle exceptions properly.
253642e0638SJordan Rose           Bldr.generateSink(ME, Pred, State);
2543136cf9dSAnna Zaks           continue;
2553136cf9dSAnna Zaks         }
256a9ce612fSTed Kremenek       }
2576bad4905SJordan Rose     } else {
2588b3f9386STed Kremenek       // Check for special class methods that are known to not return
2598b3f9386STed Kremenek       // and that we should treat as a sink.
2608b3f9386STed Kremenek       if (ObjCNoRet.isImplicitNoReturn(ME)) {
2613136cf9dSAnna Zaks         // If we raise an exception, for now treat it as a sink.
2623136cf9dSAnna Zaks         // Eventually we will want to handle exceptions properly.
263642e0638SJordan Rose         Bldr.generateSink(ME, Pred, Pred->getState());
2643136cf9dSAnna Zaks         continue;
2653136cf9dSAnna Zaks       }
26683f1495fSAnna Zaks     }
26783f1495fSAnna Zaks 
268fcd016e5SJordan Rose     defaultEvalCall(Bldr, Pred, *UpdatedMsg);
269a9ce612fSTed Kremenek   }
270a9ce612fSTed Kremenek 
2713ccf14ebSArtem Dergachev   // If there were constructors called for object-type arguments, clean them up.
2723ccf14ebSArtem Dergachev   ExplodedNodeSet dstArgCleanup;
2733ccf14ebSArtem Dergachev   for (auto I : dstEval)
2743ccf14ebSArtem Dergachev     finishArgumentConstruction(dstArgCleanup, I, *Msg);
2753ccf14ebSArtem Dergachev 
276afe7c2c2SJordan Rose   ExplodedNodeSet dstPostvisit;
2773ccf14ebSArtem Dergachev   getCheckerManager().runCheckersForPostCall(dstPostvisit, dstArgCleanup,
278fcd016e5SJordan Rose                                              *Msg, *this);
279afe7c2c2SJordan Rose 
280a9ce612fSTed Kremenek   // Finally, perform the post-condition check of the ObjCMessageExpr and store
281a9ce612fSTed Kremenek   // the created nodes in 'Dst'.
282afe7c2c2SJordan Rose   getCheckerManager().runCheckersForPostObjCMessage(Dst, dstPostvisit,
283fcd016e5SJordan Rose                                                     *Msg, *this);
284a9ce612fSTed Kremenek }
285