110e25031STed Kremenek //===- ExprEngineCXX.cpp - ExprEngine support for C++ -----------*- C++ -*-===// 210e25031STed 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 610e25031STed Kremenek // 710e25031STed Kremenek //===----------------------------------------------------------------------===// 810e25031STed Kremenek // 910e25031STed Kremenek // This file defines the C++ expression evaluation engine. 1010e25031STed Kremenek // 1110e25031STed Kremenek //===----------------------------------------------------------------------===// 1210e25031STed Kremenek 1310e25031STed Kremenek #include "clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h" 144068481bSArtem Dergachev #include "clang/Analysis/ConstructionContext.h" 1510e25031STed Kremenek #include "clang/AST/DeclCXX.h" 168fdb59f9STed Kremenek #include "clang/AST/StmtCXX.h" 17e8ba3ec4SArtem Dergachev #include "clang/AST/ParentMap.h" 1817a8757aSJordan Rose #include "clang/Basic/PrettyStackTrace.h" 193a02247dSChandler Carruth #include "clang/StaticAnalyzer/Core/CheckerManager.h" 203a02247dSChandler Carruth #include "clang/StaticAnalyzer/Core/PathSensitive/AnalysisManager.h" 213a02247dSChandler Carruth #include "clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h" 2210e25031STed Kremenek 2310e25031STed Kremenek using namespace clang; 2410e25031STed Kremenek using namespace ento; 2510e25031STed Kremenek 2610e25031STed Kremenek void ExprEngine::CreateCXXTemporaryObject(const MaterializeTemporaryExpr *ME, 2710e25031STed Kremenek ExplodedNode *Pred, 2810e25031STed Kremenek ExplodedNodeSet &Dst) { 29d94854a4STed Kremenek StmtNodeBuilder Bldr(Pred, Dst, *currBldrCtx); 302a14c695STed Kremenek const Expr *tempExpr = ME->GetTemporaryExpr()->IgnoreParens(); 3149b1e38eSTed Kremenek ProgramStateRef state = Pred->getState(); 32632e3b7eSTed Kremenek const LocationContext *LCtx = Pred->getLocationContext(); 3310e25031STed Kremenek 34861a1740SJordan Rose state = createTemporaryRegionIfNeeded(state, LCtx, tempExpr, ME); 355772f82dSJordan Rose Bldr.generateNode(ME, Pred, state); 3610e25031STed Kremenek } 3710e25031STed Kremenek 383d7b7f52SJordan Rose // FIXME: This is the sort of code that should eventually live in a Core 393d7b7f52SJordan Rose // checker rather than as a special case in ExprEngine. 4088bb563cSJordan Rose void ExprEngine::performTrivialCopy(NodeBuilder &Bldr, ExplodedNode *Pred, 413d7b7f52SJordan Rose const CallEvent &Call) { 423d7b7f52SJordan Rose SVal ThisVal; 433d7b7f52SJordan Rose bool AlwaysReturnsLValue; 444449e7f0SArtem Dergachev const CXXRecordDecl *ThisRD = nullptr; 453d7b7f52SJordan Rose if (const CXXConstructorCall *Ctor = dyn_cast<CXXConstructorCall>(&Call)) { 463d7b7f52SJordan Rose assert(Ctor->getDecl()->isTrivial()); 473d7b7f52SJordan Rose assert(Ctor->getDecl()->isCopyOrMoveConstructor()); 483d7b7f52SJordan Rose ThisVal = Ctor->getCXXThisVal(); 494449e7f0SArtem Dergachev ThisRD = Ctor->getDecl()->getParent(); 503d7b7f52SJordan Rose AlwaysReturnsLValue = false; 513d7b7f52SJordan Rose } else { 523d7b7f52SJordan Rose assert(cast<CXXMethodDecl>(Call.getDecl())->isTrivial()); 533d7b7f52SJordan Rose assert(cast<CXXMethodDecl>(Call.getDecl())->getOverloadedOperator() == 543d7b7f52SJordan Rose OO_Equal); 553d7b7f52SJordan Rose ThisVal = cast<CXXInstanceCall>(Call).getCXXThisVal(); 564449e7f0SArtem Dergachev ThisRD = cast<CXXMethodDecl>(Call.getDecl())->getParent(); 573d7b7f52SJordan Rose AlwaysReturnsLValue = true; 583d7b7f52SJordan Rose } 5988bb563cSJordan Rose 604449e7f0SArtem Dergachev assert(ThisRD); 614449e7f0SArtem Dergachev if (ThisRD->isEmpty()) { 624449e7f0SArtem Dergachev // Do nothing for empty classes. Otherwise it'd retrieve an UnknownVal 634449e7f0SArtem Dergachev // and bind it and RegionStore would think that the actual value 644449e7f0SArtem Dergachev // in this region at this offset is unknown. 654449e7f0SArtem Dergachev return; 664449e7f0SArtem Dergachev } 674449e7f0SArtem Dergachev 6888bb563cSJordan Rose const LocationContext *LCtx = Pred->getLocationContext(); 6988bb563cSJordan Rose 7088bb563cSJordan Rose ExplodedNodeSet Dst; 7188bb563cSJordan Rose Bldr.takeNodes(Pred); 7288bb563cSJordan Rose 7388bb563cSJordan Rose SVal V = Call.getArgSVal(0); 7488bb563cSJordan Rose 753d7b7f52SJordan Rose // If the value being copied is not unknown, load from its location to get 763d7b7f52SJordan Rose // an aggregate rvalue. 7705785d16SDavid Blaikie if (Optional<Loc> L = V.getAs<Loc>()) 7888bb563cSJordan Rose V = Pred->getState()->getSVal(*L); 793d7b7f52SJordan Rose else 8075f9d3acSArtem Dergachev assert(V.isUnknownOrUndef()); 8188bb563cSJordan Rose 823d7b7f52SJordan Rose const Expr *CallExpr = Call.getOriginExpr(); 833d7b7f52SJordan Rose evalBind(Dst, CallExpr, Pred, ThisVal, V, true); 8488bb563cSJordan Rose 853d7b7f52SJordan Rose PostStmt PS(CallExpr, LCtx); 8688bb563cSJordan Rose for (ExplodedNodeSet::iterator I = Dst.begin(), E = Dst.end(); 8788bb563cSJordan Rose I != E; ++I) { 8888bb563cSJordan Rose ProgramStateRef State = (*I)->getState(); 893d7b7f52SJordan Rose if (AlwaysReturnsLValue) 903d7b7f52SJordan Rose State = State->BindExpr(CallExpr, LCtx, ThisVal); 913d7b7f52SJordan Rose else 9288bb563cSJordan Rose State = bindReturnValue(Call, LCtx, State); 9388bb563cSJordan Rose Bldr.generateNode(PS, State, *I); 9488bb563cSJordan Rose } 9588bb563cSJordan Rose } 9688bb563cSJordan Rose 97bc74eb1cSJordan Rose 98168e29f6SArtem Dergachev SVal ExprEngine::makeZeroElementRegion(ProgramStateRef State, SVal LValue, 99690ab040SArtem Dergachev QualType &Ty, bool &IsArray) { 100bc74eb1cSJordan Rose SValBuilder &SVB = State->getStateManager().getSValBuilder(); 101bc74eb1cSJordan Rose ASTContext &Ctx = SVB.getContext(); 102bc74eb1cSJordan Rose 103bc74eb1cSJordan Rose while (const ArrayType *AT = Ctx.getAsArrayType(Ty)) { 104bc74eb1cSJordan Rose Ty = AT->getElementType(); 105bc74eb1cSJordan Rose LValue = State->getLValue(Ty, SVB.makeZeroArrayIndex(), LValue); 106690ab040SArtem Dergachev IsArray = true; 107bc74eb1cSJordan Rose } 108bc74eb1cSJordan Rose 109bc74eb1cSJordan Rose return LValue; 110bc74eb1cSJordan Rose } 111bc74eb1cSJordan Rose 112239452caSArtem Dergachev std::pair<ProgramStateRef, SVal> ExprEngine::prepareForObjectConstruction( 113239452caSArtem Dergachev const Expr *E, ProgramStateRef State, const LocationContext *LCtx, 114239452caSArtem Dergachev const ConstructionContext *CC, EvalCallOptions &CallOpts) { 11517906498SArtem Dergachev SValBuilder &SVB = getSValBuilder(); 11617906498SArtem Dergachev MemRegionManager &MRMgr = SVB.getRegionManager(); 11717906498SArtem Dergachev ASTContext &ACtx = SVB.getContext(); 11805375eb4SJordan Rose 119afb158c2SArtem Dergachev // See if we're constructing an existing region by looking at the 120afb158c2SArtem Dergachev // current construction context. 121661ab34aSArtem Dergachev if (CC) { 1224068481bSArtem Dergachev switch (CC->getKind()) { 12353b8ce0eSArtem Dergachev case ConstructionContext::CXX17ElidedCopyVariableKind: 1244068481bSArtem Dergachev case ConstructionContext::SimpleVariableKind: { 12553b8ce0eSArtem Dergachev const auto *DSCC = cast<VariableConstructionContext>(CC); 1264068481bSArtem Dergachev const auto *DS = DSCC->getDeclStmt(); 127e231bd34SArtem Dergachev const auto *Var = cast<VarDecl>(DS->getSingleDecl()); 128bc74eb1cSJordan Rose SVal LValue = State->getLValue(Var, LCtx); 12920edae87SJordan Rose QualType Ty = Var->getType(); 1301fe52474SArtem Dergachev LValue = 1311fe52474SArtem Dergachev makeZeroElementRegion(State, LValue, Ty, CallOpts.IsArrayCtorOrDtor); 1321fe52474SArtem Dergachev State = 1331fe52474SArtem Dergachev addObjectUnderConstruction(State, DSCC->getDeclStmt(), LCtx, LValue); 1341fe52474SArtem Dergachev return std::make_pair(State, LValue); 13520edae87SJordan Rose } 13653b8ce0eSArtem Dergachev case ConstructionContext::CXX17ElidedCopyConstructorInitializerKind: 137922455feSArtem Dergachev case ConstructionContext::SimpleConstructorInitializerKind: { 1384068481bSArtem Dergachev const auto *ICC = cast<ConstructorInitializerConstructionContext>(CC); 1394068481bSArtem Dergachev const auto *Init = ICC->getCXXCtorInitializer(); 14054529a34SJordan Rose assert(Init->isAnyMemberInitializer()); 14154529a34SJordan Rose const CXXMethodDecl *CurCtor = cast<CXXMethodDecl>(LCtx->getDecl()); 142412c0af2SDevin Coughlin Loc ThisPtr = 14317906498SArtem Dergachev SVB.getCXXThis(CurCtor, LCtx->getStackFrame()); 14454529a34SJordan Rose SVal ThisVal = State->getSVal(ThisPtr); 14554529a34SJordan Rose 146bc74eb1cSJordan Rose const ValueDecl *Field; 147bc74eb1cSJordan Rose SVal FieldVal; 14854529a34SJordan Rose if (Init->isIndirectMemberInitializer()) { 149bc74eb1cSJordan Rose Field = Init->getIndirectMember(); 150bc74eb1cSJordan Rose FieldVal = State->getLValue(Init->getIndirectMember(), ThisVal); 15154529a34SJordan Rose } else { 152bc74eb1cSJordan Rose Field = Init->getMember(); 153bc74eb1cSJordan Rose FieldVal = State->getLValue(Init->getMember(), ThisVal); 15454529a34SJordan Rose } 155bc74eb1cSJordan Rose 156bc74eb1cSJordan Rose QualType Ty = Field->getType(); 157690ab040SArtem Dergachev FieldVal = makeZeroElementRegion(State, FieldVal, Ty, 158168e29f6SArtem Dergachev CallOpts.IsArrayCtorOrDtor); 159a84374dcSArtem Dergachev State = addObjectUnderConstruction(State, Init, LCtx, FieldVal); 160239452caSArtem Dergachev return std::make_pair(State, FieldVal); 16154529a34SJordan Rose } 1624068481bSArtem Dergachev case ConstructionContext::NewAllocatedObjectKind: { 163549f9cd4SKristof Umann if (AMgr.getAnalyzerOptions().MayInlineCXXAllocator) { 1644068481bSArtem Dergachev const auto *NECC = cast<NewAllocatedObjectConstructionContext>(CC); 1654068481bSArtem Dergachev const auto *NE = NECC->getCXXNewExpr(); 166f83d5479SArtem Dergachev SVal V = *getObjectUnderConstruction(State, NE, LCtx); 167f83d5479SArtem Dergachev if (const SubRegion *MR = 168f83d5479SArtem Dergachev dyn_cast_or_null<SubRegion>(V.getAsRegion())) { 1694068481bSArtem Dergachev if (NE->isArray()) { 1704068481bSArtem Dergachev // TODO: In fact, we need to call the constructor for every 1714068481bSArtem Dergachev // allocated element, not just the first one! 1724068481bSArtem Dergachev CallOpts.IsArrayCtorOrDtor = true; 173239452caSArtem Dergachev return std::make_pair( 174239452caSArtem Dergachev State, loc::MemRegionVal(getStoreManager().GetElementZeroRegion( 175239452caSArtem Dergachev MR, NE->getType()->getPointeeType()))); 1764068481bSArtem Dergachev } 177239452caSArtem Dergachev return std::make_pair(State, V); 1784068481bSArtem Dergachev } 179f83d5479SArtem Dergachev // TODO: Detect when the allocator returns a null pointer. 180f83d5479SArtem Dergachev // Constructor shall not be called in this case. 1814068481bSArtem Dergachev } 1824068481bSArtem Dergachev break; 1834068481bSArtem Dergachev } 184f28d7f17SArtem Dergachev case ConstructionContext::SimpleReturnedValueKind: 185f28d7f17SArtem Dergachev case ConstructionContext::CXX17ElidedCopyReturnedValueKind: { 186239452caSArtem Dergachev // The temporary is to be managed by the parent stack frame. 187239452caSArtem Dergachev // So build it in the parent stack frame if we're not in the 188239452caSArtem Dergachev // top frame of the analysis. 189dd18b11bSGeorge Karpenkov const StackFrameContext *SFC = LCtx->getStackFrame(); 190239452caSArtem Dergachev if (const LocationContext *CallerLCtx = SFC->getParent()) { 191239452caSArtem Dergachev auto RTC = (*SFC->getCallSiteBlock())[SFC->getIndex()] 192239452caSArtem Dergachev .getAs<CFGCXXRecordTypedCall>(); 193239452caSArtem Dergachev if (!RTC) { 194239452caSArtem Dergachev // We were unable to find the correct construction context for the 195239452caSArtem Dergachev // call in the parent stack frame. This is equivalent to not being 196239452caSArtem Dergachev // able to find construction context at all. 197239452caSArtem Dergachev break; 198239452caSArtem Dergachev } 199*b3fc9df4SArtem Dergachev if (isa<BlockInvocationContext>(CallerLCtx)) { 200*b3fc9df4SArtem Dergachev // Unwrap block invocation contexts. They're mostly part of 201*b3fc9df4SArtem Dergachev // the current stack frame. 202*b3fc9df4SArtem Dergachev CallerLCtx = CallerLCtx->getParent(); 203*b3fc9df4SArtem Dergachev assert(!isa<BlockInvocationContext>(CallerLCtx)); 204*b3fc9df4SArtem Dergachev } 205f28d7f17SArtem Dergachev return prepareForObjectConstruction( 206f28d7f17SArtem Dergachev cast<Expr>(SFC->getCallSite()), State, CallerLCtx, 207f28d7f17SArtem Dergachev RTC->getConstructionContext(), CallOpts); 208239452caSArtem Dergachev } else { 20917906498SArtem Dergachev // We are on the top frame of the analysis. We do not know where is the 21017906498SArtem Dergachev // object returned to. Conjure a symbolic region for the return value. 21117906498SArtem Dergachev // TODO: We probably need a new MemRegion kind to represent the storage 21217906498SArtem Dergachev // of that SymbolicRegion, so that we cound produce a fancy symbol 21317906498SArtem Dergachev // instead of an anonymous conjured symbol. 21417906498SArtem Dergachev // TODO: Do we need to track the region to avoid having it dead 21517906498SArtem Dergachev // too early? It does die too early, at least in C++17, but because 21617906498SArtem Dergachev // putting anything into a SymbolicRegion causes an immediate escape, 21717906498SArtem Dergachev // it doesn't cause any leak false positives. 21817906498SArtem Dergachev const auto *RCC = cast<ReturnedValueConstructionContext>(CC); 21917906498SArtem Dergachev // Make sure that this doesn't coincide with any other symbol 22017906498SArtem Dergachev // conjured for the returned expression. 22117906498SArtem Dergachev static const int TopLevelSymRegionTag = 0; 22217906498SArtem Dergachev const Expr *RetE = RCC->getReturnStmt()->getRetValue(); 22317906498SArtem Dergachev assert(RetE && "Void returns should not have a construction context"); 22417906498SArtem Dergachev QualType ReturnTy = RetE->getType(); 22517906498SArtem Dergachev QualType RegionTy = ACtx.getPointerType(ReturnTy); 22617906498SArtem Dergachev SVal V = SVB.conjureSymbolVal(&TopLevelSymRegionTag, RetE, SFC, 22717906498SArtem Dergachev RegionTy, currBldrCtx->blockCount()); 228239452caSArtem Dergachev return std::make_pair(State, V); 229239452caSArtem Dergachev } 230f28d7f17SArtem Dergachev llvm_unreachable("Unhandled return value construction context!"); 231239452caSArtem Dergachev } 2329a209ad1SArtem Dergachev case ConstructionContext::ElidedTemporaryObjectKind: { 233549f9cd4SKristof Umann assert(AMgr.getAnalyzerOptions().ShouldElideConstructors); 2349a209ad1SArtem Dergachev const auto *TCC = cast<ElidedTemporaryObjectConstructionContext>(CC); 2359a209ad1SArtem Dergachev const CXXBindTemporaryExpr *BTE = TCC->getCXXBindTemporaryExpr(); 2369a209ad1SArtem Dergachev const MaterializeTemporaryExpr *MTE = TCC->getMaterializedTemporaryExpr(); 2379a209ad1SArtem Dergachev const CXXConstructExpr *CE = TCC->getConstructorAfterElision(); 2389a209ad1SArtem Dergachev 2399a209ad1SArtem Dergachev // Support pre-C++17 copy elision. We'll have the elidable copy 2409a209ad1SArtem Dergachev // constructor in the AST and in the CFG, but we'll skip it 2419a209ad1SArtem Dergachev // and construct directly into the final object. This call 2429a209ad1SArtem Dergachev // also sets the CallOpts flags for us. 2439a209ad1SArtem Dergachev SVal V; 244a657a32cSArtem Dergachev // If the elided copy/move constructor is not supported, there's still 245a657a32cSArtem Dergachev // benefit in trying to model the non-elided constructor. 246a657a32cSArtem Dergachev // Stash our state before trying to elide, as it'll get overwritten. 247a657a32cSArtem Dergachev ProgramStateRef PreElideState = State; 248a657a32cSArtem Dergachev EvalCallOptions PreElideCallOpts = CallOpts; 249a657a32cSArtem Dergachev 2509a209ad1SArtem Dergachev std::tie(State, V) = prepareForObjectConstruction( 2519a209ad1SArtem Dergachev CE, State, LCtx, TCC->getConstructionContextAfterElision(), CallOpts); 2529a209ad1SArtem Dergachev 253a657a32cSArtem Dergachev // FIXME: This definition of "copy elision has not failed" is unreliable. 254a657a32cSArtem Dergachev // It doesn't indicate that the constructor will actually be inlined 255a657a32cSArtem Dergachev // later; it is still up to evalCall() to decide. 256a657a32cSArtem Dergachev if (!CallOpts.IsCtorOrDtorWithImproperlyModeledTargetRegion) { 2579a209ad1SArtem Dergachev // Remember that we've elided the constructor. 2589a209ad1SArtem Dergachev State = addObjectUnderConstruction(State, CE, LCtx, V); 2599a209ad1SArtem Dergachev 2609a209ad1SArtem Dergachev // Remember that we've elided the destructor. 2619a209ad1SArtem Dergachev if (BTE) 2629a209ad1SArtem Dergachev State = elideDestructor(State, BTE, LCtx); 2639a209ad1SArtem Dergachev 2649a209ad1SArtem Dergachev // Instead of materialization, shamelessly return 2659a209ad1SArtem Dergachev // the final object destination. 2669a209ad1SArtem Dergachev if (MTE) 2679a209ad1SArtem Dergachev State = addObjectUnderConstruction(State, MTE, LCtx, V); 2689a209ad1SArtem Dergachev 2699a209ad1SArtem Dergachev return std::make_pair(State, V); 270a657a32cSArtem Dergachev } else { 271a657a32cSArtem Dergachev // Copy elision failed. Revert the changes and proceed as if we have 272a657a32cSArtem Dergachev // a simple temporary. 273a657a32cSArtem Dergachev State = PreElideState; 274a657a32cSArtem Dergachev CallOpts = PreElideCallOpts; 275a657a32cSArtem Dergachev } 276080b8709SSimon Pilgrim LLVM_FALLTHROUGH; 2779a209ad1SArtem Dergachev } 278ff267df0SArtem Dergachev case ConstructionContext::SimpleTemporaryObjectKind: { 279a657a32cSArtem Dergachev const auto *TCC = cast<TemporaryObjectConstructionContext>(CC); 280239452caSArtem Dergachev const CXXBindTemporaryExpr *BTE = TCC->getCXXBindTemporaryExpr(); 281239452caSArtem Dergachev const MaterializeTemporaryExpr *MTE = TCC->getMaterializedTemporaryExpr(); 282d18639bdSArtem Dergachev SVal V = UnknownVal(); 283239452caSArtem Dergachev 284239452caSArtem Dergachev if (MTE) { 2858cd7961aSArtem Dergachev if (const ValueDecl *VD = MTE->getExtendingDecl()) { 286643102dfSArtem Dergachev assert(MTE->getStorageDuration() != SD_FullExpression); 287643102dfSArtem Dergachev if (!VD->getType()->isReferenceType()) { 288643102dfSArtem Dergachev // We're lifetime-extended by a surrounding aggregate. 289643102dfSArtem Dergachev // Automatic destructors aren't quite working in this case 290643102dfSArtem Dergachev // on the CFG side. We should warn the caller about that. 291310bca01SArtem Dergachev // FIXME: Is there a better way to retrieve this information from 292310bca01SArtem Dergachev // the MaterializeTemporaryExpr? 293310bca01SArtem Dergachev CallOpts.IsTemporaryLifetimeExtendedViaAggregate = true; 294310bca01SArtem Dergachev } 2958cd7961aSArtem Dergachev } 2964fb586c3SArtem Dergachev 2974fb586c3SArtem Dergachev if (MTE->getStorageDuration() == SD_Static || 2984fb586c3SArtem Dergachev MTE->getStorageDuration() == SD_Thread) 2994fb586c3SArtem Dergachev V = loc::MemRegionVal(MRMgr.getCXXStaticTempObjectRegion(E)); 300239452caSArtem Dergachev } 301239452caSArtem Dergachev 3024fb586c3SArtem Dergachev if (V.isUnknown()) 3034fb586c3SArtem Dergachev V = loc::MemRegionVal(MRMgr.getCXXTempObjectRegion(E, LCtx)); 304239452caSArtem Dergachev 305239452caSArtem Dergachev if (BTE) 306239452caSArtem Dergachev State = addObjectUnderConstruction(State, BTE, LCtx, V); 307239452caSArtem Dergachev 308239452caSArtem Dergachev if (MTE) 309239452caSArtem Dergachev State = addObjectUnderConstruction(State, MTE, LCtx, V); 310239452caSArtem Dergachev 3114068481bSArtem Dergachev CallOpts.IsTemporaryCtorOrDtor = true; 312239452caSArtem Dergachev return std::make_pair(State, V); 3134068481bSArtem Dergachev } 314a657a32cSArtem Dergachev case ConstructionContext::ArgumentKind: { 3153ccf14ebSArtem Dergachev // Arguments are technically temporaries. 3163ccf14ebSArtem Dergachev CallOpts.IsTemporaryCtorOrDtor = true; 3173ccf14ebSArtem Dergachev 3183ccf14ebSArtem Dergachev const auto *ACC = cast<ArgumentConstructionContext>(CC); 3193ccf14ebSArtem Dergachev const Expr *E = ACC->getCallLikeExpr(); 3203ccf14ebSArtem Dergachev unsigned Idx = ACC->getIndex(); 3213ccf14ebSArtem Dergachev const CXXBindTemporaryExpr *BTE = ACC->getCXXBindTemporaryExpr(); 3223ccf14ebSArtem Dergachev 3233ccf14ebSArtem Dergachev CallEventManager &CEMgr = getStateManager().getCallEventManager(); 3243ccf14ebSArtem Dergachev SVal V = UnknownVal(); 3253ccf14ebSArtem Dergachev auto getArgLoc = [&](CallEventRef<> Caller) -> Optional<SVal> { 3263ccf14ebSArtem Dergachev const LocationContext *FutureSFC = Caller->getCalleeStackFrame(); 3273ccf14ebSArtem Dergachev // Return early if we are unable to reliably foresee 3283ccf14ebSArtem Dergachev // the future stack frame. 3293ccf14ebSArtem Dergachev if (!FutureSFC) 3303ccf14ebSArtem Dergachev return None; 3313ccf14ebSArtem Dergachev 3323ccf14ebSArtem Dergachev // This should be equivalent to Caller->getDecl() for now, but 3333ccf14ebSArtem Dergachev // FutureSFC->getDecl() is likely to support better stuff (like 3343ccf14ebSArtem Dergachev // virtual functions) earlier. 3353ccf14ebSArtem Dergachev const Decl *CalleeD = FutureSFC->getDecl(); 3363ccf14ebSArtem Dergachev 3373ccf14ebSArtem Dergachev // FIXME: Support for variadic arguments is not implemented here yet. 3383ccf14ebSArtem Dergachev if (CallEvent::isVariadic(CalleeD)) 3393ccf14ebSArtem Dergachev return None; 3403ccf14ebSArtem Dergachev 3413ccf14ebSArtem Dergachev // Operator arguments do not correspond to operator parameters 3423ccf14ebSArtem Dergachev // because this-argument is implemented as a normal argument in 3433ccf14ebSArtem Dergachev // operator call expressions but not in operator declarations. 3443ccf14ebSArtem Dergachev const VarRegion *VR = Caller->getParameterLocation( 3453ccf14ebSArtem Dergachev *Caller->getAdjustedParameterIndex(Idx)); 3463ccf14ebSArtem Dergachev if (!VR) 3473ccf14ebSArtem Dergachev return None; 3483ccf14ebSArtem Dergachev 3493ccf14ebSArtem Dergachev return loc::MemRegionVal(VR); 3503ccf14ebSArtem Dergachev }; 3513ccf14ebSArtem Dergachev 3523ccf14ebSArtem Dergachev if (const auto *CE = dyn_cast<CallExpr>(E)) { 3533ccf14ebSArtem Dergachev CallEventRef<> Caller = CEMgr.getSimpleCall(CE, State, LCtx); 3543ccf14ebSArtem Dergachev if (auto OptV = getArgLoc(Caller)) 3553ccf14ebSArtem Dergachev V = *OptV; 3563ccf14ebSArtem Dergachev else 357a657a32cSArtem Dergachev break; 3583ccf14ebSArtem Dergachev State = addObjectUnderConstruction(State, {CE, Idx}, LCtx, V); 3593ccf14ebSArtem Dergachev } else if (const auto *CCE = dyn_cast<CXXConstructExpr>(E)) { 3603ccf14ebSArtem Dergachev // Don't bother figuring out the target region for the future 3613ccf14ebSArtem Dergachev // constructor because we won't need it. 3623ccf14ebSArtem Dergachev CallEventRef<> Caller = 3633ccf14ebSArtem Dergachev CEMgr.getCXXConstructorCall(CCE, /*Target=*/nullptr, State, LCtx); 3643ccf14ebSArtem Dergachev if (auto OptV = getArgLoc(Caller)) 3653ccf14ebSArtem Dergachev V = *OptV; 3663ccf14ebSArtem Dergachev else 3673ccf14ebSArtem Dergachev break; 3683ccf14ebSArtem Dergachev State = addObjectUnderConstruction(State, {CCE, Idx}, LCtx, V); 3693ccf14ebSArtem Dergachev } else if (const auto *ME = dyn_cast<ObjCMessageExpr>(E)) { 3703ccf14ebSArtem Dergachev CallEventRef<> Caller = CEMgr.getObjCMethodCall(ME, State, LCtx); 3713ccf14ebSArtem Dergachev if (auto OptV = getArgLoc(Caller)) 3723ccf14ebSArtem Dergachev V = *OptV; 3733ccf14ebSArtem Dergachev else 3743ccf14ebSArtem Dergachev break; 3753ccf14ebSArtem Dergachev State = addObjectUnderConstruction(State, {ME, Idx}, LCtx, V); 3763ccf14ebSArtem Dergachev } 3773ccf14ebSArtem Dergachev 3783ccf14ebSArtem Dergachev assert(!V.isUnknown()); 3793ccf14ebSArtem Dergachev 3803ccf14ebSArtem Dergachev if (BTE) 3813ccf14ebSArtem Dergachev State = addObjectUnderConstruction(State, BTE, LCtx, V); 3823ccf14ebSArtem Dergachev 3833ccf14ebSArtem Dergachev return std::make_pair(State, V); 384a657a32cSArtem Dergachev } 3854068481bSArtem Dergachev } 38605375eb4SJordan Rose } 387c9318304SJordan Rose // If we couldn't find an existing region to construct into, assume we're 388690ab040SArtem Dergachev // constructing a temporary. Notify the caller of our failure. 389168e29f6SArtem Dergachev CallOpts.IsCtorOrDtorWithImproperlyModeledTargetRegion = true; 390239452caSArtem Dergachev return std::make_pair( 391239452caSArtem Dergachev State, loc::MemRegionVal(MRMgr.getCXXTempObjectRegion(E, LCtx))); 392c9318304SJordan Rose } 39305375eb4SJordan Rose 39417f41608SJordan Rose void ExprEngine::VisitCXXConstructExpr(const CXXConstructExpr *CE, 39517f41608SJordan Rose ExplodedNode *Pred, 39617f41608SJordan Rose ExplodedNodeSet &destNodes) { 39717f41608SJordan Rose const LocationContext *LCtx = Pred->getLocationContext(); 39817f41608SJordan Rose ProgramStateRef State = Pred->getState(); 39917f41608SJordan Rose 400f83d5479SArtem Dergachev SVal Target = UnknownVal(); 40117f41608SJordan Rose 4029a209ad1SArtem Dergachev if (Optional<SVal> ElidedTarget = 4039a209ad1SArtem Dergachev getObjectUnderConstruction(State, CE, LCtx)) { 4049a209ad1SArtem Dergachev // We've previously modeled an elidable constructor by pretending that it in 4059a209ad1SArtem Dergachev // fact constructs into the correct target. This constructor can therefore 4069a209ad1SArtem Dergachev // be skipped. 4079a209ad1SArtem Dergachev Target = *ElidedTarget; 4089a209ad1SArtem Dergachev StmtNodeBuilder Bldr(Pred, destNodes, *currBldrCtx); 4099a209ad1SArtem Dergachev State = finishObjectConstruction(State, CE, LCtx); 4109a209ad1SArtem Dergachev if (auto L = Target.getAs<Loc>()) 4119a209ad1SArtem Dergachev State = State->BindExpr(CE, LCtx, State->getSVal(*L, CE->getType())); 4129a209ad1SArtem Dergachev Bldr.generateNode(CE, Pred, State); 4139a209ad1SArtem Dergachev return; 4149a209ad1SArtem Dergachev } 4159a209ad1SArtem Dergachev 41617f41608SJordan Rose // FIXME: Handle arrays, which run the same constructor for every element. 41717f41608SJordan Rose // For now, we just run the first constructor (which should still invalidate 41817f41608SJordan Rose // the entire array). 41917f41608SJordan Rose 420690ab040SArtem Dergachev EvalCallOptions CallOpts; 421661ab34aSArtem Dergachev auto C = getCurrentCFGElement().getAs<CFGConstructor>(); 4224068481bSArtem Dergachev assert(C || getCurrentCFGElement().getAs<CFGStmt>()); 423661ab34aSArtem Dergachev const ConstructionContext *CC = C ? C->getConstructionContext() : nullptr; 424661ab34aSArtem Dergachev 42517f41608SJordan Rose switch (CE->getConstructionKind()) { 42617f41608SJordan Rose case CXXConstructExpr::CK_Complete: { 427239452caSArtem Dergachev std::tie(State, Target) = 428239452caSArtem Dergachev prepareForObjectConstruction(CE, State, LCtx, CC, CallOpts); 429b970505dSJordan Rose break; 430b970505dSJordan Rose } 431b3b976f0SJordan Rose case CXXConstructExpr::CK_VirtualBase: 432b3b976f0SJordan Rose // Make sure we are not calling virtual base class initializers twice. 433b3b976f0SJordan Rose // Only the most-derived object should initialize virtual base classes. 434dd18b11bSGeorge Karpenkov if (const Stmt *Outer = LCtx->getStackFrame()->getCallSite()) { 435b3b976f0SJordan Rose const CXXConstructExpr *OuterCtor = dyn_cast<CXXConstructExpr>(Outer); 436b3b976f0SJordan Rose if (OuterCtor) { 437b3b976f0SJordan Rose switch (OuterCtor->getConstructionKind()) { 438b970505dSJordan Rose case CXXConstructExpr::CK_NonVirtualBase: 439b970505dSJordan Rose case CXXConstructExpr::CK_VirtualBase: 440b3b976f0SJordan Rose // Bail out! 441b3b976f0SJordan Rose destNodes.Add(Pred); 442b3b976f0SJordan Rose return; 443b3b976f0SJordan Rose case CXXConstructExpr::CK_Complete: 444b3b976f0SJordan Rose case CXXConstructExpr::CK_Delegating: 445b3b976f0SJordan Rose break; 446b3b976f0SJordan Rose } 447b3b976f0SJordan Rose } 448b3b976f0SJordan Rose } 4494dc0b1acSReid Kleckner LLVM_FALLTHROUGH; 450b3b976f0SJordan Rose case CXXConstructExpr::CK_NonVirtualBase: 451e8ba3ec4SArtem Dergachev // In C++17, classes with non-virtual bases may be aggregates, so they would 452e8ba3ec4SArtem Dergachev // be initialized as aggregates without a constructor call, so we may have 453e8ba3ec4SArtem Dergachev // a base class constructed directly into an initializer list without 454e8ba3ec4SArtem Dergachev // having the derived-class constructor call on the previous stack frame. 455e8ba3ec4SArtem Dergachev // Initializer lists may be nested into more initializer lists that 456e8ba3ec4SArtem Dergachev // correspond to surrounding aggregate initializations. 457e8ba3ec4SArtem Dergachev // FIXME: For now this code essentially bails out. We need to find the 458e8ba3ec4SArtem Dergachev // correct target region and set it. 459e8ba3ec4SArtem Dergachev // FIXME: Instead of relying on the ParentMap, we should have the 460e8ba3ec4SArtem Dergachev // trigger-statement (InitListExpr in this case) passed down from CFG or 461e8ba3ec4SArtem Dergachev // otherwise always available during construction. 462e8ba3ec4SArtem Dergachev if (dyn_cast_or_null<InitListExpr>(LCtx->getParentMap().getParent(CE))) { 463e8ba3ec4SArtem Dergachev MemRegionManager &MRMgr = getSValBuilder().getRegionManager(); 464f83d5479SArtem Dergachev Target = loc::MemRegionVal(MRMgr.getCXXTempObjectRegion(CE, LCtx)); 465168e29f6SArtem Dergachev CallOpts.IsCtorOrDtorWithImproperlyModeledTargetRegion = true; 466e8ba3ec4SArtem Dergachev break; 467e8ba3ec4SArtem Dergachev } 4684dc0b1acSReid Kleckner LLVM_FALLTHROUGH; 469b970505dSJordan Rose case CXXConstructExpr::CK_Delegating: { 470b970505dSJordan Rose const CXXMethodDecl *CurCtor = cast<CXXMethodDecl>(LCtx->getDecl()); 471b970505dSJordan Rose Loc ThisPtr = getSValBuilder().getCXXThis(CurCtor, 472dd18b11bSGeorge Karpenkov LCtx->getStackFrame()); 47305375eb4SJordan Rose SVal ThisVal = State->getSVal(ThisPtr); 474b970505dSJordan Rose 475b970505dSJordan Rose if (CE->getConstructionKind() == CXXConstructExpr::CK_Delegating) { 476f83d5479SArtem Dergachev Target = ThisVal; 477b970505dSJordan Rose } else { 478b970505dSJordan Rose // Cast to the base type. 479599f85abSJordan Rose bool IsVirtual = 480599f85abSJordan Rose (CE->getConstructionKind() == CXXConstructExpr::CK_VirtualBase); 481599f85abSJordan Rose SVal BaseVal = getStoreManager().evalDerivedToBase(ThisVal, CE->getType(), 482599f85abSJordan Rose IsVirtual); 483f83d5479SArtem Dergachev Target = BaseVal; 484b970505dSJordan Rose } 485b970505dSJordan Rose break; 486b970505dSJordan Rose } 487b970505dSJordan Rose } 488b970505dSJordan Rose 489239452caSArtem Dergachev if (State != Pred->getState()) { 490239452caSArtem Dergachev static SimpleProgramPointTag T("ExprEngine", 491239452caSArtem Dergachev "Prepare for object construction"); 492239452caSArtem Dergachev ExplodedNodeSet DstPrepare; 493239452caSArtem Dergachev StmtNodeBuilder BldrPrepare(Pred, DstPrepare, *currBldrCtx); 494239452caSArtem Dergachev BldrPrepare.generateNode(CE, Pred, State, &T, ProgramPoint::PreStmtKind); 495239452caSArtem Dergachev assert(DstPrepare.size() <= 1); 496239452caSArtem Dergachev if (DstPrepare.size() == 0) 497239452caSArtem Dergachev return; 498239452caSArtem Dergachev Pred = *BldrPrepare.begin(); 499239452caSArtem Dergachev } 500239452caSArtem Dergachev 501fcd016e5SJordan Rose CallEventManager &CEMgr = getStateManager().getCallEventManager(); 502fcd016e5SJordan Rose CallEventRef<CXXConstructorCall> Call = 503f83d5479SArtem Dergachev CEMgr.getCXXConstructorCall(CE, Target.getAsRegion(), State, LCtx); 50410e25031STed Kremenek 505f3c12ac3SJordan Rose ExplodedNodeSet DstPreVisit; 506f3c12ac3SJordan Rose getCheckerManager().runCheckersForPreStmt(DstPreVisit, Pred, CE, *this); 507b8e28654SJordan Rose 508661ab34aSArtem Dergachev // FIXME: Is it possible and/or useful to do this before PreStmt? 509b8e28654SJordan Rose ExplodedNodeSet PreInitialized; 510b8e28654SJordan Rose { 511b8e28654SJordan Rose StmtNodeBuilder Bldr(DstPreVisit, PreInitialized, *currBldrCtx); 512b8e28654SJordan Rose for (ExplodedNodeSet::iterator I = DstPreVisit.begin(), 513b8e28654SJordan Rose E = DstPreVisit.end(); 514b8e28654SJordan Rose I != E; ++I) { 515b8e28654SJordan Rose ProgramStateRef State = (*I)->getState(); 516661ab34aSArtem Dergachev if (CE->requiresZeroInitialization()) { 517b8e28654SJordan Rose // FIXME: Once we properly handle constructors in new-expressions, we'll 518b8e28654SJordan Rose // need to invalidate the region before setting a default value, to make 519b8e28654SJordan Rose // sure there aren't any lingering bindings around. This probably needs 520b8e28654SJordan Rose // to happen regardless of whether or not the object is zero-initialized 521b8e28654SJordan Rose // to handle random fields of a placement-initialized object picking up 522b8e28654SJordan Rose // old bindings. We might only want to do it when we need to, though. 523b8e28654SJordan Rose // FIXME: This isn't actually correct for arrays -- we need to zero- 524b8e28654SJordan Rose // initialize the entire array, not just the first element -- but our 525b8e28654SJordan Rose // handling of arrays everywhere else is weak as well, so this shouldn't 526b8e28654SJordan Rose // actually make things worse. Placement new makes this tricky as well, 527b8e28654SJordan Rose // since it's then possible to be initializing one part of a multi- 528b8e28654SJordan Rose // dimensional array. 529f83d5479SArtem Dergachev State = State->bindDefaultZero(Target, LCtx); 530661ab34aSArtem Dergachev } 531661ab34aSArtem Dergachev 5320dbb783cSCraig Topper Bldr.generateNode(CE, *I, State, /*tag=*/nullptr, 5330dbb783cSCraig Topper ProgramPoint::PreStmtKind); 534b8e28654SJordan Rose } 535b8e28654SJordan Rose } 536b8e28654SJordan Rose 537afe7c2c2SJordan Rose ExplodedNodeSet DstPreCall; 538b8e28654SJordan Rose getCheckerManager().runCheckersForPreCall(DstPreCall, PreInitialized, 539fcd016e5SJordan Rose *Call, *this); 54010e25031STed Kremenek 54188bb563cSJordan Rose ExplodedNodeSet DstEvaluated; 54288bb563cSJordan Rose StmtNodeBuilder Bldr(DstPreCall, DstEvaluated, *currBldrCtx); 54388bb563cSJordan Rose 544e83cb092SJordan Rose if (CE->getConstructor()->isTrivial() && 545e83cb092SJordan Rose CE->getConstructor()->isCopyOrMoveConstructor() && 546168e29f6SArtem Dergachev !CallOpts.IsArrayCtorOrDtor) { 547e83cb092SJordan Rose // FIXME: Handle other kinds of trivial constructors as well. 548e83cb092SJordan Rose for (ExplodedNodeSet::iterator I = DstPreCall.begin(), E = DstPreCall.end(); 54988bb563cSJordan Rose I != E; ++I) 55088bb563cSJordan Rose performTrivialCopy(Bldr, *I, *Call); 55188bb563cSJordan Rose 55288bb563cSJordan Rose } else { 553afe7c2c2SJordan Rose for (ExplodedNodeSet::iterator I = DstPreCall.begin(), E = DstPreCall.end(); 554f3c12ac3SJordan Rose I != E; ++I) 555690ab040SArtem Dergachev defaultEvalCall(Bldr, *I, *Call, CallOpts); 55688bb563cSJordan Rose } 55710e25031STed Kremenek 5582a8c18d9SAlexander Kornienko // If the CFG was constructed without elements for temporary destructors 5595b1ee2faSDevin Coughlin // and the just-called constructor created a temporary object then 5605b1ee2faSDevin Coughlin // stop exploration if the temporary object has a noreturn constructor. 5615b1ee2faSDevin Coughlin // This can lose coverage because the destructor, if it were present 5625b1ee2faSDevin Coughlin // in the CFG, would be called at the end of the full expression or 5635b1ee2faSDevin Coughlin // later (for life-time extended temporaries) -- but avoids infeasible 5645b1ee2faSDevin Coughlin // paths when no-return temporary destructors are used for assertions. 5655b1ee2faSDevin Coughlin const AnalysisDeclContext *ADC = LCtx->getAnalysisDeclContext(); 5665b1ee2faSDevin Coughlin if (!ADC->getCFGBuildOptions().AddTemporaryDtors) { 5675b1ee2faSDevin Coughlin const MemRegion *Target = Call->getCXXThisVal().getAsRegion(); 5685b1ee2faSDevin Coughlin if (Target && isa<CXXTempObjectRegion>(Target) && 5695b1ee2faSDevin Coughlin Call->getDecl()->getParent()->isAnyDestructorNoReturn()) { 5705b1ee2faSDevin Coughlin 5713da72051SArtem Dergachev // If we've inlined the constructor, then DstEvaluated would be empty. 5723da72051SArtem Dergachev // In this case we still want a sink, which could be implemented 5733da72051SArtem Dergachev // in processCallExit. But we don't have that implemented at the moment, 5743da72051SArtem Dergachev // so if you hit this assertion, see if you can avoid inlining 5753da72051SArtem Dergachev // the respective constructor when analyzer-config cfg-temporary-dtors 5763da72051SArtem Dergachev // is set to false. 5773da72051SArtem Dergachev // Otherwise there's nothing wrong with inlining such constructor. 5783da72051SArtem Dergachev assert(!DstEvaluated.empty() && 5793da72051SArtem Dergachev "We should not have inlined this constructor!"); 5803da72051SArtem Dergachev 5815b1ee2faSDevin Coughlin for (ExplodedNode *N : DstEvaluated) { 5825b1ee2faSDevin Coughlin Bldr.generateSink(CE, N, N->getState()); 5835b1ee2faSDevin Coughlin } 5845b1ee2faSDevin Coughlin 5855b1ee2faSDevin Coughlin // There is no need to run the PostCall and PostStmt checker 5865b1ee2faSDevin Coughlin // callbacks because we just generated sinks on all nodes in th 5875b1ee2faSDevin Coughlin // frontier. 5885b1ee2faSDevin Coughlin return; 5895b1ee2faSDevin Coughlin } 5905b1ee2faSDevin Coughlin } 5915b1ee2faSDevin Coughlin 5923ccf14ebSArtem Dergachev ExplodedNodeSet DstPostArgumentCleanup; 5933ccf14ebSArtem Dergachev for (auto I : DstEvaluated) 5943ccf14ebSArtem Dergachev finishArgumentConstruction(DstPostArgumentCleanup, I, *Call); 5953ccf14ebSArtem Dergachev 5963ccf14ebSArtem Dergachev // If there were other constructors called for object-type arguments 5973ccf14ebSArtem Dergachev // of this constructor, clean them up. 598afe7c2c2SJordan Rose ExplodedNodeSet DstPostCall; 5993ccf14ebSArtem Dergachev getCheckerManager().runCheckersForPostCall(DstPostCall, 6003ccf14ebSArtem Dergachev DstPostArgumentCleanup, 601fcd016e5SJordan Rose *Call, *this); 602afe7c2c2SJordan Rose getCheckerManager().runCheckersForPostStmt(destNodes, DstPostCall, CE, *this); 60310e25031STed Kremenek } 60410e25031STed Kremenek 605b970505dSJordan Rose void ExprEngine::VisitCXXDestructor(QualType ObjectType, 60610e25031STed Kremenek const MemRegion *Dest, 60710e25031STed Kremenek const Stmt *S, 6082bc9674bSJordan Rose bool IsBaseDtor, 60910e25031STed Kremenek ExplodedNode *Pred, 610168e29f6SArtem Dergachev ExplodedNodeSet &Dst, 611168e29f6SArtem Dergachev const EvalCallOptions &CallOpts) { 612097a0497SArtem Dergachev assert(S && "A destructor without a trigger!"); 613fcd016e5SJordan Rose const LocationContext *LCtx = Pred->getLocationContext(); 614fcd016e5SJordan Rose ProgramStateRef State = Pred->getState(); 615fcd016e5SJordan Rose 616b970505dSJordan Rose const CXXRecordDecl *RecordDecl = ObjectType->getAsCXXRecordDecl(); 617b970505dSJordan Rose assert(RecordDecl && "Only CXXRecordDecls should have destructors"); 618b970505dSJordan Rose const CXXDestructorDecl *DtorDecl = RecordDecl->getDestructor(); 619b970505dSJordan Rose 620097a0497SArtem Dergachev // FIXME: There should always be a Decl, otherwise the destructor call 621097a0497SArtem Dergachev // shouldn't have been added to the CFG in the first place. 622097a0497SArtem Dergachev if (!DtorDecl) { 623097a0497SArtem Dergachev // Skip the invalid destructor. We cannot simply return because 624097a0497SArtem Dergachev // it would interrupt the analysis instead. 625097a0497SArtem Dergachev static SimpleProgramPointTag T("ExprEngine", "SkipInvalidDestructor"); 626097a0497SArtem Dergachev // FIXME: PostImplicitCall with a null decl may crash elsewhere anyway. 627097a0497SArtem Dergachev PostImplicitCall PP(/*Decl=*/nullptr, S->getEndLoc(), LCtx, &T); 628097a0497SArtem Dergachev NodeBuilder Bldr(Pred, Dst, *currBldrCtx); 629097a0497SArtem Dergachev Bldr.generateNode(PP, Pred->getState(), Pred); 630097a0497SArtem Dergachev return; 631097a0497SArtem Dergachev } 632097a0497SArtem Dergachev 633fcd016e5SJordan Rose CallEventManager &CEMgr = getStateManager().getCallEventManager(); 634fcd016e5SJordan Rose CallEventRef<CXXDestructorCall> Call = 6352bc9674bSJordan Rose CEMgr.getCXXDestructorCall(DtorDecl, S, Dest, IsBaseDtor, State, LCtx); 636142adc49STed Kremenek 63717a8757aSJordan Rose PrettyStackTraceLoc CrashInfo(getContext().getSourceManager(), 63817a8757aSJordan Rose Call->getSourceRange().getBegin(), 63917a8757aSJordan Rose "Error evaluating destructor"); 64017a8757aSJordan Rose 6414ee71b8aSJordan Rose ExplodedNodeSet DstPreCall; 6424ee71b8aSJordan Rose getCheckerManager().runCheckersForPreCall(DstPreCall, Pred, 643fcd016e5SJordan Rose *Call, *this); 64410e25031STed Kremenek 6454ee71b8aSJordan Rose ExplodedNodeSet DstInvalidated; 646d94854a4STed Kremenek StmtNodeBuilder Bldr(DstPreCall, DstInvalidated, *currBldrCtx); 6474ee71b8aSJordan Rose for (ExplodedNodeSet::iterator I = DstPreCall.begin(), E = DstPreCall.end(); 6484ee71b8aSJordan Rose I != E; ++I) 649690ab040SArtem Dergachev defaultEvalCall(Bldr, *I, *Call, CallOpts); 6504ee71b8aSJordan Rose 6514ee71b8aSJordan Rose getCheckerManager().runCheckersForPostCall(Dst, DstInvalidated, 652fcd016e5SJordan Rose *Call, *this); 65310e25031STed Kremenek } 65410e25031STed Kremenek 6558b808d64SJordan Rose void ExprEngine::VisitCXXNewAllocatorCall(const CXXNewExpr *CNE, 6568b808d64SJordan Rose ExplodedNode *Pred, 6578b808d64SJordan Rose ExplodedNodeSet &Dst) { 6588b808d64SJordan Rose ProgramStateRef State = Pred->getState(); 6598b808d64SJordan Rose const LocationContext *LCtx = Pred->getLocationContext(); 6608b808d64SJordan Rose PrettyStackTraceLoc CrashInfo(getContext().getSourceManager(), 661a6e4358fSStephen Kelly CNE->getBeginLoc(), 6628b808d64SJordan Rose "Error evaluating New Allocator Call"); 6638b808d64SJordan Rose CallEventManager &CEMgr = getStateManager().getCallEventManager(); 6648b808d64SJordan Rose CallEventRef<CXXAllocatorCall> Call = 6658b808d64SJordan Rose CEMgr.getCXXAllocatorCall(CNE, State, LCtx); 6668b808d64SJordan Rose 6678b808d64SJordan Rose ExplodedNodeSet DstPreCall; 6688b808d64SJordan Rose getCheckerManager().runCheckersForPreCall(DstPreCall, Pred, 6698b808d64SJordan Rose *Call, *this); 6708b808d64SJordan Rose 67155796302SArtem Dergachev ExplodedNodeSet DstPostCall; 67255796302SArtem Dergachev StmtNodeBuilder CallBldr(DstPreCall, DstPostCall, *currBldrCtx); 6731084de52SArtem Dergachev for (auto I : DstPreCall) { 6741084de52SArtem Dergachev // FIXME: Provide evalCall for checkers? 67555796302SArtem Dergachev defaultEvalCall(CallBldr, I, *Call); 6761084de52SArtem Dergachev } 677beba5307SArtem Dergachev // If the call is inlined, DstPostCall will be empty and we bail out now. 67855796302SArtem Dergachev 67955796302SArtem Dergachev // Store return value of operator new() for future use, until the actual 68055796302SArtem Dergachev // CXXNewExpr gets processed. 68155796302SArtem Dergachev ExplodedNodeSet DstPostValue; 68255796302SArtem Dergachev StmtNodeBuilder ValueBldr(DstPostCall, DstPostValue, *currBldrCtx); 68355796302SArtem Dergachev for (auto I : DstPostCall) { 684beba5307SArtem Dergachev // FIXME: Because CNE serves as the "call site" for the allocator (due to 685beba5307SArtem Dergachev // lack of a better expression in the AST), the conjured return value symbol 686beba5307SArtem Dergachev // is going to be of the same type (C++ object pointer type). Technically 687beba5307SArtem Dergachev // this is not correct because the operator new's prototype always says that 688beba5307SArtem Dergachev // it returns a 'void *'. So we should change the type of the symbol, 689beba5307SArtem Dergachev // and then evaluate the cast over the symbolic pointer from 'void *' to 690beba5307SArtem Dergachev // the object pointer type. But without changing the symbol's type it 691beba5307SArtem Dergachev // is breaking too much to evaluate the no-op symbolic cast over it, so we 692beba5307SArtem Dergachev // skip it for now. 69355796302SArtem Dergachev ProgramStateRef State = I->getState(); 69450e0372fSArtem Dergachev SVal RetVal = State->getSVal(CNE, LCtx); 69550e0372fSArtem Dergachev 69650e0372fSArtem Dergachev // If this allocation function is not declared as non-throwing, failures 69750e0372fSArtem Dergachev // /must/ be signalled by exceptions, and thus the return value will never 69850e0372fSArtem Dergachev // be NULL. -fno-exceptions does not influence this semantics. 69950e0372fSArtem Dergachev // FIXME: GCC has a -fcheck-new option, which forces it to consider the case 70050e0372fSArtem Dergachev // where new can return NULL. If we end up supporting that option, we can 70150e0372fSArtem Dergachev // consider adding a check for it here. 70250e0372fSArtem Dergachev // C++11 [basic.stc.dynamic.allocation]p3. 70350e0372fSArtem Dergachev if (const FunctionDecl *FD = CNE->getOperatorNew()) { 70450e0372fSArtem Dergachev QualType Ty = FD->getType(); 70550e0372fSArtem Dergachev if (const auto *ProtoType = Ty->getAs<FunctionProtoType>()) 706eaf11ad7SRichard Smith if (!ProtoType->isNothrow()) 70750e0372fSArtem Dergachev State = State->assume(RetVal.castAs<DefinedOrUnknownSVal>(), true); 70850e0372fSArtem Dergachev } 70950e0372fSArtem Dergachev 710f83d5479SArtem Dergachev ValueBldr.generateNode( 711f83d5479SArtem Dergachev CNE, I, addObjectUnderConstruction(State, CNE, LCtx, RetVal)); 71255796302SArtem Dergachev } 71355796302SArtem Dergachev 71413b2026bSArtem Dergachev ExplodedNodeSet DstPostPostCallCallback; 71513b2026bSArtem Dergachev getCheckerManager().runCheckersForPostCall(DstPostPostCallCallback, 71613b2026bSArtem Dergachev DstPostValue, *Call, *this); 71713b2026bSArtem Dergachev for (auto I : DstPostPostCallCallback) { 71813b2026bSArtem Dergachev getCheckerManager().runCheckersForNewAllocator( 719f83d5479SArtem Dergachev CNE, *getObjectUnderConstruction(I->getState(), CNE, LCtx), Dst, I, 720f83d5479SArtem Dergachev *this); 72113b2026bSArtem Dergachev } 7228b808d64SJordan Rose } 7238b808d64SJordan Rose 72410e25031STed Kremenek void ExprEngine::VisitCXXNewExpr(const CXXNewExpr *CNE, ExplodedNode *Pred, 72510e25031STed Kremenek ExplodedNodeSet &Dst) { 726a4ee064cSJordan Rose // FIXME: Much of this should eventually migrate to CXXAllocatorCall. 727a4ee064cSJordan Rose // Also, we need to decide how allocators actually work -- they're not 728a4ee064cSJordan Rose // really part of the CXXNewExpr because they happen BEFORE the 729a4ee064cSJordan Rose // CXXConstructExpr subexpression. See PR12014 for some discussion. 73010e25031STed Kremenek 731d94854a4STed Kremenek unsigned blockCount = currBldrCtx->blockCount(); 732d519cae8STed Kremenek const LocationContext *LCtx = Pred->getLocationContext(); 73355796302SArtem Dergachev SVal symVal = UnknownVal(); 7348b662704SAnton Yartsev FunctionDecl *FD = CNE->getOperatorNew(); 73510e25031STed Kremenek 7361084de52SArtem Dergachev bool IsStandardGlobalOpNewFunction = 7371084de52SArtem Dergachev FD->isReplaceableGlobalAllocationFunction(); 7388b662704SAnton Yartsev 73955796302SArtem Dergachev ProgramStateRef State = Pred->getState(); 74055796302SArtem Dergachev 74155796302SArtem Dergachev // Retrieve the stored operator new() return value. 742549f9cd4SKristof Umann if (AMgr.getAnalyzerOptions().MayInlineCXXAllocator) { 743f83d5479SArtem Dergachev symVal = *getObjectUnderConstruction(State, CNE, LCtx); 744f83d5479SArtem Dergachev State = finishObjectConstruction(State, CNE, LCtx); 74555796302SArtem Dergachev } 74655796302SArtem Dergachev 7478b662704SAnton Yartsev // We assume all standard global 'operator new' functions allocate memory in 7488b662704SAnton Yartsev // heap. We realize this is an approximation that might not correctly model 7498b662704SAnton Yartsev // a custom global allocator. 75055796302SArtem Dergachev if (symVal.isUnknown()) { 7518b662704SAnton Yartsev if (IsStandardGlobalOpNewFunction) 7528b662704SAnton Yartsev symVal = svalBuilder.getConjuredHeapSymbolVal(CNE, LCtx, blockCount); 7538b662704SAnton Yartsev else 7540dbb783cSCraig Topper symVal = svalBuilder.conjureSymbolVal(nullptr, CNE, LCtx, CNE->getType(), 7558b662704SAnton Yartsev blockCount); 75655796302SArtem Dergachev } 7578b662704SAnton Yartsev 758fcd016e5SJordan Rose CallEventManager &CEMgr = getStateManager().getCallEventManager(); 759fcd016e5SJordan Rose CallEventRef<CXXAllocatorCall> Call = 760fcd016e5SJordan Rose CEMgr.getCXXAllocatorCall(CNE, State, LCtx); 761fcd016e5SJordan Rose 762549f9cd4SKristof Umann if (!AMgr.getAnalyzerOptions().MayInlineCXXAllocator) { 763a4ee064cSJordan Rose // Invalidate placement args. 764fcd016e5SJordan Rose // FIXME: Once we figure out how we want allocators to work, 765fcd016e5SJordan Rose // we should be using the usual pre-/(default-)eval-/post-call checks here. 766fcd016e5SJordan Rose State = Call->invalidateRegions(blockCount); 767b8cb8359SJordan Rose if (!State) 768b8cb8359SJordan Rose return; 769a4ee064cSJordan Rose 770fce1b03eSPavel Labath // If this allocation function is not declared as non-throwing, failures 77150e0372fSArtem Dergachev // /must/ be signalled by exceptions, and thus the return value will never 77250e0372fSArtem Dergachev // be NULL. -fno-exceptions does not influence this semantics. 773fce1b03eSPavel Labath // FIXME: GCC has a -fcheck-new option, which forces it to consider the case 774fce1b03eSPavel Labath // where new can return NULL. If we end up supporting that option, we can 775fce1b03eSPavel Labath // consider adding a check for it here. 7763957fd58SJordan Rose // C++11 [basic.stc.dynamic.allocation]p3. 777fce1b03eSPavel Labath if (FD) { 7783957fd58SJordan Rose QualType Ty = FD->getType(); 77950e0372fSArtem Dergachev if (const auto *ProtoType = Ty->getAs<FunctionProtoType>()) 780eaf11ad7SRichard Smith if (!ProtoType->isNothrow()) 78155796302SArtem Dergachev if (auto dSymVal = symVal.getAs<DefinedOrUnknownSVal>()) 78255796302SArtem Dergachev State = State->assume(*dSymVal, true); 7833957fd58SJordan Rose } 78450e0372fSArtem Dergachev } 7853957fd58SJordan Rose 786b8cb8359SJordan Rose StmtNodeBuilder Bldr(Pred, Dst, *currBldrCtx); 787b8cb8359SJordan Rose 788beba5307SArtem Dergachev SVal Result = symVal; 789beba5307SArtem Dergachev 79010e25031STed Kremenek if (CNE->isArray()) { 79110e25031STed Kremenek // FIXME: allocating an array requires simulating the constructors. 79210e25031STed Kremenek // For now, just return a symbolicated region. 7931084de52SArtem Dergachev if (const SubRegion *NewReg = 7941084de52SArtem Dergachev dyn_cast_or_null<SubRegion>(symVal.getAsRegion())) { 795a4ee064cSJordan Rose QualType ObjTy = CNE->getType()->getAs<PointerType>()->getPointeeType(); 796a4ee064cSJordan Rose const ElementRegion *EleReg = 797a4ee064cSJordan Rose getStoreManager().GetElementZeroRegion(NewReg, ObjTy); 798beba5307SArtem Dergachev Result = loc::MemRegionVal(EleReg); 799beba5307SArtem Dergachev } 800beba5307SArtem Dergachev State = State->BindExpr(CNE, Pred->getLocationContext(), Result); 80144320367SJordan Rose Bldr.generateNode(CNE, Pred, State); 80210e25031STed Kremenek return; 80310e25031STed Kremenek } 80410e25031STed Kremenek 805a4ee064cSJordan Rose // FIXME: Once we have proper support for CXXConstructExprs inside 806a4ee064cSJordan Rose // CXXNewExpr, we need to make sure that the constructed object is not 807a4ee064cSJordan Rose // immediately invalidated here. (The placement call should happen before 808a4ee064cSJordan Rose // the constructor call anyway.) 80944320367SJordan Rose if (FD && FD->isReservedGlobalPlacementOperator()) { 81044320367SJordan Rose // Non-array placement new should always return the placement location. 81144320367SJordan Rose SVal PlacementLoc = State->getSVal(CNE->getPlacementArg(0), LCtx); 812b8cb8359SJordan Rose Result = svalBuilder.evalCast(PlacementLoc, CNE->getType(), 813aaf83184SJordan Rose CNE->getPlacementArg(0)->getType()); 81444320367SJordan Rose } 81544320367SJordan Rose 816b8cb8359SJordan Rose // Bind the address of the object, then check to see if we cached out. 817b8cb8359SJordan Rose State = State->BindExpr(CNE, LCtx, Result); 8188f6b4b04SJordan Rose ExplodedNode *NewN = Bldr.generateNode(CNE, Pred, State); 8198f6b4b04SJordan Rose if (!NewN) 8208f6b4b04SJordan Rose return; 8213503cb35SJordan Rose 8225089f3b3SJordan Rose // If the type is not a record, we won't have a CXXConstructExpr as an 8235089f3b3SJordan Rose // initializer. Copy the value over. 8245089f3b3SJordan Rose if (const Expr *Init = CNE->getInitializer()) { 8255089f3b3SJordan Rose if (!isa<CXXConstructExpr>(Init)) { 8263503cb35SJordan Rose assert(Bldr.getResults().size() == 1); 8278f6b4b04SJordan Rose Bldr.takeNodes(NewN); 828b8cb8359SJordan Rose evalBind(Dst, CNE, NewN, Result, State->getSVal(Init, LCtx), 829b8cb8359SJordan Rose /*FirstInit=*/IsStandardGlobalOpNewFunction); 8303503cb35SJordan Rose } 8313503cb35SJordan Rose } 83210e25031STed Kremenek } 83310e25031STed Kremenek 83410e25031STed Kremenek void ExprEngine::VisitCXXDeleteExpr(const CXXDeleteExpr *CDE, 83510e25031STed Kremenek ExplodedNode *Pred, ExplodedNodeSet &Dst) { 836d94854a4STed Kremenek StmtNodeBuilder Bldr(Pred, Dst, *currBldrCtx); 83764aea652SErik Verbruggen ProgramStateRef state = Pred->getState(); 83864aea652SErik Verbruggen Bldr.generateNode(CDE, Pred, state); 83910e25031STed Kremenek } 84010e25031STed Kremenek 8418fdb59f9STed Kremenek void ExprEngine::VisitCXXCatchStmt(const CXXCatchStmt *CS, 8428fdb59f9STed Kremenek ExplodedNode *Pred, 8438fdb59f9STed Kremenek ExplodedNodeSet &Dst) { 8448fdb59f9STed Kremenek const VarDecl *VD = CS->getExceptionDecl(); 845b1279b5cSTed Kremenek if (!VD) { 846b1279b5cSTed Kremenek Dst.Add(Pred); 847b1279b5cSTed Kremenek return; 848b1279b5cSTed Kremenek } 849b1279b5cSTed Kremenek 8508fdb59f9STed Kremenek const LocationContext *LCtx = Pred->getLocationContext(); 851d227833cSTed Kremenek SVal V = svalBuilder.conjureSymbolVal(CS, LCtx, VD->getType(), 852d94854a4STed Kremenek currBldrCtx->blockCount()); 8538fdb59f9STed Kremenek ProgramStateRef state = Pred->getState(); 854b570195cSAnna Zaks state = state->bindLoc(state->getLValue(VD, LCtx), V, LCtx); 8558fdb59f9STed Kremenek 856d94854a4STed Kremenek StmtNodeBuilder Bldr(Pred, Dst, *currBldrCtx); 8578fdb59f9STed Kremenek Bldr.generateNode(CS, Pred, state); 8588fdb59f9STed Kremenek } 8598fdb59f9STed Kremenek 86010e25031STed Kremenek void ExprEngine::VisitCXXThisExpr(const CXXThisExpr *TE, ExplodedNode *Pred, 86110e25031STed Kremenek ExplodedNodeSet &Dst) { 862d94854a4STed Kremenek StmtNodeBuilder Bldr(Pred, Dst, *currBldrCtx); 863e594034fSAnna Zaks 86410e25031STed Kremenek // Get the this object region from StoreManager. 865632e3b7eSTed Kremenek const LocationContext *LCtx = Pred->getLocationContext(); 86610e25031STed Kremenek const MemRegion *R = 86710e25031STed Kremenek svalBuilder.getRegionManager().getCXXThisRegion( 86810e25031STed Kremenek getContext().getCanonicalType(TE->getType()), 869632e3b7eSTed Kremenek LCtx); 87010e25031STed Kremenek 87149b1e38eSTed Kremenek ProgramStateRef state = Pred->getState(); 87210e25031STed Kremenek SVal V = state->getSVal(loc::MemRegionVal(R)); 873632e3b7eSTed Kremenek Bldr.generateNode(TE, Pred, state->BindExpr(TE, LCtx, V)); 87410e25031STed Kremenek } 87515843343SGabor Horvath 87615843343SGabor Horvath void ExprEngine::VisitLambdaExpr(const LambdaExpr *LE, ExplodedNode *Pred, 87715843343SGabor Horvath ExplodedNodeSet &Dst) { 87815843343SGabor Horvath const LocationContext *LocCtxt = Pred->getLocationContext(); 87915843343SGabor Horvath 88015843343SGabor Horvath // Get the region of the lambda itself. 88115843343SGabor Horvath const MemRegion *R = svalBuilder.getRegionManager().getCXXTempObjectRegion( 88215843343SGabor Horvath LE, LocCtxt); 88315843343SGabor Horvath SVal V = loc::MemRegionVal(R); 88415843343SGabor Horvath 88515843343SGabor Horvath ProgramStateRef State = Pred->getState(); 88615843343SGabor Horvath 88715843343SGabor Horvath // If we created a new MemRegion for the lambda, we should explicitly bind 88815843343SGabor Horvath // the captures. 88915843343SGabor Horvath CXXRecordDecl::field_iterator CurField = LE->getLambdaClass()->field_begin(); 89015843343SGabor Horvath for (LambdaExpr::const_capture_init_iterator i = LE->capture_init_begin(), 89115843343SGabor Horvath e = LE->capture_init_end(); 89215843343SGabor Horvath i != e; ++i, ++CurField) { 8937bdca8b2SDevin Coughlin FieldDecl *FieldForCapture = *CurField; 8947bdca8b2SDevin Coughlin SVal FieldLoc = State->getLValue(FieldForCapture, V); 8957bdca8b2SDevin Coughlin 8967bdca8b2SDevin Coughlin SVal InitVal; 8977bdca8b2SDevin Coughlin if (!FieldForCapture->hasCapturedVLAType()) { 8987bdca8b2SDevin Coughlin Expr *InitExpr = *i; 8997bdca8b2SDevin Coughlin assert(InitExpr && "Capture missing initialization expression"); 9007bdca8b2SDevin Coughlin InitVal = State->getSVal(InitExpr, LocCtxt); 9017bdca8b2SDevin Coughlin } else { 9027bdca8b2SDevin Coughlin // The field stores the length of a captured variable-length array. 9037bdca8b2SDevin Coughlin // These captures don't have initialization expressions; instead we 9047bdca8b2SDevin Coughlin // get the length from the VLAType size expression. 9057bdca8b2SDevin Coughlin Expr *SizeExpr = FieldForCapture->getCapturedVLAType()->getSizeExpr(); 9067bdca8b2SDevin Coughlin InitVal = State->getSVal(SizeExpr, LocCtxt); 9077bdca8b2SDevin Coughlin } 9087bdca8b2SDevin Coughlin 909b570195cSAnna Zaks State = State->bindLoc(FieldLoc, InitVal, LocCtxt); 91015843343SGabor Horvath } 91115843343SGabor Horvath 91215843343SGabor Horvath // Decay the Loc into an RValue, because there might be a 91315843343SGabor Horvath // MaterializeTemporaryExpr node above this one which expects the bound value 91415843343SGabor Horvath // to be an RValue. 91515843343SGabor Horvath SVal LambdaRVal = State->getSVal(R); 91615843343SGabor Horvath 91715843343SGabor Horvath ExplodedNodeSet Tmp; 91815843343SGabor Horvath StmtNodeBuilder Bldr(Pred, Tmp, *currBldrCtx); 91915843343SGabor Horvath // FIXME: is this the right program point kind? 92015843343SGabor Horvath Bldr.generateNode(LE, Pred, 92115843343SGabor Horvath State->BindExpr(LE, LocCtxt, LambdaRVal), 92215843343SGabor Horvath nullptr, ProgramPoint::PostLValueKind); 92315843343SGabor Horvath 92415843343SGabor Horvath // FIXME: Move all post/pre visits to ::Visit(). 92515843343SGabor Horvath getCheckerManager().runCheckersForPostStmt(Dst, Tmp, LE, *this); 92615843343SGabor Horvath } 927