1b000b770SStanislav Gatev //===-- UncheckedOptionalAccessModel.cpp ------------------------*- C++ -*-===// 2b000b770SStanislav Gatev // 3b000b770SStanislav Gatev // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. 4b000b770SStanislav Gatev // See https://llvm.org/LICENSE.txt for license information. 5b000b770SStanislav Gatev // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception 6b000b770SStanislav Gatev // 7b000b770SStanislav Gatev //===----------------------------------------------------------------------===// 8b000b770SStanislav Gatev // 9b000b770SStanislav Gatev // This file defines a dataflow analysis that detects unsafe uses of optional 10b000b770SStanislav Gatev // values. 11b000b770SStanislav Gatev // 12b000b770SStanislav Gatev //===----------------------------------------------------------------------===// 13b000b770SStanislav Gatev 14af98b0afSStanislav Gatev #include "clang/Analysis/FlowSensitive/Models/UncheckedOptionalAccessModel.h" 15af98b0afSStanislav Gatev #include "clang/AST/ASTContext.h" 167e63a0d4SYitzhak Mandelbaum #include "clang/AST/DeclCXX.h" 17af98b0afSStanislav Gatev #include "clang/AST/Expr.h" 18af98b0afSStanislav Gatev #include "clang/AST/ExprCXX.h" 19af98b0afSStanislav Gatev #include "clang/AST/Stmt.h" 20af98b0afSStanislav Gatev #include "clang/ASTMatchers/ASTMatchers.h" 2109b462efSYitzhak Mandelbaum #include "clang/ASTMatchers/ASTMatchersMacros.h" 227538b360SWei Yi Tee #include "clang/Analysis/CFG.h" 237538b360SWei Yi Tee #include "clang/Analysis/FlowSensitive/CFGMatchSwitch.h" 24af98b0afSStanislav Gatev #include "clang/Analysis/FlowSensitive/DataflowEnvironment.h" 25cf1f978dSSam Estep #include "clang/Analysis/FlowSensitive/NoopLattice.h" 260086a355SYitzhak Mandelbaum #include "clang/Analysis/FlowSensitive/StorageLocation.h" 27af98b0afSStanislav Gatev #include "clang/Analysis/FlowSensitive/Value.h" 2858fe7f96SSam Estep #include "clang/Basic/SourceLocation.h" 29af98b0afSStanislav Gatev #include "llvm/ADT/StringRef.h" 30af98b0afSStanislav Gatev #include "llvm/Support/Casting.h" 31d34fbf2dSYitzhak Mandelbaum #include "llvm/Support/ErrorHandling.h" 32af98b0afSStanislav Gatev #include <cassert> 339e0fc676SStanislav Gatev #include <memory> 34a1580d7bSKazu Hirata #include <optional> 359e0fc676SStanislav Gatev #include <utility> 3658fe7f96SSam Estep #include <vector> 37af98b0afSStanislav Gatev 38af98b0afSStanislav Gatev namespace clang { 39af98b0afSStanislav Gatev namespace dataflow { 4009b462efSYitzhak Mandelbaum 4109b462efSYitzhak Mandelbaum static bool isTopLevelNamespaceWithName(const NamespaceDecl &NS, 4209b462efSYitzhak Mandelbaum llvm::StringRef Name) { 4309b462efSYitzhak Mandelbaum return NS.getDeclName().isIdentifier() && NS.getName() == Name && 4409b462efSYitzhak Mandelbaum NS.getParent() != nullptr && NS.getParent()->isTranslationUnit(); 4509b462efSYitzhak Mandelbaum } 4609b462efSYitzhak Mandelbaum 4709b462efSYitzhak Mandelbaum static bool hasOptionalClassName(const CXXRecordDecl &RD) { 4809b462efSYitzhak Mandelbaum if (!RD.getDeclName().isIdentifier()) 4909b462efSYitzhak Mandelbaum return false; 5009b462efSYitzhak Mandelbaum 5109b462efSYitzhak Mandelbaum if (RD.getName() == "optional") { 5209b462efSYitzhak Mandelbaum if (const auto *N = dyn_cast_or_null<NamespaceDecl>(RD.getDeclContext())) 5309b462efSYitzhak Mandelbaum return N->isStdNamespace() || isTopLevelNamespaceWithName(*N, "absl"); 5409b462efSYitzhak Mandelbaum return false; 5509b462efSYitzhak Mandelbaum } 5609b462efSYitzhak Mandelbaum 5709b462efSYitzhak Mandelbaum if (RD.getName() == "Optional") { 5809b462efSYitzhak Mandelbaum // Check whether namespace is "::base". 5909b462efSYitzhak Mandelbaum const auto *N = dyn_cast_or_null<NamespaceDecl>(RD.getDeclContext()); 6009b462efSYitzhak Mandelbaum return N != nullptr && isTopLevelNamespaceWithName(*N, "base"); 6109b462efSYitzhak Mandelbaum } 6209b462efSYitzhak Mandelbaum 6309b462efSYitzhak Mandelbaum return false; 6409b462efSYitzhak Mandelbaum } 6509b462efSYitzhak Mandelbaum 66af98b0afSStanislav Gatev namespace { 67af98b0afSStanislav Gatev 68af98b0afSStanislav Gatev using namespace ::clang::ast_matchers; 69cf1f978dSSam Estep using LatticeTransferState = TransferState<NoopLattice>; 70af98b0afSStanislav Gatev 7109b462efSYitzhak Mandelbaum AST_MATCHER(CXXRecordDecl, hasOptionalClassNameMatcher) { 7209b462efSYitzhak Mandelbaum return hasOptionalClassName(Node); 7309b462efSYitzhak Mandelbaum } 7409b462efSYitzhak Mandelbaum 757e63a0d4SYitzhak Mandelbaum DeclarationMatcher optionalClass() { 76af98b0afSStanislav Gatev return classTemplateSpecializationDecl( 7709b462efSYitzhak Mandelbaum hasOptionalClassNameMatcher(), 78af98b0afSStanislav Gatev hasTemplateArgument(0, refersToType(type().bind("T")))); 79af98b0afSStanislav Gatev } 80af98b0afSStanislav Gatev 816adfc64eSYitzhak Mandelbaum auto optionalOrAliasType() { 8265e710c3SStanislav Gatev return hasUnqualifiedDesugaredType( 8365e710c3SStanislav Gatev recordType(hasDeclaration(optionalClass()))); 8465e710c3SStanislav Gatev } 8565e710c3SStanislav Gatev 866adfc64eSYitzhak Mandelbaum /// Matches any of the spellings of the optional types and sugar, aliases, etc. 876adfc64eSYitzhak Mandelbaum auto hasOptionalType() { return hasType(optionalOrAliasType()); } 886adfc64eSYitzhak Mandelbaum 89a184a0d8SYitzhak Mandelbaum auto isOptionalMemberCallWithName( 90a184a0d8SYitzhak Mandelbaum llvm::StringRef MemberName, 916ad0788cSKazu Hirata const std::optional<StatementMatcher> &Ignorable = std::nullopt) { 92a184a0d8SYitzhak Mandelbaum auto Exception = unless(Ignorable ? expr(anyOf(*Ignorable, cxxThisExpr())) 93a184a0d8SYitzhak Mandelbaum : cxxThisExpr()); 94af98b0afSStanislav Gatev return cxxMemberCallExpr( 9509b462efSYitzhak Mandelbaum on(expr(Exception, 9609b462efSYitzhak Mandelbaum anyOf(hasOptionalType(), 9709b462efSYitzhak Mandelbaum hasType(pointerType(pointee(optionalOrAliasType())))))), 9809b462efSYitzhak Mandelbaum callee(cxxMethodDecl(hasName(MemberName)))); 99af98b0afSStanislav Gatev } 100af98b0afSStanislav Gatev 101a184a0d8SYitzhak Mandelbaum auto isOptionalOperatorCallWithName( 102a184a0d8SYitzhak Mandelbaum llvm::StringRef operator_name, 1036ad0788cSKazu Hirata const std::optional<StatementMatcher> &Ignorable = std::nullopt) { 104a184a0d8SYitzhak Mandelbaum return cxxOperatorCallExpr( 105a184a0d8SYitzhak Mandelbaum hasOverloadedOperatorName(operator_name), 106a184a0d8SYitzhak Mandelbaum callee(cxxMethodDecl(ofClass(optionalClass()))), 107a184a0d8SYitzhak Mandelbaum Ignorable ? callExpr(unless(hasArgument(0, *Ignorable))) : callExpr()); 108af98b0afSStanislav Gatev } 109af98b0afSStanislav Gatev 110092a530cSStanislav Gatev auto isMakeOptionalCall() { 1119e0fc676SStanislav Gatev return callExpr( 1129e0fc676SStanislav Gatev callee(functionDecl(hasAnyName( 1139e0fc676SStanislav Gatev "std::make_optional", "base::make_optional", "absl::make_optional"))), 1149e0fc676SStanislav Gatev hasOptionalType()); 1159e0fc676SStanislav Gatev } 1169e0fc676SStanislav Gatev 117390029beSYitzhak Mandelbaum auto nulloptTypeDecl() { 118390029beSYitzhak Mandelbaum return namedDecl( 119390029beSYitzhak Mandelbaum hasAnyName("std::nullopt_t", "absl::nullopt_t", "base::nullopt_t")); 120092a530cSStanislav Gatev } 121092a530cSStanislav Gatev 122390029beSYitzhak Mandelbaum auto hasNulloptType() { return hasType(nulloptTypeDecl()); } 123390029beSYitzhak Mandelbaum 124390029beSYitzhak Mandelbaum // `optional` or `nullopt_t` 125390029beSYitzhak Mandelbaum auto hasAnyOptionalType() { 126390029beSYitzhak Mandelbaum return hasType(hasUnqualifiedDesugaredType( 127390029beSYitzhak Mandelbaum recordType(hasDeclaration(anyOf(nulloptTypeDecl(), optionalClass()))))); 128390029beSYitzhak Mandelbaum } 129390029beSYitzhak Mandelbaum 130092a530cSStanislav Gatev auto inPlaceClass() { 131092a530cSStanislav Gatev return recordDecl( 132092a530cSStanislav Gatev hasAnyName("std::in_place_t", "absl::in_place_t", "base::in_place_t")); 133092a530cSStanislav Gatev } 134092a530cSStanislav Gatev 135092a530cSStanislav Gatev auto isOptionalNulloptConstructor() { 1360086a355SYitzhak Mandelbaum return cxxConstructExpr( 1370086a355SYitzhak Mandelbaum hasOptionalType(), 1380086a355SYitzhak Mandelbaum hasDeclaration(cxxConstructorDecl(parameterCountIs(1), 1390086a355SYitzhak Mandelbaum hasParameter(0, hasNulloptType())))); 140092a530cSStanislav Gatev } 141092a530cSStanislav Gatev 142092a530cSStanislav Gatev auto isOptionalInPlaceConstructor() { 143092a530cSStanislav Gatev return cxxConstructExpr(hasOptionalType(), 144092a530cSStanislav Gatev hasArgument(0, hasType(inPlaceClass()))); 145092a530cSStanislav Gatev } 146092a530cSStanislav Gatev 147092a530cSStanislav Gatev auto isOptionalValueOrConversionConstructor() { 148092a530cSStanislav Gatev return cxxConstructExpr( 149092a530cSStanislav Gatev hasOptionalType(), 150092a530cSStanislav Gatev unless(hasDeclaration( 151092a530cSStanislav Gatev cxxConstructorDecl(anyOf(isCopyConstructor(), isMoveConstructor())))), 152092a530cSStanislav Gatev argumentCountIs(1), hasArgument(0, unless(hasNulloptType()))); 153092a530cSStanislav Gatev } 154092a530cSStanislav Gatev 155b000b770SStanislav Gatev auto isOptionalValueOrConversionAssignment() { 156b000b770SStanislav Gatev return cxxOperatorCallExpr( 157b000b770SStanislav Gatev hasOverloadedOperatorName("="), 158b000b770SStanislav Gatev callee(cxxMethodDecl(ofClass(optionalClass()))), 159b000b770SStanislav Gatev unless(hasDeclaration(cxxMethodDecl( 160b000b770SStanislav Gatev anyOf(isCopyAssignmentOperator(), isMoveAssignmentOperator())))), 161b000b770SStanislav Gatev argumentCountIs(2), hasArgument(1, unless(hasNulloptType()))); 162b000b770SStanislav Gatev } 163b000b770SStanislav Gatev 164390029beSYitzhak Mandelbaum auto isNulloptConstructor() { 165390029beSYitzhak Mandelbaum return cxxConstructExpr(hasNulloptType(), argumentCountIs(1), 166390029beSYitzhak Mandelbaum hasArgument(0, hasNulloptType())); 167390029beSYitzhak Mandelbaum } 168390029beSYitzhak Mandelbaum 169b000b770SStanislav Gatev auto isOptionalNulloptAssignment() { 170b000b770SStanislav Gatev return cxxOperatorCallExpr(hasOverloadedOperatorName("="), 171b000b770SStanislav Gatev callee(cxxMethodDecl(ofClass(optionalClass()))), 172b000b770SStanislav Gatev argumentCountIs(2), 173b000b770SStanislav Gatev hasArgument(1, hasNulloptType())); 174b000b770SStanislav Gatev } 175b000b770SStanislav Gatev 1762ddd57aeSStanislav Gatev auto isStdSwapCall() { 1772ddd57aeSStanislav Gatev return callExpr(callee(functionDecl(hasName("std::swap"))), 1782ddd57aeSStanislav Gatev argumentCountIs(2), hasArgument(0, hasOptionalType()), 1792ddd57aeSStanislav Gatev hasArgument(1, hasOptionalType())); 1802ddd57aeSStanislav Gatev } 1812ddd57aeSStanislav Gatev 18225956d55SAMS21 auto isStdForwardCall() { 18325956d55SAMS21 return callExpr(callee(functionDecl(hasName("std::forward"))), 18425956d55SAMS21 argumentCountIs(1), hasArgument(0, hasOptionalType())); 18525956d55SAMS21 } 18625956d55SAMS21 1877f076004SYitzhak Mandelbaum constexpr llvm::StringLiteral ValueOrCallID = "ValueOrCall"; 1887f076004SYitzhak Mandelbaum 1897f076004SYitzhak Mandelbaum auto isValueOrStringEmptyCall() { 1907f076004SYitzhak Mandelbaum // `opt.value_or("").empty()` 1917f076004SYitzhak Mandelbaum return cxxMemberCallExpr( 1927f076004SYitzhak Mandelbaum callee(cxxMethodDecl(hasName("empty"))), 1937f076004SYitzhak Mandelbaum onImplicitObjectArgument(ignoringImplicit( 1947f076004SYitzhak Mandelbaum cxxMemberCallExpr(on(expr(unless(cxxThisExpr()))), 1957f076004SYitzhak Mandelbaum callee(cxxMethodDecl(hasName("value_or"), 1967f076004SYitzhak Mandelbaum ofClass(optionalClass()))), 1977f076004SYitzhak Mandelbaum hasArgument(0, stringLiteral(hasSize(0)))) 1987f076004SYitzhak Mandelbaum .bind(ValueOrCallID)))); 1997f076004SYitzhak Mandelbaum } 2007f076004SYitzhak Mandelbaum 2017f076004SYitzhak Mandelbaum auto isValueOrNotEqX() { 2027f076004SYitzhak Mandelbaum auto ComparesToSame = [](ast_matchers::internal::Matcher<Stmt> Arg) { 2037f076004SYitzhak Mandelbaum return hasOperands( 2047f076004SYitzhak Mandelbaum ignoringImplicit( 2057f076004SYitzhak Mandelbaum cxxMemberCallExpr(on(expr(unless(cxxThisExpr()))), 2067f076004SYitzhak Mandelbaum callee(cxxMethodDecl(hasName("value_or"), 2077f076004SYitzhak Mandelbaum ofClass(optionalClass()))), 2087f076004SYitzhak Mandelbaum hasArgument(0, Arg)) 2097f076004SYitzhak Mandelbaum .bind(ValueOrCallID)), 2107f076004SYitzhak Mandelbaum ignoringImplicit(Arg)); 2117f076004SYitzhak Mandelbaum }; 2127f076004SYitzhak Mandelbaum 2137f076004SYitzhak Mandelbaum // `opt.value_or(X) != X`, for X is `nullptr`, `""`, or `0`. Ideally, we'd 2147f076004SYitzhak Mandelbaum // support this pattern for any expression, but the AST does not have a 2157f076004SYitzhak Mandelbaum // generic expression comparison facility, so we specialize to common cases 2167f076004SYitzhak Mandelbaum // seen in practice. FIXME: define a matcher that compares values across 2177f076004SYitzhak Mandelbaum // nodes, which would let us generalize this to any `X`. 2187f076004SYitzhak Mandelbaum return binaryOperation(hasOperatorName("!="), 2197f076004SYitzhak Mandelbaum anyOf(ComparesToSame(cxxNullPtrLiteralExpr()), 2207f076004SYitzhak Mandelbaum ComparesToSame(stringLiteral(hasSize(0))), 2217f076004SYitzhak Mandelbaum ComparesToSame(integerLiteral(equals(0))))); 2227f076004SYitzhak Mandelbaum } 2237f076004SYitzhak Mandelbaum 22465e710c3SStanislav Gatev auto isCallReturningOptional() { 225cd0d5261SSam Estep return callExpr(hasType(qualType(anyOf( 226cd0d5261SSam Estep optionalOrAliasType(), referenceType(pointee(optionalOrAliasType())))))); 22765e710c3SStanislav Gatev } 22865e710c3SStanislav Gatev 229390029beSYitzhak Mandelbaum template <typename L, typename R> 230390029beSYitzhak Mandelbaum auto isComparisonOperatorCall(L lhs_arg_matcher, R rhs_arg_matcher) { 231390029beSYitzhak Mandelbaum return cxxOperatorCallExpr( 232390029beSYitzhak Mandelbaum anyOf(hasOverloadedOperatorName("=="), hasOverloadedOperatorName("!=")), 233390029beSYitzhak Mandelbaum argumentCountIs(2), hasArgument(0, lhs_arg_matcher), 234390029beSYitzhak Mandelbaum hasArgument(1, rhs_arg_matcher)); 235390029beSYitzhak Mandelbaum } 236390029beSYitzhak Mandelbaum 237*ffb4f4dbSDmitri Gribenko /// Ensures that `Expr` is mapped to a `BoolValue` and returns it. 238390029beSYitzhak Mandelbaum BoolValue &forceBoolValue(Environment &Env, const Expr &Expr) { 239390029beSYitzhak Mandelbaum auto *Value = cast_or_null<BoolValue>(Env.getValue(Expr, SkipPast::None)); 240390029beSYitzhak Mandelbaum if (Value != nullptr) 241390029beSYitzhak Mandelbaum return *Value; 242390029beSYitzhak Mandelbaum 243390029beSYitzhak Mandelbaum auto &Loc = Env.createStorageLocation(Expr); 244390029beSYitzhak Mandelbaum Value = &Env.makeAtomicBoolValue(); 245390029beSYitzhak Mandelbaum Env.setValue(Loc, *Value); 246390029beSYitzhak Mandelbaum Env.setStorageLocation(Expr, Loc); 247390029beSYitzhak Mandelbaum return *Value; 248390029beSYitzhak Mandelbaum } 249390029beSYitzhak Mandelbaum 2508fcdd625SStanislav Gatev /// Sets `HasValueVal` as the symbolic value that represents the "has_value" 2518fcdd625SStanislav Gatev /// property of the optional value `OptionalVal`. 2528fcdd625SStanislav Gatev void setHasValue(Value &OptionalVal, BoolValue &HasValueVal) { 2538fcdd625SStanislav Gatev OptionalVal.setProperty("has_value", HasValueVal); 2548fcdd625SStanislav Gatev } 2558fcdd625SStanislav Gatev 2569e0fc676SStanislav Gatev /// Creates a symbolic value for an `optional` value using `HasValueVal` as the 2579e0fc676SStanislav Gatev /// symbolic value of its "has_value" property. 2589e0fc676SStanislav Gatev StructValue &createOptionalValue(Environment &Env, BoolValue &HasValueVal) { 259745a957fSMartin Braenne auto &OptionalVal = Env.create<StructValue>(); 260745a957fSMartin Braenne setHasValue(OptionalVal, HasValueVal); 261745a957fSMartin Braenne return OptionalVal; 2629e0fc676SStanislav Gatev } 2639e0fc676SStanislav Gatev 264af98b0afSStanislav Gatev /// Returns the symbolic value that represents the "has_value" property of the 26549ed5bf5SWei Yi Tee /// optional value `OptionalVal`. Returns null if `OptionalVal` is null. 266dd38caf3SYitzhak Mandelbaum BoolValue *getHasValue(Environment &Env, Value *OptionalVal) { 267dd38caf3SYitzhak Mandelbaum if (OptionalVal != nullptr) { 268dd38caf3SYitzhak Mandelbaum auto *HasValueVal = 269dd38caf3SYitzhak Mandelbaum cast_or_null<BoolValue>(OptionalVal->getProperty("has_value")); 270dd38caf3SYitzhak Mandelbaum if (HasValueVal == nullptr) { 271dd38caf3SYitzhak Mandelbaum HasValueVal = &Env.makeAtomicBoolValue(); 272dd38caf3SYitzhak Mandelbaum OptionalVal->setProperty("has_value", *HasValueVal); 273dd38caf3SYitzhak Mandelbaum } 274dd38caf3SYitzhak Mandelbaum return HasValueVal; 275af98b0afSStanislav Gatev } 276af98b0afSStanislav Gatev return nullptr; 277af98b0afSStanislav Gatev } 278af98b0afSStanislav Gatev 279092a530cSStanislav Gatev /// Returns true if and only if `Type` is an optional type. 280c0725865SYitzhak Mandelbaum bool isOptionalType(QualType Type) { 281092a530cSStanislav Gatev if (!Type->isRecordType()) 282092a530cSStanislav Gatev return false; 283cd22e0dcSYitzhak Mandelbaum const CXXRecordDecl *D = Type->getAsCXXRecordDecl(); 28409b462efSYitzhak Mandelbaum return D != nullptr && hasOptionalClassName(*D); 285092a530cSStanislav Gatev } 286092a530cSStanislav Gatev 287092a530cSStanislav Gatev /// Returns the number of optional wrappers in `Type`. 288092a530cSStanislav Gatev /// 289092a530cSStanislav Gatev /// For example, if `Type` is `optional<optional<int>>`, the result of this 290092a530cSStanislav Gatev /// function will be 2. 291092a530cSStanislav Gatev int countOptionalWrappers(const ASTContext &ASTCtx, QualType Type) { 292c0725865SYitzhak Mandelbaum if (!isOptionalType(Type)) 293092a530cSStanislav Gatev return 0; 294092a530cSStanislav Gatev return 1 + countOptionalWrappers( 295092a530cSStanislav Gatev ASTCtx, 296092a530cSStanislav Gatev cast<ClassTemplateSpecializationDecl>(Type->getAsRecordDecl()) 297092a530cSStanislav Gatev ->getTemplateArgs() 298092a530cSStanislav Gatev .get(0) 299092a530cSStanislav Gatev .getAsType() 300092a530cSStanislav Gatev .getDesugaredType(ASTCtx)); 301092a530cSStanislav Gatev } 302092a530cSStanislav Gatev 303dd38caf3SYitzhak Mandelbaum /// Tries to initialize the `optional`'s value (that is, contents), and return 304dd38caf3SYitzhak Mandelbaum /// its location. Returns nullptr if the value can't be represented. 305dd38caf3SYitzhak Mandelbaum StorageLocation *maybeInitializeOptionalValueMember(QualType Q, 306dd38caf3SYitzhak Mandelbaum Value &OptionalVal, 307dd38caf3SYitzhak Mandelbaum Environment &Env) { 308dd38caf3SYitzhak Mandelbaum // The "value" property represents a synthetic field. As such, it needs 309dd38caf3SYitzhak Mandelbaum // `StorageLocation`, like normal fields (and other variables). So, we model 310dd38caf3SYitzhak Mandelbaum // it with a `ReferenceValue`, since that includes a storage location. Once 311dd38caf3SYitzhak Mandelbaum // the property is set, it will be shared by all environments that access the 312dd38caf3SYitzhak Mandelbaum // `Value` representing the optional (here, `OptionalVal`). 313dd38caf3SYitzhak Mandelbaum if (auto *ValueProp = OptionalVal.getProperty("value")) { 314dd38caf3SYitzhak Mandelbaum auto *ValueRef = clang::cast<ReferenceValue>(ValueProp); 31597d69cdaSWei Yi Tee auto &ValueLoc = ValueRef->getReferentLoc(); 316dd38caf3SYitzhak Mandelbaum if (Env.getValue(ValueLoc) == nullptr) { 317dd38caf3SYitzhak Mandelbaum // The property was previously set, but the value has been lost. This can 318dd38caf3SYitzhak Mandelbaum // happen, for example, because of an environment merge (where the two 319dd38caf3SYitzhak Mandelbaum // environments mapped the property to different values, which resulted in 320dd38caf3SYitzhak Mandelbaum // them both being discarded), or when two blocks in the CFG, with neither 321dd38caf3SYitzhak Mandelbaum // a dominator of the other, visit the same optional value, or even when a 322dd38caf3SYitzhak Mandelbaum // block is revisited during testing to collect per-statement state. 323dd38caf3SYitzhak Mandelbaum // FIXME: This situation means that the optional contents are not shared 324dd38caf3SYitzhak Mandelbaum // between branches and the like. Practically, this lack of sharing 325dd38caf3SYitzhak Mandelbaum // reduces the precision of the model when the contents are relevant to 326dd38caf3SYitzhak Mandelbaum // the check, like another optional or a boolean that influences control 327dd38caf3SYitzhak Mandelbaum // flow. 328dd38caf3SYitzhak Mandelbaum auto *ValueVal = Env.createValue(ValueLoc.getType()); 329dd38caf3SYitzhak Mandelbaum if (ValueVal == nullptr) 330dd38caf3SYitzhak Mandelbaum return nullptr; 331dd38caf3SYitzhak Mandelbaum Env.setValue(ValueLoc, *ValueVal); 332dd38caf3SYitzhak Mandelbaum } 333dd38caf3SYitzhak Mandelbaum return &ValueLoc; 334dd38caf3SYitzhak Mandelbaum } 335dd38caf3SYitzhak Mandelbaum 336c849843cSMartin Braenne auto Ty = Q.getNonReferenceType(); 337dd38caf3SYitzhak Mandelbaum auto *ValueVal = Env.createValue(Ty); 338dd38caf3SYitzhak Mandelbaum if (ValueVal == nullptr) 339dd38caf3SYitzhak Mandelbaum return nullptr; 340dd38caf3SYitzhak Mandelbaum auto &ValueLoc = Env.createStorageLocation(Ty); 341dd38caf3SYitzhak Mandelbaum Env.setValue(ValueLoc, *ValueVal); 342745a957fSMartin Braenne auto &ValueRef = Env.create<ReferenceValue>(ValueLoc); 343745a957fSMartin Braenne OptionalVal.setProperty("value", ValueRef); 344dd38caf3SYitzhak Mandelbaum return &ValueLoc; 345dd38caf3SYitzhak Mandelbaum } 346dd38caf3SYitzhak Mandelbaum 347092a530cSStanislav Gatev void initializeOptionalReference(const Expr *OptionalExpr, 348092a530cSStanislav Gatev const MatchFinder::MatchResult &, 349af98b0afSStanislav Gatev LatticeTransferState &State) { 35049ed5bf5SWei Yi Tee if (auto *OptionalVal = 35149ed5bf5SWei Yi Tee State.Env.getValue(*OptionalExpr, SkipPast::Reference)) { 352af98b0afSStanislav Gatev if (OptionalVal->getProperty("has_value") == nullptr) { 3538fcdd625SStanislav Gatev setHasValue(*OptionalVal, State.Env.makeAtomicBoolValue()); 354af98b0afSStanislav Gatev } 355af98b0afSStanislav Gatev } 356af98b0afSStanislav Gatev } 357af98b0afSStanislav Gatev 3588fcdd625SStanislav Gatev /// Returns true if and only if `OptionalVal` is initialized and known to be 359*ffb4f4dbSDmitri Gribenko /// empty in `Env`. 3608fcdd625SStanislav Gatev bool isEmptyOptional(const Value &OptionalVal, const Environment &Env) { 3618fcdd625SStanislav Gatev auto *HasValueVal = 3628fcdd625SStanislav Gatev cast_or_null<BoolValue>(OptionalVal.getProperty("has_value")); 3638fcdd625SStanislav Gatev return HasValueVal != nullptr && 3648fcdd625SStanislav Gatev Env.flowConditionImplies(Env.makeNot(*HasValueVal)); 3658fcdd625SStanislav Gatev } 3668fcdd625SStanislav Gatev 3678fcdd625SStanislav Gatev /// Returns true if and only if `OptionalVal` is initialized and known to be 368*ffb4f4dbSDmitri Gribenko /// non-empty in `Env`. 3698fcdd625SStanislav Gatev bool isNonEmptyOptional(const Value &OptionalVal, const Environment &Env) { 3708fcdd625SStanislav Gatev auto *HasValueVal = 3718fcdd625SStanislav Gatev cast_or_null<BoolValue>(OptionalVal.getProperty("has_value")); 3728fcdd625SStanislav Gatev return HasValueVal != nullptr && Env.flowConditionImplies(*HasValueVal); 3738fcdd625SStanislav Gatev } 3748fcdd625SStanislav Gatev 37548bc7150SMartin Braenne StorageLocation *maybeSkipPointer(StorageLocation *Loc, 37648bc7150SMartin Braenne const Environment &Env) { 37748bc7150SMartin Braenne if (Loc == nullptr) 37848bc7150SMartin Braenne return nullptr; 37948bc7150SMartin Braenne if (auto *Val = dyn_cast_or_null<PointerValue>(Env.getValue(*Loc))) 38048bc7150SMartin Braenne return &Val->getPointeeLoc(); 38148bc7150SMartin Braenne return Loc; 38248bc7150SMartin Braenne } 38348bc7150SMartin Braenne 38448bc7150SMartin Braenne Value *getValueBehindPossiblePointer(const Expr &E, const Environment &Env) { 38548bc7150SMartin Braenne Value *Val = Env.getValue(E, SkipPast::Reference); 38648bc7150SMartin Braenne if (auto *PointerVal = dyn_cast_or_null<PointerValue>(Val)) 38748bc7150SMartin Braenne return Env.getValue(PointerVal->getPointeeLoc()); 38848bc7150SMartin Braenne return Val; 38948bc7150SMartin Braenne } 39048bc7150SMartin Braenne 391092a530cSStanislav Gatev void transferUnwrapCall(const Expr *UnwrapExpr, const Expr *ObjectExpr, 392af98b0afSStanislav Gatev LatticeTransferState &State) { 39349ed5bf5SWei Yi Tee if (auto *OptionalVal = 39448bc7150SMartin Braenne getValueBehindPossiblePointer(*ObjectExpr, State.Env)) { 395dd38caf3SYitzhak Mandelbaum if (State.Env.getStorageLocation(*UnwrapExpr, SkipPast::None) == nullptr) 396dd38caf3SYitzhak Mandelbaum if (auto *Loc = maybeInitializeOptionalValueMember( 397dd38caf3SYitzhak Mandelbaum UnwrapExpr->getType(), *OptionalVal, State.Env)) 398dd38caf3SYitzhak Mandelbaum State.Env.setStorageLocation(*UnwrapExpr, *Loc); 399af98b0afSStanislav Gatev } 400dd38caf3SYitzhak Mandelbaum } 401af98b0afSStanislav Gatev 402092a530cSStanislav Gatev void transferMakeOptionalCall(const CallExpr *E, 403092a530cSStanislav Gatev const MatchFinder::MatchResult &, 404092a530cSStanislav Gatev LatticeTransferState &State) { 4059e0fc676SStanislav Gatev auto &Loc = State.Env.createStorageLocation(*E); 4069e0fc676SStanislav Gatev State.Env.setStorageLocation(*E, Loc); 4079e0fc676SStanislav Gatev State.Env.setValue( 4089e0fc676SStanislav Gatev Loc, createOptionalValue(State.Env, State.Env.getBoolLiteralValue(true))); 4099e0fc676SStanislav Gatev } 4109e0fc676SStanislav Gatev 411092a530cSStanislav Gatev void transferOptionalHasValueCall(const CXXMemberCallExpr *CallExpr, 412092a530cSStanislav Gatev const MatchFinder::MatchResult &, 413af98b0afSStanislav Gatev LatticeTransferState &State) { 414dd38caf3SYitzhak Mandelbaum if (auto *HasValueVal = getHasValue( 41548bc7150SMartin Braenne State.Env, getValueBehindPossiblePointer( 41648bc7150SMartin Braenne *CallExpr->getImplicitObjectArgument(), State.Env))) { 417af98b0afSStanislav Gatev auto &CallExprLoc = State.Env.createStorageLocation(*CallExpr); 418af98b0afSStanislav Gatev State.Env.setValue(CallExprLoc, *HasValueVal); 419af98b0afSStanislav Gatev State.Env.setStorageLocation(*CallExpr, CallExprLoc); 420af98b0afSStanislav Gatev } 421af98b0afSStanislav Gatev } 422af98b0afSStanislav Gatev 4237f076004SYitzhak Mandelbaum /// `ModelPred` builds a logical formula relating the predicate in 4247f076004SYitzhak Mandelbaum /// `ValueOrPredExpr` to the optional's `has_value` property. 4257f076004SYitzhak Mandelbaum void transferValueOrImpl(const clang::Expr *ValueOrPredExpr, 4267f076004SYitzhak Mandelbaum const MatchFinder::MatchResult &Result, 4277f076004SYitzhak Mandelbaum LatticeTransferState &State, 4287f076004SYitzhak Mandelbaum BoolValue &(*ModelPred)(Environment &Env, 4297f076004SYitzhak Mandelbaum BoolValue &ExprVal, 4307f076004SYitzhak Mandelbaum BoolValue &HasValueVal)) { 4317f076004SYitzhak Mandelbaum auto &Env = State.Env; 4327f076004SYitzhak Mandelbaum 4337f076004SYitzhak Mandelbaum const auto *ObjectArgumentExpr = 4347f076004SYitzhak Mandelbaum Result.Nodes.getNodeAs<clang::CXXMemberCallExpr>(ValueOrCallID) 4357f076004SYitzhak Mandelbaum ->getImplicitObjectArgument(); 4367f076004SYitzhak Mandelbaum 437dd38caf3SYitzhak Mandelbaum auto *HasValueVal = getHasValue( 43848bc7150SMartin Braenne State.Env, getValueBehindPossiblePointer(*ObjectArgumentExpr, State.Env)); 439dd38caf3SYitzhak Mandelbaum if (HasValueVal == nullptr) 4407f076004SYitzhak Mandelbaum return; 4417f076004SYitzhak Mandelbaum 442390029beSYitzhak Mandelbaum Env.addToFlowCondition( 443390029beSYitzhak Mandelbaum ModelPred(Env, forceBoolValue(Env, *ValueOrPredExpr), *HasValueVal)); 4447f076004SYitzhak Mandelbaum } 4457f076004SYitzhak Mandelbaum 4467f076004SYitzhak Mandelbaum void transferValueOrStringEmptyCall(const clang::Expr *ComparisonExpr, 4477f076004SYitzhak Mandelbaum const MatchFinder::MatchResult &Result, 4487f076004SYitzhak Mandelbaum LatticeTransferState &State) { 4497f076004SYitzhak Mandelbaum return transferValueOrImpl(ComparisonExpr, Result, State, 4507f076004SYitzhak Mandelbaum [](Environment &Env, BoolValue &ExprVal, 4517f076004SYitzhak Mandelbaum BoolValue &HasValueVal) -> BoolValue & { 4527f076004SYitzhak Mandelbaum // If the result is *not* empty, then we know the 4537f076004SYitzhak Mandelbaum // optional must have been holding a value. If 4547f076004SYitzhak Mandelbaum // `ExprVal` is true, though, we don't learn 4557f076004SYitzhak Mandelbaum // anything definite about `has_value`, so we 4567f076004SYitzhak Mandelbaum // don't add any corresponding implications to 4577f076004SYitzhak Mandelbaum // the flow condition. 4587f076004SYitzhak Mandelbaum return Env.makeImplication(Env.makeNot(ExprVal), 4597f076004SYitzhak Mandelbaum HasValueVal); 4607f076004SYitzhak Mandelbaum }); 4617f076004SYitzhak Mandelbaum } 4627f076004SYitzhak Mandelbaum 4637f076004SYitzhak Mandelbaum void transferValueOrNotEqX(const Expr *ComparisonExpr, 4647f076004SYitzhak Mandelbaum const MatchFinder::MatchResult &Result, 4657f076004SYitzhak Mandelbaum LatticeTransferState &State) { 4667f076004SYitzhak Mandelbaum transferValueOrImpl(ComparisonExpr, Result, State, 4677f076004SYitzhak Mandelbaum [](Environment &Env, BoolValue &ExprVal, 4687f076004SYitzhak Mandelbaum BoolValue &HasValueVal) -> BoolValue & { 4697f076004SYitzhak Mandelbaum // We know that if `(opt.value_or(X) != X)` then 4707f076004SYitzhak Mandelbaum // `opt.hasValue()`, even without knowing further 4717f076004SYitzhak Mandelbaum // details about the contents of `opt`. 4727f076004SYitzhak Mandelbaum return Env.makeImplication(ExprVal, HasValueVal); 4737f076004SYitzhak Mandelbaum }); 4747f076004SYitzhak Mandelbaum } 4757f076004SYitzhak Mandelbaum 47665e710c3SStanislav Gatev void transferCallReturningOptional(const CallExpr *E, 47765e710c3SStanislav Gatev const MatchFinder::MatchResult &Result, 47865e710c3SStanislav Gatev LatticeTransferState &State) { 47965e710c3SStanislav Gatev if (State.Env.getStorageLocation(*E, SkipPast::None) != nullptr) 48065e710c3SStanislav Gatev return; 48165e710c3SStanislav Gatev 48265e710c3SStanislav Gatev auto &Loc = State.Env.createStorageLocation(*E); 48365e710c3SStanislav Gatev State.Env.setStorageLocation(*E, Loc); 48465e710c3SStanislav Gatev State.Env.setValue( 48565e710c3SStanislav Gatev Loc, createOptionalValue(State.Env, State.Env.makeAtomicBoolValue())); 48665e710c3SStanislav Gatev } 48765e710c3SStanislav Gatev 4880e8d4a6dSYitzhak Mandelbaum void assignOptionalValue(const Expr &E, Environment &Env, 489092a530cSStanislav Gatev BoolValue &HasValueVal) { 49048bc7150SMartin Braenne if (auto *OptionalLoc = maybeSkipPointer( 49148bc7150SMartin Braenne Env.getStorageLocation(E, SkipPast::Reference), Env)) { 4920e8d4a6dSYitzhak Mandelbaum Env.setValue(*OptionalLoc, createOptionalValue(Env, HasValueVal)); 4939e0fc676SStanislav Gatev } 4949e0fc676SStanislav Gatev } 4959e0fc676SStanislav Gatev 496b000b770SStanislav Gatev /// Returns a symbolic value for the "has_value" property of an `optional<T>` 497b000b770SStanislav Gatev /// value that is constructed/assigned from a value of type `U` or `optional<U>` 498b000b770SStanislav Gatev /// where `T` is constructible from `U`. 499390029beSYitzhak Mandelbaum BoolValue &valueOrConversionHasValue(const FunctionDecl &F, const Expr &E, 500b000b770SStanislav Gatev const MatchFinder::MatchResult &MatchRes, 501b000b770SStanislav Gatev LatticeTransferState &State) { 5020086a355SYitzhak Mandelbaum assert(F.getTemplateSpecializationArgs() != nullptr); 503b000b770SStanislav Gatev assert(F.getTemplateSpecializationArgs()->size() > 0); 504b000b770SStanislav Gatev 505c849843cSMartin Braenne const int TemplateParamOptionalWrappersCount = 506c849843cSMartin Braenne countOptionalWrappers(*MatchRes.Context, F.getTemplateSpecializationArgs() 507c849843cSMartin Braenne ->get(0) 508c849843cSMartin Braenne .getAsType() 509c849843cSMartin Braenne .getNonReferenceType()); 510c849843cSMartin Braenne const int ArgTypeOptionalWrappersCount = countOptionalWrappers( 511c849843cSMartin Braenne *MatchRes.Context, E.getType().getNonReferenceType()); 512b000b770SStanislav Gatev 513b000b770SStanislav Gatev // Check if this is a constructor/assignment call for `optional<T>` with 514b000b770SStanislav Gatev // argument of type `U` such that `T` is constructible from `U`. 515b000b770SStanislav Gatev if (TemplateParamOptionalWrappersCount == ArgTypeOptionalWrappersCount) 516b000b770SStanislav Gatev return State.Env.getBoolLiteralValue(true); 517b000b770SStanislav Gatev 518b000b770SStanislav Gatev // This is a constructor/assignment call for `optional<T>` with argument of 519b000b770SStanislav Gatev // type `optional<U>` such that `T` is constructible from `U`. 520dd38caf3SYitzhak Mandelbaum if (auto *HasValueVal = 521dd38caf3SYitzhak Mandelbaum getHasValue(State.Env, State.Env.getValue(E, SkipPast::Reference))) 522dd38caf3SYitzhak Mandelbaum return *HasValueVal; 523b000b770SStanislav Gatev return State.Env.makeAtomicBoolValue(); 524b000b770SStanislav Gatev } 525b000b770SStanislav Gatev 526092a530cSStanislav Gatev void transferValueOrConversionConstructor( 527092a530cSStanislav Gatev const CXXConstructExpr *E, const MatchFinder::MatchResult &MatchRes, 5289e0fc676SStanislav Gatev LatticeTransferState &State) { 529092a530cSStanislav Gatev assert(E->getNumArgs() > 0); 530092a530cSStanislav Gatev 5310e8d4a6dSYitzhak Mandelbaum assignOptionalValue(*E, State.Env, 532390029beSYitzhak Mandelbaum valueOrConversionHasValue(*E->getConstructor(), 533b000b770SStanislav Gatev *E->getArg(0), MatchRes, 534b000b770SStanislav Gatev State)); 535b000b770SStanislav Gatev } 536092a530cSStanislav Gatev 537b000b770SStanislav Gatev void transferAssignment(const CXXOperatorCallExpr *E, BoolValue &HasValueVal, 538b000b770SStanislav Gatev LatticeTransferState &State) { 539b000b770SStanislav Gatev assert(E->getNumArgs() > 0); 540b000b770SStanislav Gatev 541b000b770SStanislav Gatev auto *OptionalLoc = 542b000b770SStanislav Gatev State.Env.getStorageLocation(*E->getArg(0), SkipPast::Reference); 543a9ad689eSSam Estep if (OptionalLoc == nullptr) 544a9ad689eSSam Estep return; 545b000b770SStanislav Gatev 546b000b770SStanislav Gatev State.Env.setValue(*OptionalLoc, createOptionalValue(State.Env, HasValueVal)); 547b000b770SStanislav Gatev 548b000b770SStanislav Gatev // Assign a storage location for the whole expression. 549b000b770SStanislav Gatev State.Env.setStorageLocation(*E, *OptionalLoc); 550b000b770SStanislav Gatev } 551b000b770SStanislav Gatev 552b000b770SStanislav Gatev void transferValueOrConversionAssignment( 553b000b770SStanislav Gatev const CXXOperatorCallExpr *E, const MatchFinder::MatchResult &MatchRes, 554b000b770SStanislav Gatev LatticeTransferState &State) { 555b000b770SStanislav Gatev assert(E->getNumArgs() > 1); 556b000b770SStanislav Gatev transferAssignment(E, 557390029beSYitzhak Mandelbaum valueOrConversionHasValue(*E->getDirectCallee(), 55806decd0bSKazu Hirata *E->getArg(1), MatchRes, State), 559b000b770SStanislav Gatev State); 560b000b770SStanislav Gatev } 561b000b770SStanislav Gatev 562b000b770SStanislav Gatev void transferNulloptAssignment(const CXXOperatorCallExpr *E, 563b000b770SStanislav Gatev const MatchFinder::MatchResult &, 564b000b770SStanislav Gatev LatticeTransferState &State) { 565b000b770SStanislav Gatev transferAssignment(E, State.Env.getBoolLiteralValue(false), State); 5669e0fc676SStanislav Gatev } 5679e0fc676SStanislav Gatev 56848bc7150SMartin Braenne void transferSwap(StorageLocation *Loc1, StorageLocation *Loc2, 569d4fb829bSYitzhak Mandelbaum Environment &Env) { 570d4fb829bSYitzhak Mandelbaum // We account for cases where one or both of the optionals are not modeled, 571d4fb829bSYitzhak Mandelbaum // either lacking associated storage locations, or lacking values associated 572d4fb829bSYitzhak Mandelbaum // to such storage locations. 5732ddd57aeSStanislav Gatev 574d4fb829bSYitzhak Mandelbaum if (Loc1 == nullptr) { 575d4fb829bSYitzhak Mandelbaum if (Loc2 != nullptr) 576d4fb829bSYitzhak Mandelbaum Env.setValue(*Loc2, createOptionalValue(Env, Env.makeAtomicBoolValue())); 577d4fb829bSYitzhak Mandelbaum return; 578d4fb829bSYitzhak Mandelbaum } 579d4fb829bSYitzhak Mandelbaum if (Loc2 == nullptr) { 580d4fb829bSYitzhak Mandelbaum Env.setValue(*Loc1, createOptionalValue(Env, Env.makeAtomicBoolValue())); 581d4fb829bSYitzhak Mandelbaum return; 582d4fb829bSYitzhak Mandelbaum } 5832ddd57aeSStanislav Gatev 584d4fb829bSYitzhak Mandelbaum // Both expressions have locations, though they may not have corresponding 585d4fb829bSYitzhak Mandelbaum // values. In that case, we create a fresh value at this point. Note that if 586d4fb829bSYitzhak Mandelbaum // two branches both do this, they will not share the value, but it at least 587d4fb829bSYitzhak Mandelbaum // allows for local reasoning about the value. To avoid the above, we would 588d4fb829bSYitzhak Mandelbaum // need *lazy* value allocation. 589d4fb829bSYitzhak Mandelbaum // FIXME: allocate values lazily, instead of just creating a fresh value. 590d4fb829bSYitzhak Mandelbaum auto *Val1 = Env.getValue(*Loc1); 591d4fb829bSYitzhak Mandelbaum if (Val1 == nullptr) 592d4fb829bSYitzhak Mandelbaum Val1 = &createOptionalValue(Env, Env.makeAtomicBoolValue()); 593d4fb829bSYitzhak Mandelbaum 594d4fb829bSYitzhak Mandelbaum auto *Val2 = Env.getValue(*Loc2); 595d4fb829bSYitzhak Mandelbaum if (Val2 == nullptr) 596d4fb829bSYitzhak Mandelbaum Val2 = &createOptionalValue(Env, Env.makeAtomicBoolValue()); 597d4fb829bSYitzhak Mandelbaum 598d4fb829bSYitzhak Mandelbaum Env.setValue(*Loc1, *Val2); 599d4fb829bSYitzhak Mandelbaum Env.setValue(*Loc2, *Val1); 6002ddd57aeSStanislav Gatev } 6012ddd57aeSStanislav Gatev 6022ddd57aeSStanislav Gatev void transferSwapCall(const CXXMemberCallExpr *E, 6032ddd57aeSStanislav Gatev const MatchFinder::MatchResult &, 6042ddd57aeSStanislav Gatev LatticeTransferState &State) { 6052ddd57aeSStanislav Gatev assert(E->getNumArgs() == 1); 60648bc7150SMartin Braenne transferSwap(maybeSkipPointer( 60748bc7150SMartin Braenne State.Env.getStorageLocation(*E->getImplicitObjectArgument(), 60848bc7150SMartin Braenne SkipPast::Reference), 60948bc7150SMartin Braenne State.Env), 61048bc7150SMartin Braenne State.Env.getStorageLocation(*E->getArg(0), SkipPast::Reference), 61148bc7150SMartin Braenne State.Env); 6122ddd57aeSStanislav Gatev } 6132ddd57aeSStanislav Gatev 6142ddd57aeSStanislav Gatev void transferStdSwapCall(const CallExpr *E, const MatchFinder::MatchResult &, 6152ddd57aeSStanislav Gatev LatticeTransferState &State) { 6162ddd57aeSStanislav Gatev assert(E->getNumArgs() == 2); 61748bc7150SMartin Braenne transferSwap(State.Env.getStorageLocation(*E->getArg(0), SkipPast::Reference), 61848bc7150SMartin Braenne State.Env.getStorageLocation(*E->getArg(1), SkipPast::Reference), 61948bc7150SMartin Braenne State.Env); 6202ddd57aeSStanislav Gatev } 6212ddd57aeSStanislav Gatev 62225956d55SAMS21 void transferStdForwardCall(const CallExpr *E, const MatchFinder::MatchResult &, 62325956d55SAMS21 LatticeTransferState &State) { 62425956d55SAMS21 assert(E->getNumArgs() == 1); 62525956d55SAMS21 62625956d55SAMS21 StorageLocation *LocRet = State.Env.getStorageLocation(*E, SkipPast::None); 62725956d55SAMS21 if (LocRet != nullptr) 62825956d55SAMS21 return; 62925956d55SAMS21 63025956d55SAMS21 StorageLocation *LocArg = 63125956d55SAMS21 State.Env.getStorageLocation(*E->getArg(0), SkipPast::Reference); 63225956d55SAMS21 63325956d55SAMS21 if (LocArg == nullptr) 63425956d55SAMS21 return; 63525956d55SAMS21 63625956d55SAMS21 Value *ValArg = State.Env.getValue(*LocArg); 63725956d55SAMS21 if (ValArg == nullptr) 63825956d55SAMS21 ValArg = &createOptionalValue(State.Env, State.Env.makeAtomicBoolValue()); 63925956d55SAMS21 64025956d55SAMS21 // Create a new storage location 64125956d55SAMS21 LocRet = &State.Env.createStorageLocation(*E); 64225956d55SAMS21 State.Env.setStorageLocation(*E, *LocRet); 64325956d55SAMS21 64425956d55SAMS21 State.Env.setValue(*LocRet, *ValArg); 64525956d55SAMS21 } 64625956d55SAMS21 647390029beSYitzhak Mandelbaum BoolValue &evaluateEquality(Environment &Env, BoolValue &EqVal, BoolValue &LHS, 648390029beSYitzhak Mandelbaum BoolValue &RHS) { 649390029beSYitzhak Mandelbaum // Logically, an optional<T> object is composed of two values - a `has_value` 650390029beSYitzhak Mandelbaum // bit and a value of type T. Equality of optional objects compares both 651390029beSYitzhak Mandelbaum // values. Therefore, merely comparing the `has_value` bits isn't sufficient: 652390029beSYitzhak Mandelbaum // when two optional objects are engaged, the equality of their respective 653390029beSYitzhak Mandelbaum // values of type T matters. Since we only track the `has_value` bits, we 654390029beSYitzhak Mandelbaum // can't make any conclusions about equality when we know that two optional 655390029beSYitzhak Mandelbaum // objects are engaged. 656390029beSYitzhak Mandelbaum // 657390029beSYitzhak Mandelbaum // We express this as two facts about the equality: 658390029beSYitzhak Mandelbaum // a) EqVal => (LHS & RHS) v (!RHS & !LHS) 659390029beSYitzhak Mandelbaum // If they are equal, then either both are set or both are unset. 660390029beSYitzhak Mandelbaum // b) (!LHS & !RHS) => EqVal 661390029beSYitzhak Mandelbaum // If neither is set, then they are equal. 662390029beSYitzhak Mandelbaum // We rewrite b) as !EqVal => (LHS v RHS), for a more compact formula. 663390029beSYitzhak Mandelbaum return Env.makeAnd( 664390029beSYitzhak Mandelbaum Env.makeImplication( 665390029beSYitzhak Mandelbaum EqVal, Env.makeOr(Env.makeAnd(LHS, RHS), 666390029beSYitzhak Mandelbaum Env.makeAnd(Env.makeNot(LHS), Env.makeNot(RHS)))), 667390029beSYitzhak Mandelbaum Env.makeImplication(Env.makeNot(EqVal), Env.makeOr(LHS, RHS))); 668390029beSYitzhak Mandelbaum } 669390029beSYitzhak Mandelbaum 670390029beSYitzhak Mandelbaum void transferOptionalAndOptionalCmp(const clang::CXXOperatorCallExpr *CmpExpr, 671390029beSYitzhak Mandelbaum const MatchFinder::MatchResult &, 672390029beSYitzhak Mandelbaum LatticeTransferState &State) { 673390029beSYitzhak Mandelbaum Environment &Env = State.Env; 674390029beSYitzhak Mandelbaum auto *CmpValue = &forceBoolValue(Env, *CmpExpr); 675390029beSYitzhak Mandelbaum if (auto *LHasVal = getHasValue( 676390029beSYitzhak Mandelbaum Env, Env.getValue(*CmpExpr->getArg(0), SkipPast::Reference))) 677390029beSYitzhak Mandelbaum if (auto *RHasVal = getHasValue( 678390029beSYitzhak Mandelbaum Env, Env.getValue(*CmpExpr->getArg(1), SkipPast::Reference))) { 679390029beSYitzhak Mandelbaum if (CmpExpr->getOperator() == clang::OO_ExclaimEqual) 680390029beSYitzhak Mandelbaum CmpValue = &State.Env.makeNot(*CmpValue); 681390029beSYitzhak Mandelbaum Env.addToFlowCondition( 682390029beSYitzhak Mandelbaum evaluateEquality(Env, *CmpValue, *LHasVal, *RHasVal)); 683390029beSYitzhak Mandelbaum } 684390029beSYitzhak Mandelbaum } 685390029beSYitzhak Mandelbaum 686390029beSYitzhak Mandelbaum void transferOptionalAndValueCmp(const clang::CXXOperatorCallExpr *CmpExpr, 687390029beSYitzhak Mandelbaum const clang::Expr *E, Environment &Env) { 688390029beSYitzhak Mandelbaum auto *CmpValue = &forceBoolValue(Env, *CmpExpr); 689390029beSYitzhak Mandelbaum if (auto *HasVal = getHasValue(Env, Env.getValue(*E, SkipPast::Reference))) { 690390029beSYitzhak Mandelbaum if (CmpExpr->getOperator() == clang::OO_ExclaimEqual) 691390029beSYitzhak Mandelbaum CmpValue = &Env.makeNot(*CmpValue); 692390029beSYitzhak Mandelbaum Env.addToFlowCondition(evaluateEquality(Env, *CmpValue, *HasVal, 693390029beSYitzhak Mandelbaum Env.getBoolLiteralValue(true))); 694390029beSYitzhak Mandelbaum } 695390029beSYitzhak Mandelbaum } 696390029beSYitzhak Mandelbaum 6976ad0788cSKazu Hirata std::optional<StatementMatcher> 698a184a0d8SYitzhak Mandelbaum ignorableOptional(const UncheckedOptionalAccessModelOptions &Options) { 6995d22d1f5SYitzhak Mandelbaum if (Options.IgnoreSmartPointerDereference) { 7005d22d1f5SYitzhak Mandelbaum auto SmartPtrUse = expr(ignoringParenImpCasts(cxxOperatorCallExpr( 7015d22d1f5SYitzhak Mandelbaum anyOf(hasOverloadedOperatorName("->"), hasOverloadedOperatorName("*")), 7025d22d1f5SYitzhak Mandelbaum unless(hasArgument(0, expr(hasOptionalType())))))); 7035d22d1f5SYitzhak Mandelbaum return expr( 7045d22d1f5SYitzhak Mandelbaum anyOf(SmartPtrUse, memberExpr(hasObjectExpression(SmartPtrUse)))); 7055d22d1f5SYitzhak Mandelbaum } 70634e0d057SKazu Hirata return std::nullopt; 707a184a0d8SYitzhak Mandelbaum } 708a184a0d8SYitzhak Mandelbaum 70958fe7f96SSam Estep StatementMatcher 7106ad0788cSKazu Hirata valueCall(const std::optional<StatementMatcher> &IgnorableOptional) { 71158fe7f96SSam Estep return isOptionalMemberCallWithName("value", IgnorableOptional); 71258fe7f96SSam Estep } 71358fe7f96SSam Estep 71458fe7f96SSam Estep StatementMatcher 7156ad0788cSKazu Hirata valueOperatorCall(const std::optional<StatementMatcher> &IgnorableOptional) { 71658fe7f96SSam Estep return expr(anyOf(isOptionalOperatorCallWithName("*", IgnorableOptional), 71758fe7f96SSam Estep isOptionalOperatorCallWithName("->", IgnorableOptional))); 71858fe7f96SSam Estep } 71958fe7f96SSam Estep 7205d22d1f5SYitzhak Mandelbaum auto buildTransferMatchSwitch() { 721b000b770SStanislav Gatev // FIXME: Evaluate the efficiency of matchers. If using matchers results in a 722b000b770SStanislav Gatev // lot of duplicated work (e.g. string comparisons), consider providing APIs 723b000b770SStanislav Gatev // that avoid it through memoization. 7247538b360SWei Yi Tee return CFGMatchSwitchBuilder<LatticeTransferState>() 725af98b0afSStanislav Gatev // Attach a symbolic "has_value" state to optional values that we see for 726af98b0afSStanislav Gatev // the first time. 7277538b360SWei Yi Tee .CaseOfCFGStmt<Expr>( 7286adfc64eSYitzhak Mandelbaum expr(anyOf(declRefExpr(), memberExpr()), hasOptionalType()), 729af98b0afSStanislav Gatev initializeOptionalReference) 730af98b0afSStanislav Gatev 7319e0fc676SStanislav Gatev // make_optional 7327538b360SWei Yi Tee .CaseOfCFGStmt<CallExpr>(isMakeOptionalCall(), transferMakeOptionalCall) 733092a530cSStanislav Gatev 7340e8d4a6dSYitzhak Mandelbaum // optional::optional (in place) 7357538b360SWei Yi Tee .CaseOfCFGStmt<CXXConstructExpr>( 736092a530cSStanislav Gatev isOptionalInPlaceConstructor(), 737092a530cSStanislav Gatev [](const CXXConstructExpr *E, const MatchFinder::MatchResult &, 738092a530cSStanislav Gatev LatticeTransferState &State) { 7390e8d4a6dSYitzhak Mandelbaum assignOptionalValue(*E, State.Env, 7400e8d4a6dSYitzhak Mandelbaum State.Env.getBoolLiteralValue(true)); 741092a530cSStanislav Gatev }) 7420e8d4a6dSYitzhak Mandelbaum // nullopt_t::nullopt_t 7437538b360SWei Yi Tee .CaseOfCFGStmt<CXXConstructExpr>( 744390029beSYitzhak Mandelbaum isNulloptConstructor(), 745092a530cSStanislav Gatev [](const CXXConstructExpr *E, const MatchFinder::MatchResult &, 746092a530cSStanislav Gatev LatticeTransferState &State) { 7470e8d4a6dSYitzhak Mandelbaum assignOptionalValue(*E, State.Env, 748092a530cSStanislav Gatev State.Env.getBoolLiteralValue(false)); 749092a530cSStanislav Gatev }) 7500e8d4a6dSYitzhak Mandelbaum // optional::optional(nullopt_t) 751390029beSYitzhak Mandelbaum .CaseOfCFGStmt<CXXConstructExpr>( 752390029beSYitzhak Mandelbaum isOptionalNulloptConstructor(), 753390029beSYitzhak Mandelbaum [](const CXXConstructExpr *E, const MatchFinder::MatchResult &, 754390029beSYitzhak Mandelbaum LatticeTransferState &State) { 7550e8d4a6dSYitzhak Mandelbaum assignOptionalValue(*E, State.Env, 7560e8d4a6dSYitzhak Mandelbaum State.Env.getBoolLiteralValue(false)); 757390029beSYitzhak Mandelbaum }) 7580e8d4a6dSYitzhak Mandelbaum // optional::optional (value/conversion) 7597538b360SWei Yi Tee .CaseOfCFGStmt<CXXConstructExpr>(isOptionalValueOrConversionConstructor(), 760092a530cSStanislav Gatev transferValueOrConversionConstructor) 7619e0fc676SStanislav Gatev 762b000b770SStanislav Gatev // optional::operator= 7637538b360SWei Yi Tee .CaseOfCFGStmt<CXXOperatorCallExpr>( 7647538b360SWei Yi Tee isOptionalValueOrConversionAssignment(), 765b000b770SStanislav Gatev transferValueOrConversionAssignment) 7667538b360SWei Yi Tee .CaseOfCFGStmt<CXXOperatorCallExpr>(isOptionalNulloptAssignment(), 767b000b770SStanislav Gatev transferNulloptAssignment) 768b000b770SStanislav Gatev 769af98b0afSStanislav Gatev // optional::value 7707538b360SWei Yi Tee .CaseOfCFGStmt<CXXMemberCallExpr>( 7715d22d1f5SYitzhak Mandelbaum valueCall(std::nullopt), 772092a530cSStanislav Gatev [](const CXXMemberCallExpr *E, const MatchFinder::MatchResult &, 773092a530cSStanislav Gatev LatticeTransferState &State) { 774af98b0afSStanislav Gatev transferUnwrapCall(E, E->getImplicitObjectArgument(), State); 775af98b0afSStanislav Gatev }) 776af98b0afSStanislav Gatev 777af98b0afSStanislav Gatev // optional::operator*, optional::operator-> 77848bc7150SMartin Braenne // FIXME: This does something slightly strange for `operator->`. 77948bc7150SMartin Braenne // `transferUnwrapCall()` may create a new value of type `T` for the 78048bc7150SMartin Braenne // `optional<T>`, and it associates that value with `E`. In the case of 78148bc7150SMartin Braenne // `operator->`, `E` is a pointer. As a result, we associate an 78248bc7150SMartin Braenne // expression of pointer type with a storage location of non-pointer type 78348bc7150SMartin Braenne // `T`. This can confound other code that expects expressions of 78448bc7150SMartin Braenne // pointer type to be associated with `PointerValue`s, such as the 78548bc7150SMartin Braenne // centrally provided accessors `getImplicitObjectLocation()` and 78648bc7150SMartin Braenne // `getBaseObjectLocation()`, and this is the reason we need to use our 78748bc7150SMartin Braenne // own 'maybeSkipPointer()` and `getValueBehindPossiblePointer()` instead 78848bc7150SMartin Braenne // of these accessors. 7895d22d1f5SYitzhak Mandelbaum .CaseOfCFGStmt<CallExpr>(valueOperatorCall(std::nullopt), 7907538b360SWei Yi Tee [](const CallExpr *E, 7917538b360SWei Yi Tee const MatchFinder::MatchResult &, 792092a530cSStanislav Gatev LatticeTransferState &State) { 793af98b0afSStanislav Gatev transferUnwrapCall(E, E->getArg(0), State); 794af98b0afSStanislav Gatev }) 795af98b0afSStanislav Gatev 796af98b0afSStanislav Gatev // optional::has_value 7977538b360SWei Yi Tee .CaseOfCFGStmt<CXXMemberCallExpr>( 7987538b360SWei Yi Tee isOptionalMemberCallWithName("has_value"), 799af98b0afSStanislav Gatev transferOptionalHasValueCall) 800af98b0afSStanislav Gatev 8019e0fc676SStanislav Gatev // optional::operator bool 8027538b360SWei Yi Tee .CaseOfCFGStmt<CXXMemberCallExpr>( 8037538b360SWei Yi Tee isOptionalMemberCallWithName("operator bool"), 8049e0fc676SStanislav Gatev transferOptionalHasValueCall) 8059e0fc676SStanislav Gatev 8069e0fc676SStanislav Gatev // optional::emplace 8077538b360SWei Yi Tee .CaseOfCFGStmt<CXXMemberCallExpr>( 808092a530cSStanislav Gatev isOptionalMemberCallWithName("emplace"), 809092a530cSStanislav Gatev [](const CXXMemberCallExpr *E, const MatchFinder::MatchResult &, 810092a530cSStanislav Gatev LatticeTransferState &State) { 8110e8d4a6dSYitzhak Mandelbaum assignOptionalValue(*E->getImplicitObjectArgument(), State.Env, 812092a530cSStanislav Gatev State.Env.getBoolLiteralValue(true)); 813092a530cSStanislav Gatev }) 8149e0fc676SStanislav Gatev 8159e0fc676SStanislav Gatev // optional::reset 8167538b360SWei Yi Tee .CaseOfCFGStmt<CXXMemberCallExpr>( 817092a530cSStanislav Gatev isOptionalMemberCallWithName("reset"), 818092a530cSStanislav Gatev [](const CXXMemberCallExpr *E, const MatchFinder::MatchResult &, 819092a530cSStanislav Gatev LatticeTransferState &State) { 8200e8d4a6dSYitzhak Mandelbaum assignOptionalValue(*E->getImplicitObjectArgument(), State.Env, 821092a530cSStanislav Gatev State.Env.getBoolLiteralValue(false)); 822092a530cSStanislav Gatev }) 8239e0fc676SStanislav Gatev 8242ddd57aeSStanislav Gatev // optional::swap 8257538b360SWei Yi Tee .CaseOfCFGStmt<CXXMemberCallExpr>(isOptionalMemberCallWithName("swap"), 8262ddd57aeSStanislav Gatev transferSwapCall) 8272ddd57aeSStanislav Gatev 8282ddd57aeSStanislav Gatev // std::swap 8297538b360SWei Yi Tee .CaseOfCFGStmt<CallExpr>(isStdSwapCall(), transferStdSwapCall) 8302ddd57aeSStanislav Gatev 83125956d55SAMS21 // std::forward 83225956d55SAMS21 .CaseOfCFGStmt<CallExpr>(isStdForwardCall(), transferStdForwardCall) 83325956d55SAMS21 8347f076004SYitzhak Mandelbaum // opt.value_or("").empty() 8357538b360SWei Yi Tee .CaseOfCFGStmt<Expr>(isValueOrStringEmptyCall(), 8367538b360SWei Yi Tee transferValueOrStringEmptyCall) 8377f076004SYitzhak Mandelbaum 8387f076004SYitzhak Mandelbaum // opt.value_or(X) != X 8397538b360SWei Yi Tee .CaseOfCFGStmt<Expr>(isValueOrNotEqX(), transferValueOrNotEqX) 8407f076004SYitzhak Mandelbaum 841390029beSYitzhak Mandelbaum // Comparisons (==, !=): 842390029beSYitzhak Mandelbaum .CaseOfCFGStmt<CXXOperatorCallExpr>( 843390029beSYitzhak Mandelbaum isComparisonOperatorCall(hasAnyOptionalType(), hasAnyOptionalType()), 844390029beSYitzhak Mandelbaum transferOptionalAndOptionalCmp) 845390029beSYitzhak Mandelbaum .CaseOfCFGStmt<CXXOperatorCallExpr>( 846390029beSYitzhak Mandelbaum isComparisonOperatorCall(hasOptionalType(), 847390029beSYitzhak Mandelbaum unless(hasAnyOptionalType())), 848390029beSYitzhak Mandelbaum [](const clang::CXXOperatorCallExpr *Cmp, 849390029beSYitzhak Mandelbaum const MatchFinder::MatchResult &, LatticeTransferState &State) { 850390029beSYitzhak Mandelbaum transferOptionalAndValueCmp(Cmp, Cmp->getArg(0), State.Env); 851390029beSYitzhak Mandelbaum }) 852390029beSYitzhak Mandelbaum .CaseOfCFGStmt<CXXOperatorCallExpr>( 853390029beSYitzhak Mandelbaum isComparisonOperatorCall(unless(hasAnyOptionalType()), 854390029beSYitzhak Mandelbaum hasOptionalType()), 855390029beSYitzhak Mandelbaum [](const clang::CXXOperatorCallExpr *Cmp, 856390029beSYitzhak Mandelbaum const MatchFinder::MatchResult &, LatticeTransferState &State) { 857390029beSYitzhak Mandelbaum transferOptionalAndValueCmp(Cmp, Cmp->getArg(1), State.Env); 858390029beSYitzhak Mandelbaum }) 859390029beSYitzhak Mandelbaum 86065e710c3SStanislav Gatev // returns optional 8617538b360SWei Yi Tee .CaseOfCFGStmt<CallExpr>(isCallReturningOptional(), 86265e710c3SStanislav Gatev transferCallReturningOptional) 86365e710c3SStanislav Gatev 864af98b0afSStanislav Gatev .Build(); 865af98b0afSStanislav Gatev } 866af98b0afSStanislav Gatev 86758fe7f96SSam Estep std::vector<SourceLocation> diagnoseUnwrapCall(const Expr *UnwrapExpr, 86858fe7f96SSam Estep const Expr *ObjectExpr, 86958fe7f96SSam Estep const Environment &Env) { 87048bc7150SMartin Braenne if (auto *OptionalVal = getValueBehindPossiblePointer(*ObjectExpr, Env)) { 87158fe7f96SSam Estep auto *Prop = OptionalVal->getProperty("has_value"); 87258fe7f96SSam Estep if (auto *HasValueVal = cast_or_null<BoolValue>(Prop)) { 87358fe7f96SSam Estep if (Env.flowConditionImplies(*HasValueVal)) 87458fe7f96SSam Estep return {}; 87558fe7f96SSam Estep } 87658fe7f96SSam Estep } 87758fe7f96SSam Estep 87858fe7f96SSam Estep // Record that this unwrap is *not* provably safe. 87958fe7f96SSam Estep // FIXME: include either the name of the optional (if applicable) or a source 88058fe7f96SSam Estep // range of the access for easier interpretation of the result. 88158fe7f96SSam Estep return {ObjectExpr->getBeginLoc()}; 88258fe7f96SSam Estep } 88358fe7f96SSam Estep 88458fe7f96SSam Estep auto buildDiagnoseMatchSwitch( 88558fe7f96SSam Estep const UncheckedOptionalAccessModelOptions &Options) { 88658fe7f96SSam Estep // FIXME: Evaluate the efficiency of matchers. If using matchers results in a 88758fe7f96SSam Estep // lot of duplicated work (e.g. string comparisons), consider providing APIs 88858fe7f96SSam Estep // that avoid it through memoization. 88958fe7f96SSam Estep auto IgnorableOptional = ignorableOptional(Options); 8907538b360SWei Yi Tee return CFGMatchSwitchBuilder<const Environment, std::vector<SourceLocation>>() 89158fe7f96SSam Estep // optional::value 8927538b360SWei Yi Tee .CaseOfCFGStmt<CXXMemberCallExpr>( 89358fe7f96SSam Estep valueCall(IgnorableOptional), 89458fe7f96SSam Estep [](const CXXMemberCallExpr *E, const MatchFinder::MatchResult &, 89558fe7f96SSam Estep const Environment &Env) { 89658fe7f96SSam Estep return diagnoseUnwrapCall(E, E->getImplicitObjectArgument(), Env); 89758fe7f96SSam Estep }) 89858fe7f96SSam Estep 89958fe7f96SSam Estep // optional::operator*, optional::operator-> 9007538b360SWei Yi Tee .CaseOfCFGStmt<CallExpr>( 90158fe7f96SSam Estep valueOperatorCall(IgnorableOptional), 90258fe7f96SSam Estep [](const CallExpr *E, const MatchFinder::MatchResult &, 90358fe7f96SSam Estep const Environment &Env) { 90458fe7f96SSam Estep return diagnoseUnwrapCall(E, E->getArg(0), Env); 90558fe7f96SSam Estep }) 90658fe7f96SSam Estep .Build(); 90758fe7f96SSam Estep } 90858fe7f96SSam Estep 909af98b0afSStanislav Gatev } // namespace 910af98b0afSStanislav Gatev 9117e63a0d4SYitzhak Mandelbaum ast_matchers::DeclarationMatcher 9127e63a0d4SYitzhak Mandelbaum UncheckedOptionalAccessModel::optionalClassDecl() { 9137e63a0d4SYitzhak Mandelbaum return optionalClass(); 9147e63a0d4SYitzhak Mandelbaum } 9157e63a0d4SYitzhak Mandelbaum 9165d22d1f5SYitzhak Mandelbaum UncheckedOptionalAccessModel::UncheckedOptionalAccessModel(ASTContext &Ctx) 917cf1f978dSSam Estep : DataflowAnalysis<UncheckedOptionalAccessModel, NoopLattice>(Ctx), 9185d22d1f5SYitzhak Mandelbaum TransferMatchSwitch(buildTransferMatchSwitch()) {} 919af98b0afSStanislav Gatev 9206b991ba4SYitzhak Mandelbaum void UncheckedOptionalAccessModel::transfer(const CFGElement &Elt, 9217538b360SWei Yi Tee NoopLattice &L, Environment &Env) { 922af98b0afSStanislav Gatev LatticeTransferState State(L, Env); 9236b991ba4SYitzhak Mandelbaum TransferMatchSwitch(Elt, getASTContext(), State); 924af98b0afSStanislav Gatev } 925af98b0afSStanislav Gatev 926c0725865SYitzhak Mandelbaum ComparisonResult UncheckedOptionalAccessModel::compare( 927c0725865SYitzhak Mandelbaum QualType Type, const Value &Val1, const Environment &Env1, 928c0725865SYitzhak Mandelbaum const Value &Val2, const Environment &Env2) { 929c0725865SYitzhak Mandelbaum if (!isOptionalType(Type)) 930c0725865SYitzhak Mandelbaum return ComparisonResult::Unknown; 931d34fbf2dSYitzhak Mandelbaum bool MustNonEmpty1 = isNonEmptyOptional(Val1, Env1); 932d34fbf2dSYitzhak Mandelbaum bool MustNonEmpty2 = isNonEmptyOptional(Val2, Env2); 93325956d55SAMS21 if (MustNonEmpty1 && MustNonEmpty2) 93425956d55SAMS21 return ComparisonResult::Same; 935d34fbf2dSYitzhak Mandelbaum // If exactly one is true, then they're different, no reason to check whether 936d34fbf2dSYitzhak Mandelbaum // they're definitely empty. 93725956d55SAMS21 if (MustNonEmpty1 || MustNonEmpty2) 93825956d55SAMS21 return ComparisonResult::Different; 939d34fbf2dSYitzhak Mandelbaum // Check if they're both definitely empty. 940d34fbf2dSYitzhak Mandelbaum return (isEmptyOptional(Val1, Env1) && isEmptyOptional(Val2, Env2)) 941c0725865SYitzhak Mandelbaum ? ComparisonResult::Same 942c0725865SYitzhak Mandelbaum : ComparisonResult::Different; 9438fcdd625SStanislav Gatev } 9448fcdd625SStanislav Gatev 9458fcdd625SStanislav Gatev bool UncheckedOptionalAccessModel::merge(QualType Type, const Value &Val1, 9468fcdd625SStanislav Gatev const Environment &Env1, 9478fcdd625SStanislav Gatev const Value &Val2, 9488fcdd625SStanislav Gatev const Environment &Env2, 9498fcdd625SStanislav Gatev Value &MergedVal, 9508fcdd625SStanislav Gatev Environment &MergedEnv) { 951c0725865SYitzhak Mandelbaum if (!isOptionalType(Type)) 9528fcdd625SStanislav Gatev return true; 953d34fbf2dSYitzhak Mandelbaum // FIXME: uses same approach as join for `BoolValues`. Requires non-const 954d34fbf2dSYitzhak Mandelbaum // values, though, so will require updating the interface. 9558fcdd625SStanislav Gatev auto &HasValueVal = MergedEnv.makeAtomicBoolValue(); 956d34fbf2dSYitzhak Mandelbaum bool MustNonEmpty1 = isNonEmptyOptional(Val1, Env1); 957d34fbf2dSYitzhak Mandelbaum bool MustNonEmpty2 = isNonEmptyOptional(Val2, Env2); 958d34fbf2dSYitzhak Mandelbaum if (MustNonEmpty1 && MustNonEmpty2) 9598fcdd625SStanislav Gatev MergedEnv.addToFlowCondition(HasValueVal); 960d34fbf2dSYitzhak Mandelbaum else if ( 961d34fbf2dSYitzhak Mandelbaum // Only make the costly calls to `isEmptyOptional` if we got "unknown" 962d34fbf2dSYitzhak Mandelbaum // (false) for both calls to `isNonEmptyOptional`. 963d34fbf2dSYitzhak Mandelbaum !MustNonEmpty1 && !MustNonEmpty2 && isEmptyOptional(Val1, Env1) && 964d34fbf2dSYitzhak Mandelbaum isEmptyOptional(Val2, Env2)) 9658fcdd625SStanislav Gatev MergedEnv.addToFlowCondition(MergedEnv.makeNot(HasValueVal)); 9668fcdd625SStanislav Gatev setHasValue(MergedVal, HasValueVal); 9678fcdd625SStanislav Gatev return true; 9688fcdd625SStanislav Gatev } 9698fcdd625SStanislav Gatev 970d34fbf2dSYitzhak Mandelbaum Value *UncheckedOptionalAccessModel::widen(QualType Type, Value &Prev, 971d34fbf2dSYitzhak Mandelbaum const Environment &PrevEnv, 972d34fbf2dSYitzhak Mandelbaum Value &Current, 973d34fbf2dSYitzhak Mandelbaum Environment &CurrentEnv) { 974d34fbf2dSYitzhak Mandelbaum switch (compare(Type, Prev, PrevEnv, Current, CurrentEnv)) { 975d34fbf2dSYitzhak Mandelbaum case ComparisonResult::Same: 976d34fbf2dSYitzhak Mandelbaum return &Prev; 977d34fbf2dSYitzhak Mandelbaum case ComparisonResult::Different: 978d34fbf2dSYitzhak Mandelbaum if (auto *PrevHasVal = 979d34fbf2dSYitzhak Mandelbaum cast_or_null<BoolValue>(Prev.getProperty("has_value"))) { 980d34fbf2dSYitzhak Mandelbaum if (isa<TopBoolValue>(PrevHasVal)) 981d34fbf2dSYitzhak Mandelbaum return &Prev; 982d34fbf2dSYitzhak Mandelbaum } 983d34fbf2dSYitzhak Mandelbaum if (auto *CurrentHasVal = 984d34fbf2dSYitzhak Mandelbaum cast_or_null<BoolValue>(Current.getProperty("has_value"))) { 985d34fbf2dSYitzhak Mandelbaum if (isa<TopBoolValue>(CurrentHasVal)) 986d34fbf2dSYitzhak Mandelbaum return &Current; 987d34fbf2dSYitzhak Mandelbaum } 988d34fbf2dSYitzhak Mandelbaum return &createOptionalValue(CurrentEnv, CurrentEnv.makeTopBoolValue()); 989d34fbf2dSYitzhak Mandelbaum case ComparisonResult::Unknown: 990d34fbf2dSYitzhak Mandelbaum return nullptr; 991d34fbf2dSYitzhak Mandelbaum } 992d34fbf2dSYitzhak Mandelbaum llvm_unreachable("all cases covered in switch"); 993d34fbf2dSYitzhak Mandelbaum } 994d34fbf2dSYitzhak Mandelbaum 99558fe7f96SSam Estep UncheckedOptionalAccessDiagnoser::UncheckedOptionalAccessDiagnoser( 99658fe7f96SSam Estep UncheckedOptionalAccessModelOptions Options) 99758fe7f96SSam Estep : DiagnoseMatchSwitch(buildDiagnoseMatchSwitch(Options)) {} 99858fe7f96SSam Estep 99958fe7f96SSam Estep std::vector<SourceLocation> UncheckedOptionalAccessDiagnoser::diagnose( 10007538b360SWei Yi Tee ASTContext &Ctx, const CFGElement *Elt, const Environment &Env) { 10017538b360SWei Yi Tee return DiagnoseMatchSwitch(*Elt, Ctx, Env); 100258fe7f96SSam Estep } 100358fe7f96SSam Estep 1004af98b0afSStanislav Gatev } // namespace dataflow 1005af98b0afSStanislav Gatev } // namespace clang 1006