blob: f3c2ffc58662e74993fe5f9f38c723230496d18b [file] [log] [blame]
Zhongxing Xuc6123a12009-11-24 08:24:26 +00001//=== UndefResultChecker.cpp ------------------------------------*- C++ -*-===//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
Ted Kremenek3a0678e2015-09-08 03:50:52 +000010// This defines UndefResultChecker, a builtin check in ExprEngine that
Zhongxing Xuc6123a12009-11-24 08:24:26 +000011// performs checks for undefined results of non-assignment binary operators.
12//
13//===----------------------------------------------------------------------===//
14
Argyrios Kyrtzidisd4d3cee2011-02-28 01:27:22 +000015#include "ClangSACheckers.h"
Chandler Carruth3a022472012-12-04 09:13:33 +000016#include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
Argyrios Kyrtzidis6a5674f2011-03-01 01:16:21 +000017#include "clang/StaticAnalyzer/Core/Checker.h"
Argyrios Kyrtzidisd4d3cee2011-02-28 01:27:22 +000018#include "clang/StaticAnalyzer/Core/CheckerManager.h"
19#include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
Ted Kremenekf8cbac42011-02-10 01:03:03 +000020#include "clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h"
Benjamin Kramer49038022012-02-04 13:45:25 +000021#include "llvm/ADT/SmallString.h"
Benjamin Kramer444a1302012-12-01 17:12:56 +000022#include "llvm/Support/raw_ostream.h"
Zhongxing Xuc6123a12009-11-24 08:24:26 +000023
24using namespace clang;
Ted Kremenek98857c92010-12-23 07:20:52 +000025using namespace ento;
Zhongxing Xuc6123a12009-11-24 08:24:26 +000026
27namespace {
Ted Kremenek3a0678e2015-09-08 03:50:52 +000028class UndefResultChecker
Argyrios Kyrtzidis6a5674f2011-03-01 01:16:21 +000029 : public Checker< check::PostStmt<BinaryOperator> > {
Zhongxing Xuc6123a12009-11-24 08:24:26 +000030
Ahmed Charlesb8984322014-03-07 20:03:18 +000031 mutable std::unique_ptr<BugType> BT;
32
Zhongxing Xuc6123a12009-11-24 08:24:26 +000033public:
Argyrios Kyrtzidisd4d3cee2011-02-28 01:27:22 +000034 void checkPostStmt(const BinaryOperator *B, CheckerContext &C) const;
Zhongxing Xuc6123a12009-11-24 08:24:26 +000035};
36} // end anonymous namespace
37
Daniel Marjamakie97838f2017-02-27 10:44:24 +000038static bool isArrayIndexOutOfBounds(CheckerContext &C, const Expr *Ex) {
39 ProgramStateRef state = C.getState();
40 const LocationContext *LCtx = C.getLocationContext();
41
42 if (!isa<ArraySubscriptExpr>(Ex))
43 return false;
44
45 SVal Loc = state->getSVal(Ex, LCtx);
46 if (!Loc.isValid())
47 return false;
48
49 const MemRegion *MR = Loc.castAs<loc::MemRegionVal>().getRegion();
50 const ElementRegion *ER = dyn_cast<ElementRegion>(MR);
51 if (!ER)
52 return false;
53
54 DefinedOrUnknownSVal Idx = ER->getIndex().castAs<DefinedOrUnknownSVal>();
55 DefinedOrUnknownSVal NumElements = C.getStoreManager().getSizeInElements(
56 state, ER->getSuperRegion(), ER->getValueType());
57 ProgramStateRef StInBound = state->assumeInBound(Idx, NumElements, true);
58 ProgramStateRef StOutBound = state->assumeInBound(Idx, NumElements, false);
59 return StOutBound && !StInBound;
60}
61
Argyrios Kyrtzidisd4d3cee2011-02-28 01:27:22 +000062void UndefResultChecker::checkPostStmt(const BinaryOperator *B,
63 CheckerContext &C) const {
Ted Kremenek49b1e382012-01-26 21:29:00 +000064 ProgramStateRef state = C.getState();
Ted Kremenek632e3b72012-01-06 22:09:28 +000065 const LocationContext *LCtx = C.getLocationContext();
66 if (state->getSVal(B, LCtx).isUndef()) {
Anna Zaks03256462013-06-18 23:16:15 +000067
68 // Do not report assignments of uninitialized values inside swap functions.
69 // This should allow to swap partially uninitialized structs
70 // (radar://14129997)
71 if (const FunctionDecl *EnclosingFunctionDecl =
72 dyn_cast<FunctionDecl>(C.getStackFrame()->getDecl()))
73 if (C.getCalleeName(EnclosingFunctionDecl) == "swap")
74 return;
75
Zhongxing Xuc6123a12009-11-24 08:24:26 +000076 // Generate an error node.
Devin Coughline39bd402015-09-16 22:03:05 +000077 ExplodedNode *N = C.generateErrorNode();
Zhongxing Xuc6123a12009-11-24 08:24:26 +000078 if (!N)
79 return;
Ted Kremenek3a0678e2015-09-08 03:50:52 +000080
Zhongxing Xuc6123a12009-11-24 08:24:26 +000081 if (!BT)
Alexander Kornienko4aca9b12014-02-11 21:49:21 +000082 BT.reset(
83 new BuiltinBug(this, "Result of operation is garbage or undefined"));
Zhongxing Xuc6123a12009-11-24 08:24:26 +000084
Dylan Noblesmith2c1dd272012-02-05 02:13:05 +000085 SmallString<256> sbuf;
Zhongxing Xuc6123a12009-11-24 08:24:26 +000086 llvm::raw_svector_ostream OS(sbuf);
Craig Topper0dbb7832014-05-27 02:45:47 +000087 const Expr *Ex = nullptr;
Zhongxing Xuc6123a12009-11-24 08:24:26 +000088 bool isLeft = true;
Ted Kremenek3a0678e2015-09-08 03:50:52 +000089
Ted Kremenek632e3b72012-01-06 22:09:28 +000090 if (state->getSVal(B->getLHS(), LCtx).isUndef()) {
Zhongxing Xuc6123a12009-11-24 08:24:26 +000091 Ex = B->getLHS()->IgnoreParenCasts();
92 isLeft = true;
93 }
Ted Kremenek632e3b72012-01-06 22:09:28 +000094 else if (state->getSVal(B->getRHS(), LCtx).isUndef()) {
Zhongxing Xuc6123a12009-11-24 08:24:26 +000095 Ex = B->getRHS()->IgnoreParenCasts();
96 isLeft = false;
97 }
Ted Kremenek3a0678e2015-09-08 03:50:52 +000098
Zhongxing Xuc6123a12009-11-24 08:24:26 +000099 if (Ex) {
100 OS << "The " << (isLeft ? "left" : "right")
101 << " operand of '"
102 << BinaryOperator::getOpcodeStr(B->getOpcode())
103 << "' is a garbage value";
Daniel Marjamakie97838f2017-02-27 10:44:24 +0000104 if (isArrayIndexOutOfBounds(C, Ex))
105 OS << " due to array index out of bounds";
Ted Kremenek3a0678e2015-09-08 03:50:52 +0000106 }
Zhongxing Xuc6123a12009-11-24 08:24:26 +0000107 else {
108 // Neither operand was undefined, but the result is undefined.
109 OS << "The result of the '"
110 << BinaryOperator::getOpcodeStr(B->getOpcode())
111 << "' expression is undefined";
112 }
Aaron Ballman8d3a7a52015-06-23 13:15:32 +0000113 auto report = llvm::make_unique<BugReport>(*BT, OS.str(), N);
Ted Kremenek33e88a72009-11-29 06:37:44 +0000114 if (Ex) {
115 report->addRange(Ex->getSourceRange());
Jordan Rosea0f7d352012-08-28 00:50:51 +0000116 bugreporter::trackNullOrUndefValue(N, Ex, *report);
Ted Kremenek33e88a72009-11-29 06:37:44 +0000117 }
Zhongxing Xuc6123a12009-11-24 08:24:26 +0000118 else
Jordan Rosea0f7d352012-08-28 00:50:51 +0000119 bugreporter::trackNullOrUndefValue(N, B, *report);
Ted Kremenek3a0678e2015-09-08 03:50:52 +0000120
Aaron Ballman8d3a7a52015-06-23 13:15:32 +0000121 C.emitReport(std::move(report));
Zhongxing Xuc6123a12009-11-24 08:24:26 +0000122 }
123}
Argyrios Kyrtzidisd4d3cee2011-02-28 01:27:22 +0000124
125void ento::registerUndefResultChecker(CheckerManager &mgr) {
126 mgr.registerChecker<UndefResultChecker>();
127}