blob: a18319a62b710c9fb63edeb7dbf51d51e352a351 [file] [log] [blame]
Ted Kremenek13ed7fe2007-09-14 22:49:21 +00001//==- UninitializedValues.cpp - Find Unintialized Values --------*- C++ --*-==//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file was developed by Ted Kremenek and is distributed under
6// the University of Illinois Open Source License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
10// This file implements Uninitialized Values analysis for source-level CFGs.
11//
12//===----------------------------------------------------------------------===//
13
14#include "clang/Analysis/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 Kremenek13ed7fe2007-09-14 22:49:21 +000020
Ted Kremenekcd5860c2007-09-17 19:59:27 +000021#include "llvm/ADT/SmallPtrSet.h"
22
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000023using namespace clang;
24
Ted Kremenekcd5860c2007-09-17 19:59:27 +000025//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000026// Dataflow initialization logic.
Ted Kremenekcd5860c2007-09-17 19:59:27 +000027//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000028
29namespace {
30
Ted Kremenek11e72182007-10-01 20:33:52 +000031class RegisterDecls : public CFGRecStmtDeclVisitor<RegisterDecls> {
Ted Kremenek56d516d2007-09-17 17:14:52 +000032 UninitializedValues::AnalysisDataTy& AD;
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000033public:
Ted Kremenek11e72182007-10-01 20:33:52 +000034 RegisterDecls(UninitializedValues::AnalysisDataTy& ad) : AD(ad) {}
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000035
Ted Kremenekaead1532007-09-27 18:20:22 +000036 void VisitBlockVarDecl(BlockVarDecl* VD) { AD.Register(VD); }
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000037};
38
39} // end anonymous namespace
40
41void UninitializedValues::InitializeValues(const CFG& cfg) {
Ted Kremenek11e72182007-10-01 20:33:52 +000042 RegisterDecls R(getAnalysisData());
Ted Kremeneka90b0d12007-09-18 20:59:00 +000043 cfg.VisitBlockStmts(R);
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000044}
45
Ted Kremenekcd5860c2007-09-17 19:59:27 +000046//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000047// Transfer functions.
Ted Kremenekcd5860c2007-09-17 19:59:27 +000048//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000049
50namespace {
Ted Kremenek20ee4fb2007-09-17 18:31:23 +000051
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000052class TransferFuncs : public CFGStmtVisitor<TransferFuncs,bool> {
53 UninitializedValues::ValTy V;
Ted Kremenek56d516d2007-09-17 17:14:52 +000054 UninitializedValues::AnalysisDataTy& AD;
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000055public:
Ted Kremeneka1de8c72007-09-28 00:09:38 +000056 TransferFuncs(UninitializedValues::AnalysisDataTy& ad) : AD(ad) {
Ted Kremenek2bf55142007-09-17 20:49:30 +000057 V.resetValues(AD);
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000058 }
59
60 UninitializedValues::ValTy& getVal() { return V; }
Ted Kremenek56d516d2007-09-17 17:14:52 +000061
Ted Kremenek20ee4fb2007-09-17 18:31:23 +000062 bool VisitDeclRefExpr(DeclRefExpr* DR);
63 bool VisitBinaryOperator(BinaryOperator* B);
64 bool VisitUnaryOperator(UnaryOperator* U);
65 bool VisitStmt(Stmt* S);
66 bool VisitCallExpr(CallExpr* C);
Ted Kremenekcd5860c2007-09-17 19:59:27 +000067 bool VisitDeclStmt(DeclStmt* D);
Ted Kremeneka1de8c72007-09-28 00:09:38 +000068 bool VisitConditionalOperator(ConditionalOperator* C);
69
70 bool Visit(Stmt *S);
71 bool BlockStmt_VisitExpr(Expr* E);
Ted Kremenek20ee4fb2007-09-17 18:31:23 +000072
Ted Kremenek43a16982007-09-18 21:43:18 +000073 BlockVarDecl* FindBlockVarDecl(Stmt* S);
Ted Kremenek13ed7fe2007-09-14 22:49:21 +000074};
Ted Kremenekaead1532007-09-27 18:20:22 +000075
76static const bool Initialized = true;
77static const bool Uninitialized = false;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +000078
Ted Kremenek20ee4fb2007-09-17 18:31:23 +000079bool TransferFuncs::VisitDeclRefExpr(DeclRefExpr* DR) {
Ted Kremenek2bf55142007-09-17 20:49:30 +000080 if (BlockVarDecl* VD = dyn_cast<BlockVarDecl>(DR->getDecl())) {
Ted Kremenek43a16982007-09-18 21:43:18 +000081 if (AD.Observer) AD.Observer->ObserveDeclRefExpr(V,AD,DR,VD);
Ted Kremeneka1de8c72007-09-28 00:09:38 +000082
83 // Pseudo-hack to prevent cascade of warnings. If an accessed variable
84 // is uninitialized, then we are already going to flag a warning for
85 // this variable, which a "source" of uninitialized values.
86 // We can otherwise do a full "taint" of uninitialized values. The
87 // client has both options by toggling AD.FullUninitTaint.
88
89 return AD.FullUninitTaint ? V(VD,AD) : Initialized;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +000090 }
Ted Kremenekaead1532007-09-27 18:20:22 +000091 else return Initialized;
Ted Kremenek43a16982007-09-18 21:43:18 +000092}
93
94BlockVarDecl* TransferFuncs::FindBlockVarDecl(Stmt *S) {
Ted Kremeneka1de8c72007-09-28 00:09:38 +000095 for (;;)
Ted Kremenek43a16982007-09-18 21:43:18 +000096 if (ParenExpr* P = dyn_cast<ParenExpr>(S)) {
Ted Kremeneka1de8c72007-09-28 00:09:38 +000097 S = P->getSubExpr(); continue;
Ted Kremenek43a16982007-09-18 21:43:18 +000098 }
Ted Kremeneka1de8c72007-09-28 00:09:38 +000099 else if (DeclRefExpr* DR = dyn_cast<DeclRefExpr>(S)) {
Ted Kremenek43a16982007-09-18 21:43:18 +0000100 if (BlockVarDecl* VD = dyn_cast<BlockVarDecl>(DR->getDecl()))
101 return VD;
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000102 }
103 else return NULL;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000104}
105
106bool TransferFuncs::VisitBinaryOperator(BinaryOperator* B) {
Ted Kremenek6ce2b632007-09-28 21:08:51 +0000107 if (BlockVarDecl* VD = FindBlockVarDecl(B->getLHS()))
108 if (B->isAssignmentOp()) {
109 if (AD.FullUninitTaint) {
110 if (B->getOpcode() == BinaryOperator::Assign)
111 return V(VD,AD) = Visit(B->getRHS());
112 else // Handle +=, -=, *=, etc. We do want '&', not '&&'.
113 return V(VD,AD) = Visit(B->getLHS()) & Visit(B->getRHS());
114 }
115 else {
116 Visit(B->getLHS()); Visit(B->getRHS());
117 return Initialized;
118 }
119 }
120
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000121 return VisitStmt(B);
122}
123
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000124bool TransferFuncs::VisitDeclStmt(DeclStmt* S) {
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000125 for (ScopedDecl* D = S->getDecl(); D != NULL; D = D->getNextDeclarator())
Ted Kremenekaead1532007-09-27 18:20:22 +0000126 if (BlockVarDecl* VD = dyn_cast<BlockVarDecl>(D)) {
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000127 if (Stmt* I = VD->getInit())
128 V(VD,AD) = AD.FullUninitTaint ? V(cast<Expr>(I),AD) : Initialized;
129 else V(VD,AD) = Uninitialized;
Ted Kremenekaead1532007-09-27 18:20:22 +0000130 }
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000131
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000132 return Uninitialized; // Value is never consumed.
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000133}
134
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000135bool TransferFuncs::VisitCallExpr(CallExpr* C) {
Ted Kremenek59d18272007-09-18 21:47:41 +0000136 VisitChildren(C);
Ted Kremenekaead1532007-09-27 18:20:22 +0000137 return Initialized;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000138}
139
140bool TransferFuncs::VisitUnaryOperator(UnaryOperator* U) {
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000141 if (U->getOpcode() == UnaryOperator::AddrOf)
142 if (BlockVarDecl* VD = FindBlockVarDecl(U->getSubExpr()))
143 return V(VD,AD) = Initialized;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000144
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000145 return Visit(U->getSubExpr());
146}
147
148bool TransferFuncs::VisitConditionalOperator(ConditionalOperator* C) {
149 Visit(C->getCond());
150 return Visit(C->getLHS()) & Visit(C->getRHS()); // Yes: we want &, not &&.
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000151}
152
153bool TransferFuncs::VisitStmt(Stmt* S) {
Ted Kremenekaead1532007-09-27 18:20:22 +0000154 bool x = Initialized;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000155
156 // We don't stop at the first subexpression that is Uninitialized because
157 // evaluating some subexpressions may result in propogating "Uninitialized"
158 // or "Initialized" to variables referenced in the other subexpressions.
159 for (Stmt::child_iterator I=S->child_begin(), E=S->child_end(); I!=E; ++I)
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000160 if (*I && Visit(*I) == Uninitialized) x = Uninitialized;
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000161
162 return x;
163}
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000164
165bool TransferFuncs::Visit(Stmt *S) {
166 if (AD.isTracked(static_cast<Expr*>(S))) return V(static_cast<Expr*>(S),AD);
167 else return static_cast<CFGStmtVisitor<TransferFuncs,bool>*>(this)->Visit(S);
168}
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000169
170bool TransferFuncs::BlockStmt_VisitExpr(Expr* E) {
Ted Kremenek43a16982007-09-18 21:43:18 +0000171 assert (AD.isTracked(E));
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000172 return V(E,AD) =
173 static_cast<CFGStmtVisitor<TransferFuncs,bool>*>(this)->Visit(E);
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000174}
175
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000176} // end anonymous namespace
177
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000178//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000179// Merge operator.
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000180//
181// In our transfer functions we take the approach that any
182// combination of unintialized values, e.g. Unitialized + ___ = Unitialized.
183//
184// Merges take the opposite approach.
185//
Ted Kremeneka90b0d12007-09-18 20:59:00 +0000186// In the merge of dataflow values we prefer unsoundness, and
Ted Kremenek20ee4fb2007-09-17 18:31:23 +0000187// prefer false negatives to false positives. At merges, if a value for a
188// tracked Decl is EVER initialized in any of the predecessors we treat it as
189// initialized at the confluence point.
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000190//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000191
192namespace {
Ted Kremenekaead1532007-09-27 18:20:22 +0000193 typedef ExprDeclBitVector_Types::Union Merge;
194 typedef DataflowSolver<UninitializedValues,TransferFuncs,Merge> Solver;
195}
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000196
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000197//===----------------------------------------------------------------------===//
198// Unitialized values checker. Scan an AST and flag variable uses
199//===----------------------------------------------------------------------===//
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000200
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000201UninitializedValues_ValueTypes::ObserverTy::~ObserverTy() {}
202
203namespace {
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000204class UninitializedValuesChecker : public UninitializedValues::ObserverTy {
205 ASTContext &Ctx;
206 Diagnostic &Diags;
Ted Kremenek2bf55142007-09-17 20:49:30 +0000207 llvm::SmallPtrSet<BlockVarDecl*,10> AlreadyWarned;
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000208
209public:
210 UninitializedValuesChecker(ASTContext &ctx, Diagnostic &diags)
211 : Ctx(ctx), Diags(diags) {}
212
213 virtual void ObserveDeclRefExpr(UninitializedValues::ValTy& V,
214 UninitializedValues::AnalysisDataTy& AD,
Ted Kremenek2bf55142007-09-17 20:49:30 +0000215 DeclRefExpr* DR, BlockVarDecl* VD) {
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000216
Ted Kremenek43a16982007-09-18 21:43:18 +0000217 assert ( AD.isTracked(VD) && "Unknown VarDecl.");
218
Ted Kremenekaead1532007-09-27 18:20:22 +0000219 if (V(VD,AD) == Uninitialized)
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000220 if (AlreadyWarned.insert(VD))
221 Diags.Report(DR->getSourceRange().Begin(), diag::warn_uninit_val);
222 }
223};
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000224} // end anonymous namespace
225
Ted Kremenek2bf55142007-09-17 20:49:30 +0000226namespace clang {
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000227void CheckUninitializedValues(CFG& cfg, ASTContext &Ctx, Diagnostic &Diags,
228 bool FullUninitTaint) {
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000229
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000230 // Compute the unitialized values information.
Ted Kremenek11e72182007-10-01 20:33:52 +0000231 UninitializedValues U(cfg);
Ted Kremeneka1de8c72007-09-28 00:09:38 +0000232 U.getAnalysisData().FullUninitTaint = FullUninitTaint;
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000233 Solver S(U);
Ted Kremenekcd5860c2007-09-17 19:59:27 +0000234 S.runOnCFG(cfg);
235
236 // Scan for DeclRefExprs that use uninitialized values.
237 UninitializedValuesChecker Observer(Ctx,Diags);
238 U.getAnalysisData().Observer = &Observer;
Ted Kremenek294a7c92007-09-18 21:08:21 +0000239 S.runOnAllBlocks(cfg);
Ted Kremenek13ed7fe2007-09-14 22:49:21 +0000240}
Ted Kremenek294a7c92007-09-18 21:08:21 +0000241} // end namespace clang