blob: 25a5ecb48370133e4496bdf7702c62c5a277add1 [file] [log] [blame]
Ted Kremenek13ed7fe2007-09-14 22:49:21 +00001//==- UninitializedValues.cpp - Find Unintialized Values --------*- C++ --*-==//
2//
3// The LLVM Compiler Infrastructure
4//
Chris Lattner0bc735f2007-12-29 19:59:25 +00005// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
Ted Kremenek13ed7fe2007-09-14 22:49:21 +00007//
8//===----------------------------------------------------------------------===//
9//
10// This file implements Uninitialized Values analysis for source-level CFGs.
11//
12//===----------------------------------------------------------------------===//
13
Ted Kremenekcf6e41b2007-12-21 21:42:19 +000014#include "clang/Analysis/Analyses/UninitializedValues.h"
Ted Kremenek11de5cb2007-09-20 21:42:55 +000015#include "clang/Analysis/Visitors/CFGRecStmtDeclVisitor.h"
Ted Kremenekcd5860c2007-09-17 19:59:27 +000016#include "clang/Analysis/LocalCheckers.h"
17#include "clang/Basic/Diagnostic.h"
18#include "clang/AST/ASTContext.h"
Ted Kremenek1de632b2007-09-25 21:00:24 +000019#include "clang/Analysis/FlowSensitive/DataflowSolver.h"
Ted Kremenekc2b51d82008-01-08 18:19:08 +000020#include "llvm/Support/Compiler.h"
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000021
Ted Kremenekcd5860c2007-09-17 19:59:27 +000022#include "llvm/ADT/SmallPtrSet.h"
23
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000024using namespace clang;
25
Ted Kremenekcd5860c2007-09-17 19:59:27 +000026//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000027// Dataflow initialization logic.
Ted Kremenekcd5860c2007-09-17 19:59:27 +000028//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000029
30namespace {
31
Ted Kremenekc2b51d82008-01-08 18:19:08 +000032class VISIBILITY_HIDDEN RegisterDecls
33 : public CFGRecStmtDeclVisitor<RegisterDecls> {
34
Ted Kremenek56d516d2007-09-17 17:14:52 +000035 UninitializedValues::AnalysisDataTy& AD;
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000036public:
Ted Kremenek11e72182007-10-01 20:33:52 +000037 RegisterDecls(UninitializedValues::AnalysisDataTy& ad) : AD(ad) {}
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000038
Ted Kremenekaead1532007-09-27 18:20:22 +000039 void VisitBlockVarDecl(BlockVarDecl* VD) { AD.Register(VD); }
Ted Kremenek9f9141c2007-11-20 03:01:58 +000040 CFG& getCFG() { return AD.getCFG(); }
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000041};
42
43} // end anonymous namespace
44
45void UninitializedValues::InitializeValues(const CFG& cfg) {
Ted Kremenek11e72182007-10-01 20:33:52 +000046 RegisterDecls R(getAnalysisData());
Ted Kremeneka90b0d12007-09-18 20:59:00 +000047 cfg.VisitBlockStmts(R);
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000048}
49
Ted Kremenekcd5860c2007-09-17 19:59:27 +000050//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000051// Transfer functions.
Ted Kremenekcd5860c2007-09-17 19:59:27 +000052//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000053
54namespace {
Ted Kremenekc2b51d82008-01-08 18:19:08 +000055class VISIBILITY_HIDDEN TransferFuncs
56 : public CFGStmtVisitor<TransferFuncs,bool> {
57
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000058 UninitializedValues::ValTy V;
Ted Kremenek56d516d2007-09-17 17:14:52 +000059 UninitializedValues::AnalysisDataTy& AD;
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000060public:
Ted Kremeneka1de8c72007-09-28 00:09:38 +000061 TransferFuncs(UninitializedValues::AnalysisDataTy& ad) : AD(ad) {
Ted Kremenek2bf55142007-09-17 20:49:30 +000062 V.resetValues(AD);
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000063 }
64
65 UninitializedValues::ValTy& getVal() { return V; }
Ted Kremenek9f9141c2007-11-20 03:01:58 +000066 CFG& getCFG() { return AD.getCFG(); }
Ted Kremenek56d516d2007-09-17 17:14:52 +000067
Ted Kremenek20ee4fb2007-09-17 18:31:23 +000068 bool VisitDeclRefExpr(DeclRefExpr* DR);
69 bool VisitBinaryOperator(BinaryOperator* B);
70 bool VisitUnaryOperator(UnaryOperator* U);
71 bool VisitStmt(Stmt* S);
72 bool VisitCallExpr(CallExpr* C);
Ted Kremenekcd5860c2007-09-17 19:59:27 +000073 bool VisitDeclStmt(DeclStmt* D);
Ted Kremeneka1de8c72007-09-28 00:09:38 +000074 bool VisitConditionalOperator(ConditionalOperator* C);
75
76 bool Visit(Stmt *S);
77 bool BlockStmt_VisitExpr(Expr* E);
Ted Kremenek20ee4fb2007-09-17 18:31:23 +000078
Ted Kremenek43a16982007-09-18 21:43:18 +000079 BlockVarDecl* FindBlockVarDecl(Stmt* S);
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000080};
Ted Kremenekaead1532007-09-27 18:20:22 +000081
82static const bool Initialized = true;
83static const bool Uninitialized = false;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +000084
Ted Kremenek20ee4fb2007-09-17 18:31:23 +000085bool TransferFuncs::VisitDeclRefExpr(DeclRefExpr* DR) {
Ted Kremenek2bf55142007-09-17 20:49:30 +000086 if (BlockVarDecl* VD = dyn_cast<BlockVarDecl>(DR->getDecl())) {
Ted Kremenek43a16982007-09-18 21:43:18 +000087 if (AD.Observer) AD.Observer->ObserveDeclRefExpr(V,AD,DR,VD);
Ted Kremeneka1de8c72007-09-28 00:09:38 +000088
89 // Pseudo-hack to prevent cascade of warnings. If an accessed variable
90 // is uninitialized, then we are already going to flag a warning for
91 // this variable, which a "source" of uninitialized values.
92 // We can otherwise do a full "taint" of uninitialized values. The
93 // client has both options by toggling AD.FullUninitTaint.
94
95 return AD.FullUninitTaint ? V(VD,AD) : Initialized;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +000096 }
Ted Kremenekaead1532007-09-27 18:20:22 +000097 else return Initialized;
Ted Kremenek43a16982007-09-18 21:43:18 +000098}
99
100BlockVarDecl* TransferFuncs::FindBlockVarDecl(Stmt *S) {
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000101 for (;;)
Ted Kremenek43a16982007-09-18 21:43:18 +0000102 if (ParenExpr* P = dyn_cast<ParenExpr>(S)) {
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000103 S = P->getSubExpr(); continue;
Ted Kremenek43a16982007-09-18 21:43:18 +0000104 }
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000105 else if (DeclRefExpr* DR = dyn_cast<DeclRefExpr>(S)) {
Ted Kremenek43a16982007-09-18 21:43:18 +0000106 if (BlockVarDecl* VD = dyn_cast<BlockVarDecl>(DR->getDecl()))
107 return VD;
Ted Kremenekff7c5382007-11-24 20:07:36 +0000108 else
109 return NULL;
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000110 }
111 else return NULL;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000112}
113
114bool TransferFuncs::VisitBinaryOperator(BinaryOperator* B) {
Ted Kremenek6ce2b632007-09-28 21:08:51 +0000115 if (BlockVarDecl* VD = FindBlockVarDecl(B->getLHS()))
116 if (B->isAssignmentOp()) {
Ted Kremenekff7c5382007-11-24 20:07:36 +0000117 if (B->getOpcode() == BinaryOperator::Assign)
118 return V(VD,AD) = Visit(B->getRHS());
119 else // Handle +=, -=, *=, etc. We do want '&', not '&&'.
120 return V(VD,AD) = Visit(B->getLHS()) & Visit(B->getRHS());
Ted Kremenek6ce2b632007-09-28 21:08:51 +0000121 }
122
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000123 return VisitStmt(B);
124}
125
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000126bool TransferFuncs::VisitDeclStmt(DeclStmt* S) {
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000127 for (ScopedDecl* D = S->getDecl(); D != NULL; D = D->getNextDeclarator())
Ted Kremenekaead1532007-09-27 18:20:22 +0000128 if (BlockVarDecl* VD = dyn_cast<BlockVarDecl>(D)) {
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000129 if (Stmt* I = VD->getInit())
130 V(VD,AD) = AD.FullUninitTaint ? V(cast<Expr>(I),AD) : Initialized;
Ted Kremenek81a56ec2007-12-13 05:14:22 +0000131 else {
132 // Special case for declarations of array types. For things like:
133 //
134 // char x[10];
135 //
136 // we should treat "x" as being initialized, because the variable
137 // "x" really refers to the memory block. Clearly x[1] is
138 // uninitialized, but expressions like "(char *) x" really do refer to
139 // an initialized value. This simple dataflow analysis does not reason
140 // about the contents of arrays, although it could be potentially
141 // extended to do so if the array were of constant size.
142 if (VD->getType()->isArrayType())
143 V(VD,AD) = Initialized;
144 else
145 V(VD,AD) = Uninitialized;
146 }
Ted Kremenekaead1532007-09-27 18:20:22 +0000147 }
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000148
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000149 return Uninitialized; // Value is never consumed.
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000150}
151
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000152bool TransferFuncs::VisitCallExpr(CallExpr* C) {
Ted Kremenek59d18272007-09-18 21:47:41 +0000153 VisitChildren(C);
Ted Kremenekaead1532007-09-27 18:20:22 +0000154 return Initialized;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000155}
156
157bool TransferFuncs::VisitUnaryOperator(UnaryOperator* U) {
Ted Kremenek8d9ebae2007-12-13 04:47:15 +0000158 switch (U->getOpcode()) {
159 case UnaryOperator::AddrOf:
160 if (BlockVarDecl* VD = FindBlockVarDecl(U->getSubExpr()))
161 return V(VD,AD) = Initialized;
162
163 break;
164
165 case UnaryOperator::SizeOf:
166 return Initialized;
167
168 default:
169 break;
170 }
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000171
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000172 return Visit(U->getSubExpr());
173}
174
175bool TransferFuncs::VisitConditionalOperator(ConditionalOperator* C) {
176 Visit(C->getCond());
Anders Carlsson39073232007-11-30 19:04:31 +0000177
178 bool rhsResult = Visit(C->getRHS());
179 // Handle the GNU extension for missing LHS.
180 if (Expr *lhs = C->getLHS())
181 return Visit(lhs) & rhsResult; // Yes: we want &, not &&.
182 else
183 return rhsResult;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000184}
185
186bool TransferFuncs::VisitStmt(Stmt* S) {
Ted Kremenekaead1532007-09-27 18:20:22 +0000187 bool x = Initialized;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000188
189 // We don't stop at the first subexpression that is Uninitialized because
190 // evaluating some subexpressions may result in propogating "Uninitialized"
191 // or "Initialized" to variables referenced in the other subexpressions.
192 for (Stmt::child_iterator I=S->child_begin(), E=S->child_end(); I!=E; ++I)
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000193 if (*I && Visit(*I) == Uninitialized) x = Uninitialized;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000194
195 return x;
196}
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000197
198bool TransferFuncs::Visit(Stmt *S) {
199 if (AD.isTracked(static_cast<Expr*>(S))) return V(static_cast<Expr*>(S),AD);
200 else return static_cast<CFGStmtVisitor<TransferFuncs,bool>*>(this)->Visit(S);
201}
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000202
203bool TransferFuncs::BlockStmt_VisitExpr(Expr* E) {
Ted Kremenek33d4aab2008-01-26 00:03:27 +0000204 bool x = static_cast<CFGStmtVisitor<TransferFuncs,bool>*>(this)->Visit(E);
205 if (AD.isTracked(E)) V(E,AD) = x;
206 return x;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000207}
208
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000209} // end anonymous namespace
210
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000211//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000212// Merge operator.
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000213//
214// In our transfer functions we take the approach that any
215// combination of unintialized values, e.g. Unitialized + ___ = Unitialized.
216//
217// Merges take the opposite approach.
218//
Ted Kremeneka90b0d12007-09-18 20:59:00 +0000219// In the merge of dataflow values we prefer unsoundness, and
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000220// prefer false negatives to false positives. At merges, if a value for a
221// tracked Decl is EVER initialized in any of the predecessors we treat it as
222// initialized at the confluence point.
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000223//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000224
225namespace {
Ted Kremenekaead1532007-09-27 18:20:22 +0000226 typedef ExprDeclBitVector_Types::Union Merge;
227 typedef DataflowSolver<UninitializedValues,TransferFuncs,Merge> Solver;
228}
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000229
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000230//===----------------------------------------------------------------------===//
231// Unitialized values checker. Scan an AST and flag variable uses
232//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000233
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000234UninitializedValues_ValueTypes::ObserverTy::~ObserverTy() {}
235
236namespace {
Ted Kremenekc2b51d82008-01-08 18:19:08 +0000237class VISIBILITY_HIDDEN UninitializedValuesChecker
238 : public UninitializedValues::ObserverTy {
239
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000240 ASTContext &Ctx;
241 Diagnostic &Diags;
Ted Kremenek2bf55142007-09-17 20:49:30 +0000242 llvm::SmallPtrSet<BlockVarDecl*,10> AlreadyWarned;
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000243
244public:
245 UninitializedValuesChecker(ASTContext &ctx, Diagnostic &diags)
246 : Ctx(ctx), Diags(diags) {}
247
248 virtual void ObserveDeclRefExpr(UninitializedValues::ValTy& V,
249 UninitializedValues::AnalysisDataTy& AD,
Ted Kremenek2bf55142007-09-17 20:49:30 +0000250 DeclRefExpr* DR, BlockVarDecl* VD) {
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000251
Ted Kremenek43a16982007-09-18 21:43:18 +0000252 assert ( AD.isTracked(VD) && "Unknown VarDecl.");
253
Ted Kremenekaead1532007-09-27 18:20:22 +0000254 if (V(VD,AD) == Uninitialized)
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000255 if (AlreadyWarned.insert(VD))
Ted Kremenek9c728dc2007-12-12 22:39:36 +0000256 Diags.Report(Ctx.getFullLoc(DR->getSourceRange().getBegin()),
257 diag::warn_uninit_val);
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000258 }
259};
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000260} // end anonymous namespace
261
Ted Kremenek2bf55142007-09-17 20:49:30 +0000262namespace clang {
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000263void CheckUninitializedValues(CFG& cfg, ASTContext &Ctx, Diagnostic &Diags,
264 bool FullUninitTaint) {
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000265
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000266 // Compute the unitialized values information.
Ted Kremenek11e72182007-10-01 20:33:52 +0000267 UninitializedValues U(cfg);
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000268 U.getAnalysisData().FullUninitTaint = FullUninitTaint;
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000269 Solver S(U);
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000270 S.runOnCFG(cfg);
271
272 // Scan for DeclRefExprs that use uninitialized values.
273 UninitializedValuesChecker Observer(Ctx,Diags);
274 U.getAnalysisData().Observer = &Observer;
Ted Kremenek294a7c92007-09-18 21:08:21 +0000275 S.runOnAllBlocks(cfg);
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000276}
Ted Kremenek294a7c92007-09-18 21:08:21 +0000277} // end namespace clang