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