10b57cec5SDimitry Andric //== BoolAssignmentChecker.cpp - Boolean assignment checker -----*- C++ -*--==// 20b57cec5SDimitry Andric // 30b57cec5SDimitry Andric // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. 40b57cec5SDimitry Andric // See https://llvm.org/LICENSE.txt for license information. 50b57cec5SDimitry Andric // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception 60b57cec5SDimitry Andric // 70b57cec5SDimitry Andric //===----------------------------------------------------------------------===// 80b57cec5SDimitry Andric // 90b57cec5SDimitry Andric // This defines BoolAssignmentChecker, a builtin check in ExprEngine that 100b57cec5SDimitry Andric // performs checks for assignment of non-Boolean values to Boolean variables. 110b57cec5SDimitry Andric // 120b57cec5SDimitry Andric //===----------------------------------------------------------------------===// 130b57cec5SDimitry Andric 140b57cec5SDimitry Andric #include "clang/StaticAnalyzer/Checkers/BuiltinCheckerRegistration.h" 15*81ad6265SDimitry Andric #include "clang/StaticAnalyzer/Checkers/Taint.h" 160b57cec5SDimitry Andric #include "clang/StaticAnalyzer/Core/BugReporter/BugType.h" 170b57cec5SDimitry Andric #include "clang/StaticAnalyzer/Core/Checker.h" 180b57cec5SDimitry Andric #include "clang/StaticAnalyzer/Core/CheckerManager.h" 190b57cec5SDimitry Andric #include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h" 200b57cec5SDimitry Andric 210b57cec5SDimitry Andric using namespace clang; 220b57cec5SDimitry Andric using namespace ento; 230b57cec5SDimitry Andric 240b57cec5SDimitry Andric namespace { 250b57cec5SDimitry Andric class BoolAssignmentChecker : public Checker< check::Bind > { 260b57cec5SDimitry Andric mutable std::unique_ptr<BuiltinBug> BT; 27*81ad6265SDimitry Andric void emitReport(ProgramStateRef state, CheckerContext &C, 28*81ad6265SDimitry Andric bool IsTainted = false) const; 29*81ad6265SDimitry Andric 300b57cec5SDimitry Andric public: 310b57cec5SDimitry Andric void checkBind(SVal loc, SVal val, const Stmt *S, CheckerContext &C) const; 320b57cec5SDimitry Andric }; 330b57cec5SDimitry Andric } // end anonymous namespace 340b57cec5SDimitry Andric 35*81ad6265SDimitry Andric void BoolAssignmentChecker::emitReport(ProgramStateRef state, CheckerContext &C, 36*81ad6265SDimitry Andric bool IsTainted) const { 370b57cec5SDimitry Andric if (ExplodedNode *N = C.generateNonFatalErrorNode(state)) { 380b57cec5SDimitry Andric if (!BT) 390b57cec5SDimitry Andric BT.reset(new BuiltinBug(this, "Assignment of a non-Boolean value")); 40a7dea167SDimitry Andric 41*81ad6265SDimitry Andric StringRef Msg = IsTainted ? "Might assign a tainted non-Boolean value" 42*81ad6265SDimitry Andric : "Assignment of a non-Boolean value"; 43*81ad6265SDimitry Andric C.emitReport(std::make_unique<PathSensitiveBugReport>(*BT, Msg, N)); 440b57cec5SDimitry Andric } 450b57cec5SDimitry Andric } 460b57cec5SDimitry Andric 470b57cec5SDimitry Andric static bool isBooleanType(QualType Ty) { 480b57cec5SDimitry Andric if (Ty->isBooleanType()) // C++ or C99 490b57cec5SDimitry Andric return true; 500b57cec5SDimitry Andric 510b57cec5SDimitry Andric if (const TypedefType *TT = Ty->getAs<TypedefType>()) 520b57cec5SDimitry Andric return TT->getDecl()->getName() == "BOOL" || // Objective-C 530b57cec5SDimitry Andric TT->getDecl()->getName() == "_Bool" || // stdbool.h < C99 540b57cec5SDimitry Andric TT->getDecl()->getName() == "Boolean"; // MacTypes.h 550b57cec5SDimitry Andric 560b57cec5SDimitry Andric return false; 570b57cec5SDimitry Andric } 580b57cec5SDimitry Andric 590b57cec5SDimitry Andric void BoolAssignmentChecker::checkBind(SVal loc, SVal val, const Stmt *S, 600b57cec5SDimitry Andric CheckerContext &C) const { 610b57cec5SDimitry Andric 620b57cec5SDimitry Andric // We are only interested in stores into Booleans. 630b57cec5SDimitry Andric const TypedValueRegion *TR = 640b57cec5SDimitry Andric dyn_cast_or_null<TypedValueRegion>(loc.getAsRegion()); 650b57cec5SDimitry Andric 660b57cec5SDimitry Andric if (!TR) 670b57cec5SDimitry Andric return; 680b57cec5SDimitry Andric 690b57cec5SDimitry Andric QualType valTy = TR->getValueType(); 700b57cec5SDimitry Andric 710b57cec5SDimitry Andric if (!isBooleanType(valTy)) 720b57cec5SDimitry Andric return; 730b57cec5SDimitry Andric 740b57cec5SDimitry Andric // Get the value of the right-hand side. We only care about values 750b57cec5SDimitry Andric // that are defined (UnknownVals and UndefinedVals are handled by other 760b57cec5SDimitry Andric // checkers). 775ffd83dbSDimitry Andric Optional<NonLoc> NV = val.getAs<NonLoc>(); 785ffd83dbSDimitry Andric if (!NV) 790b57cec5SDimitry Andric return; 800b57cec5SDimitry Andric 810b57cec5SDimitry Andric // Check if the assigned value meets our criteria for correctness. It must 820b57cec5SDimitry Andric // be a value that is either 0 or 1. One way to check this is to see if 830b57cec5SDimitry Andric // the value is possibly < 0 (for a negative value) or greater than 1. 840b57cec5SDimitry Andric ProgramStateRef state = C.getState(); 850b57cec5SDimitry Andric SValBuilder &svalBuilder = C.getSValBuilder(); 865ffd83dbSDimitry Andric BasicValueFactory &BVF = svalBuilder.getBasicValueFactory(); 870b57cec5SDimitry Andric ConstraintManager &CM = C.getConstraintManager(); 880b57cec5SDimitry Andric 895ffd83dbSDimitry Andric llvm::APSInt Zero = BVF.getValue(0, valTy); 905ffd83dbSDimitry Andric llvm::APSInt One = BVF.getValue(1, valTy); 910b57cec5SDimitry Andric 925ffd83dbSDimitry Andric ProgramStateRef StIn, StOut; 935ffd83dbSDimitry Andric std::tie(StIn, StOut) = CM.assumeInclusiveRangeDual(state, *NV, Zero, One); 940b57cec5SDimitry Andric 955ffd83dbSDimitry Andric if (!StIn) 965ffd83dbSDimitry Andric emitReport(StOut, C); 97*81ad6265SDimitry Andric if (StIn && StOut && taint::isTainted(state, *NV)) 98*81ad6265SDimitry Andric emitReport(StOut, C, /*IsTainted=*/true); 990b57cec5SDimitry Andric } 1000b57cec5SDimitry Andric 1010b57cec5SDimitry Andric void ento::registerBoolAssignmentChecker(CheckerManager &mgr) { 1020b57cec5SDimitry Andric mgr.registerChecker<BoolAssignmentChecker>(); 1030b57cec5SDimitry Andric } 1040b57cec5SDimitry Andric 1055ffd83dbSDimitry Andric bool ento::shouldRegisterBoolAssignmentChecker(const CheckerManager &mgr) { 1060b57cec5SDimitry Andric return true; 1070b57cec5SDimitry Andric } 108