1d99bd55aSTed Kremenek //===--- UndefinedAssignmentChecker.h ---------------------------*- C++ -*--==//
2d99bd55aSTed 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
6d99bd55aSTed Kremenek //
7d99bd55aSTed Kremenek //===----------------------------------------------------------------------===//
8d99bd55aSTed Kremenek //
9098874a2SArgyrios Kyrtzidis // This defines UndefinedAssignmentChecker, a builtin check in ExprEngine that
10d99bd55aSTed Kremenek // checks for assigning undefined values.
11d99bd55aSTed Kremenek //
12d99bd55aSTed Kremenek //===----------------------------------------------------------------------===//
13d99bd55aSTed Kremenek
1476a21502SKristof Umann #include "clang/StaticAnalyzer/Checkers/BuiltinCheckerRegistration.h"
153a02247dSChandler Carruth #include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
166a5674ffSArgyrios Kyrtzidis #include "clang/StaticAnalyzer/Core/Checker.h"
17098874a2SArgyrios Kyrtzidis #include "clang/StaticAnalyzer/Core/CheckerManager.h"
18098874a2SArgyrios Kyrtzidis #include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
19d99bd55aSTed Kremenek
20d99bd55aSTed Kremenek using namespace clang;
21d99bd55aSTed Kremenek using namespace ento;
22d99bd55aSTed Kremenek
23d99bd55aSTed Kremenek namespace {
24d99bd55aSTed Kremenek class UndefinedAssignmentChecker
256a5674ffSArgyrios Kyrtzidis : public Checker<check::Bind> {
26b8984329SAhmed Charles mutable std::unique_ptr<BugType> BT;
27098874a2SArgyrios Kyrtzidis
28d99bd55aSTed Kremenek public:
293e0f415dSAnna Zaks void checkBind(SVal location, SVal val, const Stmt *S,
303e0f415dSAnna Zaks CheckerContext &C) const;
31d99bd55aSTed Kremenek };
32d99bd55aSTed Kremenek }
33d99bd55aSTed Kremenek
checkBind(SVal location,SVal val,const Stmt * StoreE,CheckerContext & C) const34098874a2SArgyrios Kyrtzidis void UndefinedAssignmentChecker::checkBind(SVal location, SVal val,
353e0f415dSAnna Zaks const Stmt *StoreE,
36098874a2SArgyrios Kyrtzidis CheckerContext &C) const {
37d99bd55aSTed Kremenek if (!val.isUndef())
38d99bd55aSTed Kremenek return;
39d99bd55aSTed Kremenek
4003256467SAnna Zaks // Do not report assignments of uninitialized values inside swap functions.
4103256467SAnna Zaks // This should allow to swap partially uninitialized structs
4203256467SAnna Zaks // (radar://14129997)
4303256467SAnna Zaks if (const FunctionDecl *EnclosingFunctionDecl =
4403256467SAnna Zaks dyn_cast<FunctionDecl>(C.getStackFrame()->getDecl()))
4503256467SAnna Zaks if (C.getCalleeName(EnclosingFunctionDecl) == "swap")
4603256467SAnna Zaks return;
4703256467SAnna Zaks
48e39bd407SDevin Coughlin ExplodedNode *N = C.generateErrorNode();
49d99bd55aSTed Kremenek
50d99bd55aSTed Kremenek if (!N)
51d99bd55aSTed Kremenek return;
52d99bd55aSTed Kremenek
53f119bf99SArtem Dergachev static const char *const DefaultMsg =
54f119bf99SArtem Dergachev "Assigned value is garbage or undefined";
55d99bd55aSTed Kremenek if (!BT)
56f119bf99SArtem Dergachev BT.reset(new BuiltinBug(this, DefaultMsg));
57d99bd55aSTed Kremenek
58d99bd55aSTed Kremenek // Generate a report for this bug.
59f119bf99SArtem Dergachev llvm::SmallString<128> Str;
60f119bf99SArtem Dergachev llvm::raw_svector_ostream OS(Str);
61f119bf99SArtem Dergachev
620dbb783cSCraig Topper const Expr *ex = nullptr;
63d99bd55aSTed Kremenek
64d99bd55aSTed Kremenek while (StoreE) {
6588b56caaSRoman Lebedev if (const UnaryOperator *U = dyn_cast<UnaryOperator>(StoreE)) {
66f119bf99SArtem Dergachev OS << "The expression is an uninitialized value. "
6788b56caaSRoman Lebedev "The computed value will also be garbage";
6888b56caaSRoman Lebedev
6988b56caaSRoman Lebedev ex = U->getSubExpr();
7088b56caaSRoman Lebedev break;
7188b56caaSRoman Lebedev }
7288b56caaSRoman Lebedev
73d99bd55aSTed Kremenek if (const BinaryOperator *B = dyn_cast<BinaryOperator>(StoreE)) {
74d99bd55aSTed Kremenek if (B->isCompoundAssignmentOp()) {
75d703ec94SGeorge Karpenkov if (C.getSVal(B->getLHS()).isUndef()) {
76f119bf99SArtem Dergachev OS << "The left expression of the compound assignment is an "
77d99bd55aSTed Kremenek "uninitialized value. The computed value will also be garbage";
78d99bd55aSTed Kremenek ex = B->getLHS();
79d99bd55aSTed Kremenek break;
80d99bd55aSTed Kremenek }
81d99bd55aSTed Kremenek }
82d99bd55aSTed Kremenek
83d99bd55aSTed Kremenek ex = B->getRHS();
84d99bd55aSTed Kremenek break;
85d99bd55aSTed Kremenek }
86d99bd55aSTed Kremenek
87d99bd55aSTed Kremenek if (const DeclStmt *DS = dyn_cast<DeclStmt>(StoreE)) {
88630f7dafSArtem Dergachev const VarDecl *VD = cast<VarDecl>(DS->getSingleDecl());
89d99bd55aSTed Kremenek ex = VD->getInit();
90d99bd55aSTed Kremenek }
91d99bd55aSTed Kremenek
92f119bf99SArtem Dergachev if (const auto *CD =
93f119bf99SArtem Dergachev dyn_cast<CXXConstructorDecl>(C.getStackFrame()->getDecl())) {
94f119bf99SArtem Dergachev if (CD->isImplicit()) {
95f119bf99SArtem Dergachev for (auto I : CD->inits()) {
96f119bf99SArtem Dergachev if (I->getInit()->IgnoreImpCasts() == StoreE) {
97f119bf99SArtem Dergachev OS << "Value assigned to field '" << I->getMember()->getName()
98f119bf99SArtem Dergachev << "' in implicit constructor is garbage or undefined";
99f119bf99SArtem Dergachev break;
100f119bf99SArtem Dergachev }
101f119bf99SArtem Dergachev }
102f119bf99SArtem Dergachev }
103f119bf99SArtem Dergachev }
104f119bf99SArtem Dergachev
105d99bd55aSTed Kremenek break;
106d99bd55aSTed Kremenek }
107d99bd55aSTed Kremenek
108f119bf99SArtem Dergachev if (OS.str().empty())
109f119bf99SArtem Dergachev OS << DefaultMsg;
110f119bf99SArtem Dergachev
1112f169e7cSArtem Dergachev auto R = std::make_unique<PathSensitiveBugReport>(*BT, OS.str(), N);
112d99bd55aSTed Kremenek if (ex) {
113d99bd55aSTed Kremenek R->addRange(ex->getSourceRange());
114b2cf0063SGeorge Karpenkov bugreporter::trackExpressionValue(N, ex, *R);
115d99bd55aSTed Kremenek }
1168d3a7a56SAaron Ballman C.emitReport(std::move(R));
117d99bd55aSTed Kremenek }
118d99bd55aSTed Kremenek
registerUndefinedAssignmentChecker(CheckerManager & mgr)119098874a2SArgyrios Kyrtzidis void ento::registerUndefinedAssignmentChecker(CheckerManager &mgr) {
120098874a2SArgyrios Kyrtzidis mgr.registerChecker<UndefinedAssignmentChecker>();
121098874a2SArgyrios Kyrtzidis }
122058a7a45SKristof Umann
shouldRegisterUndefinedAssignmentChecker(const CheckerManager & mgr)123*bda3dd0dSKirstóf Umann bool ento::shouldRegisterUndefinedAssignmentChecker(const CheckerManager &mgr) {
124058a7a45SKristof Umann return true;
125058a7a45SKristof Umann }
126