blob: 67f0f6740ec46d5931d7c9ec2e7fbd6a1515eea1 [file] [log] [blame]
Ted Kremenek6f342132011-03-15 03:17:07 +00001//==- UninitializedValues.cpp - Find Uninitialized Values -------*- C++ --*-==//
Ted Kremenek610068c2011-01-15 02:58:47 +00002//
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//
10// This file implements uninitialized values analysis for source-level CFGs.
11//
12//===----------------------------------------------------------------------===//
13
Ted Kremenek13bd4232011-01-20 17:37:17 +000014#include <utility>
Ted Kremenek610068c2011-01-15 02:58:47 +000015#include "llvm/ADT/Optional.h"
16#include "llvm/ADT/SmallVector.h"
Argyrios Kyrtzidis049f6d02011-05-31 03:56:09 +000017#include "llvm/ADT/PackedVector.h"
Ted Kremenek610068c2011-01-15 02:58:47 +000018#include "llvm/ADT/DenseMap.h"
19#include "clang/AST/Decl.h"
20#include "clang/Analysis/CFG.h"
Ted Kremeneka8c17a52011-01-25 19:13:48 +000021#include "clang/Analysis/AnalysisContext.h"
Ted Kremenek610068c2011-01-15 02:58:47 +000022#include "clang/Analysis/Visitors/CFGRecStmtDeclVisitor.h"
Ted Kremenek6f342132011-03-15 03:17:07 +000023#include "clang/Analysis/Analyses/UninitializedValues.h"
Ted Kremenekc21fed32011-01-18 21:18:58 +000024#include "clang/Analysis/Support/SaveAndRestore.h"
Ted Kremenek610068c2011-01-15 02:58:47 +000025
26using namespace clang;
27
Ted Kremenek40900ee2011-01-27 02:29:34 +000028static bool isTrackedVar(const VarDecl *vd, const DeclContext *dc) {
Ted Kremenek1cbc3152011-03-17 03:06:11 +000029 if (vd->isLocalVarDecl() && !vd->hasGlobalStorage() &&
Ted Kremeneka21612f2011-04-07 20:02:56 +000030 !vd->isExceptionVariable() &&
Ted Kremenek1cbc3152011-03-17 03:06:11 +000031 vd->getDeclContext() == dc) {
32 QualType ty = vd->getType();
33 return ty->isScalarType() || ty->isVectorType();
34 }
35 return false;
Ted Kremenekc104e532011-01-18 04:53:25 +000036}
37
Ted Kremenek610068c2011-01-15 02:58:47 +000038//------------------------------------------------------------------------====//
Ted Kremenek136f8f22011-03-15 04:57:27 +000039// DeclToIndex: a mapping from Decls we track to value indices.
Ted Kremenek610068c2011-01-15 02:58:47 +000040//====------------------------------------------------------------------------//
41
42namespace {
Ted Kremenek136f8f22011-03-15 04:57:27 +000043class DeclToIndex {
Ted Kremenek610068c2011-01-15 02:58:47 +000044 llvm::DenseMap<const VarDecl *, unsigned> map;
45public:
Ted Kremenek136f8f22011-03-15 04:57:27 +000046 DeclToIndex() {}
Ted Kremenek610068c2011-01-15 02:58:47 +000047
48 /// Compute the actual mapping from declarations to bits.
49 void computeMap(const DeclContext &dc);
50
51 /// Return the number of declarations in the map.
52 unsigned size() const { return map.size(); }
53
54 /// Returns the bit vector index for a given declaration.
Ted Kremenekb831c672011-03-29 01:40:00 +000055 llvm::Optional<unsigned> getValueIndex(const VarDecl *d) const;
Ted Kremenek610068c2011-01-15 02:58:47 +000056};
57}
58
Ted Kremenek136f8f22011-03-15 04:57:27 +000059void DeclToIndex::computeMap(const DeclContext &dc) {
Ted Kremenek610068c2011-01-15 02:58:47 +000060 unsigned count = 0;
61 DeclContext::specific_decl_iterator<VarDecl> I(dc.decls_begin()),
62 E(dc.decls_end());
63 for ( ; I != E; ++I) {
64 const VarDecl *vd = *I;
Ted Kremenek40900ee2011-01-27 02:29:34 +000065 if (isTrackedVar(vd, &dc))
Ted Kremenek610068c2011-01-15 02:58:47 +000066 map[vd] = count++;
67 }
68}
69
Ted Kremenekb831c672011-03-29 01:40:00 +000070llvm::Optional<unsigned> DeclToIndex::getValueIndex(const VarDecl *d) const {
71 llvm::DenseMap<const VarDecl *, unsigned>::const_iterator I = map.find(d);
Ted Kremenek610068c2011-01-15 02:58:47 +000072 if (I == map.end())
73 return llvm::Optional<unsigned>();
74 return I->second;
75}
76
77//------------------------------------------------------------------------====//
78// CFGBlockValues: dataflow values for CFG blocks.
79//====------------------------------------------------------------------------//
80
Ted Kremenekf7bafc72011-03-15 04:57:38 +000081// These values are defined in such a way that a merge can be done using
82// a bitwise OR.
83enum Value { Unknown = 0x0, /* 00 */
84 Initialized = 0x1, /* 01 */
85 Uninitialized = 0x2, /* 10 */
86 MayUninitialized = 0x3 /* 11 */ };
87
88static bool isUninitialized(const Value v) {
89 return v >= Uninitialized;
90}
91static bool isAlwaysUninit(const Value v) {
92 return v == Uninitialized;
93}
Ted Kremenekafb10c42011-03-15 04:57:29 +000094
Benjamin Kramerda57f3e2011-03-26 12:38:21 +000095namespace {
Ted Kremenek496398d2011-03-15 04:57:32 +000096
Argyrios Kyrtzidis049f6d02011-05-31 03:56:09 +000097typedef llvm::PackedVector<Value, 2> ValueVector;
Ted Kremenek136f8f22011-03-15 04:57:27 +000098typedef std::pair<ValueVector *, ValueVector *> BVPair;
Ted Kremenek13bd4232011-01-20 17:37:17 +000099
Ted Kremenek610068c2011-01-15 02:58:47 +0000100class CFGBlockValues {
101 const CFG &cfg;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000102 BVPair *vals;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000103 ValueVector scratch;
Ted Kremenek4ddb3872011-03-15 05:30:12 +0000104 DeclToIndex declToIndex;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000105
Ted Kremenek136f8f22011-03-15 04:57:27 +0000106 ValueVector &lazyCreate(ValueVector *&bv);
Ted Kremenek610068c2011-01-15 02:58:47 +0000107public:
108 CFGBlockValues(const CFG &cfg);
109 ~CFGBlockValues();
110
Ted Kremenekd40066b2011-04-04 23:29:12 +0000111 unsigned getNumEntries() const { return declToIndex.size(); }
112
Ted Kremenek610068c2011-01-15 02:58:47 +0000113 void computeSetOfDeclarations(const DeclContext &dc);
Ted Kremenek136f8f22011-03-15 04:57:27 +0000114 ValueVector &getValueVector(const CFGBlock *block,
Ted Kremenek13bd4232011-01-20 17:37:17 +0000115 const CFGBlock *dstBlock);
116
Ted Kremenek136f8f22011-03-15 04:57:27 +0000117 BVPair &getValueVectors(const CFGBlock *block, bool shouldLazyCreate);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000118
Ted Kremenek136f8f22011-03-15 04:57:27 +0000119 void mergeIntoScratch(ValueVector const &source, bool isFirst);
120 bool updateValueVectorWithScratch(const CFGBlock *block);
121 bool updateValueVectors(const CFGBlock *block, const BVPair &newVals);
Ted Kremenek610068c2011-01-15 02:58:47 +0000122
123 bool hasNoDeclarations() const {
Ted Kremenek4ddb3872011-03-15 05:30:12 +0000124 return declToIndex.size() == 0;
Ted Kremenek610068c2011-01-15 02:58:47 +0000125 }
126
Ted Kremenekb831c672011-03-29 01:40:00 +0000127 bool hasEntry(const VarDecl *vd) const {
128 return declToIndex.getValueIndex(vd).hasValue();
129 }
130
Ted Kremenekf8adeef2011-04-04 20:30:58 +0000131 bool hasValues(const CFGBlock *block);
132
Ted Kremenek610068c2011-01-15 02:58:47 +0000133 void resetScratch();
Ted Kremenek136f8f22011-03-15 04:57:27 +0000134 ValueVector &getScratch() { return scratch; }
Ted Kremenek13bd4232011-01-20 17:37:17 +0000135
Ted Kremenek136f8f22011-03-15 04:57:27 +0000136 ValueVector::reference operator[](const VarDecl *vd);
Ted Kremenek610068c2011-01-15 02:58:47 +0000137};
Benjamin Kramerda57f3e2011-03-26 12:38:21 +0000138} // end anonymous namespace
Ted Kremenek610068c2011-01-15 02:58:47 +0000139
140CFGBlockValues::CFGBlockValues(const CFG &c) : cfg(c), vals(0) {
141 unsigned n = cfg.getNumBlockIDs();
142 if (!n)
143 return;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000144 vals = new std::pair<ValueVector*, ValueVector*>[n];
Chandler Carruth75c40642011-04-28 08:19:45 +0000145 memset((void*)vals, 0, sizeof(*vals) * n);
Ted Kremenek610068c2011-01-15 02:58:47 +0000146}
147
148CFGBlockValues::~CFGBlockValues() {
149 unsigned n = cfg.getNumBlockIDs();
150 if (n == 0)
151 return;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000152 for (unsigned i = 0; i < n; ++i) {
153 delete vals[i].first;
154 delete vals[i].second;
155 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000156 delete [] vals;
157}
158
159void CFGBlockValues::computeSetOfDeclarations(const DeclContext &dc) {
Ted Kremenek4ddb3872011-03-15 05:30:12 +0000160 declToIndex.computeMap(dc);
161 scratch.resize(declToIndex.size());
Ted Kremenek610068c2011-01-15 02:58:47 +0000162}
163
Ted Kremenek136f8f22011-03-15 04:57:27 +0000164ValueVector &CFGBlockValues::lazyCreate(ValueVector *&bv) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000165 if (!bv)
Ted Kremenek4ddb3872011-03-15 05:30:12 +0000166 bv = new ValueVector(declToIndex.size());
Ted Kremenek610068c2011-01-15 02:58:47 +0000167 return *bv;
168}
169
Ted Kremenek13bd4232011-01-20 17:37:17 +0000170/// This function pattern matches for a '&&' or '||' that appears at
171/// the beginning of a CFGBlock that also (1) has a terminator and
172/// (2) has no other elements. If such an expression is found, it is returned.
173static BinaryOperator *getLogicalOperatorInChain(const CFGBlock *block) {
174 if (block->empty())
175 return 0;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000176
Ted Kremenek3c0349e2011-03-01 03:15:10 +0000177 const CFGStmt *cstmt = block->front().getAs<CFGStmt>();
Ted Kremenek76709bf2011-03-15 05:22:28 +0000178 if (!cstmt)
179 return 0;
180
Ted Kremenek3c0349e2011-03-01 03:15:10 +0000181 BinaryOperator *b = llvm::dyn_cast_or_null<BinaryOperator>(cstmt->getStmt());
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000182
183 if (!b || !b->isLogicalOp())
Ted Kremenek13bd4232011-01-20 17:37:17 +0000184 return 0;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000185
Ted Kremeneke6c28032011-05-10 22:10:35 +0000186 if (block->pred_size() == 2) {
187 if (block->getTerminatorCondition() == b) {
188 if (block->succ_size() == 2)
189 return b;
190 }
191 else if (block->size() == 1)
192 return b;
193 }
194
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000195 return 0;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000196}
197
Ted Kremenek136f8f22011-03-15 04:57:27 +0000198ValueVector &CFGBlockValues::getValueVector(const CFGBlock *block,
199 const CFGBlock *dstBlock) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000200 unsigned idx = block->getBlockID();
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000201 if (dstBlock && getLogicalOperatorInChain(block)) {
202 if (*block->succ_begin() == dstBlock)
203 return lazyCreate(vals[idx].first);
204 assert(*(block->succ_begin()+1) == dstBlock);
205 return lazyCreate(vals[idx].second);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000206 }
207
208 assert(vals[idx].second == 0);
209 return lazyCreate(vals[idx].first);
210}
211
Ted Kremenekf8adeef2011-04-04 20:30:58 +0000212bool CFGBlockValues::hasValues(const CFGBlock *block) {
213 unsigned idx = block->getBlockID();
214 return vals[idx].second != 0;
215}
216
Ted Kremenek136f8f22011-03-15 04:57:27 +0000217BVPair &CFGBlockValues::getValueVectors(const clang::CFGBlock *block,
218 bool shouldLazyCreate) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000219 unsigned idx = block->getBlockID();
220 lazyCreate(vals[idx].first);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000221 if (shouldLazyCreate)
222 lazyCreate(vals[idx].second);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000223 return vals[idx];
224}
225
Ted Kremenek136f8f22011-03-15 04:57:27 +0000226void CFGBlockValues::mergeIntoScratch(ValueVector const &source,
Ted Kremenek610068c2011-01-15 02:58:47 +0000227 bool isFirst) {
228 if (isFirst)
229 scratch = source;
230 else
Argyrios Kyrtzidis049f6d02011-05-31 03:56:09 +0000231 scratch |= source;
Ted Kremenek610068c2011-01-15 02:58:47 +0000232}
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000233#if 0
Ted Kremenek136f8f22011-03-15 04:57:27 +0000234static void printVector(const CFGBlock *block, ValueVector &bv,
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000235 unsigned num) {
236
237 llvm::errs() << block->getBlockID() << " :";
238 for (unsigned i = 0; i < bv.size(); ++i) {
239 llvm::errs() << ' ' << bv[i];
240 }
241 llvm::errs() << " : " << num << '\n';
242}
243#endif
Ted Kremenek610068c2011-01-15 02:58:47 +0000244
Ted Kremenek136f8f22011-03-15 04:57:27 +0000245bool CFGBlockValues::updateValueVectorWithScratch(const CFGBlock *block) {
246 ValueVector &dst = getValueVector(block, 0);
Ted Kremenek610068c2011-01-15 02:58:47 +0000247 bool changed = (dst != scratch);
248 if (changed)
249 dst = scratch;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000250#if 0
251 printVector(block, scratch, 0);
252#endif
Ted Kremenek13bd4232011-01-20 17:37:17 +0000253 return changed;
254}
255
Ted Kremenek136f8f22011-03-15 04:57:27 +0000256bool CFGBlockValues::updateValueVectors(const CFGBlock *block,
Ted Kremenek13bd4232011-01-20 17:37:17 +0000257 const BVPair &newVals) {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000258 BVPair &vals = getValueVectors(block, true);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000259 bool changed = *newVals.first != *vals.first ||
260 *newVals.second != *vals.second;
261 *vals.first = *newVals.first;
262 *vals.second = *newVals.second;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000263#if 0
264 printVector(block, *vals.first, 1);
265 printVector(block, *vals.second, 2);
266#endif
Ted Kremenek610068c2011-01-15 02:58:47 +0000267 return changed;
268}
269
270void CFGBlockValues::resetScratch() {
271 scratch.reset();
272}
273
Ted Kremenek136f8f22011-03-15 04:57:27 +0000274ValueVector::reference CFGBlockValues::operator[](const VarDecl *vd) {
Ted Kremenek4ddb3872011-03-15 05:30:12 +0000275 const llvm::Optional<unsigned> &idx = declToIndex.getValueIndex(vd);
Ted Kremenek610068c2011-01-15 02:58:47 +0000276 assert(idx.hasValue());
277 return scratch[idx.getValue()];
278}
279
280//------------------------------------------------------------------------====//
281// Worklist: worklist for dataflow analysis.
282//====------------------------------------------------------------------------//
283
284namespace {
285class DataflowWorklist {
286 llvm::SmallVector<const CFGBlock *, 20> worklist;
Ted Kremenek496398d2011-03-15 04:57:32 +0000287 llvm::BitVector enqueuedBlocks;
Ted Kremenek610068c2011-01-15 02:58:47 +0000288public:
289 DataflowWorklist(const CFG &cfg) : enqueuedBlocks(cfg.getNumBlockIDs()) {}
290
Ted Kremenek610068c2011-01-15 02:58:47 +0000291 void enqueueSuccessors(const CFGBlock *block);
292 const CFGBlock *dequeue();
Ted Kremenek610068c2011-01-15 02:58:47 +0000293};
294}
295
Ted Kremenek610068c2011-01-15 02:58:47 +0000296void DataflowWorklist::enqueueSuccessors(const clang::CFGBlock *block) {
Chandler Carruth80520502011-07-08 11:19:06 +0000297 unsigned OldWorklistSize = worklist.size();
Ted Kremenek610068c2011-01-15 02:58:47 +0000298 for (CFGBlock::const_succ_iterator I = block->succ_begin(),
299 E = block->succ_end(); I != E; ++I) {
Chandler Carruth80520502011-07-08 11:19:06 +0000300 const CFGBlock *Successor = *I;
301 if (!Successor || enqueuedBlocks[Successor->getBlockID()])
302 continue;
303 worklist.push_back(Successor);
304 enqueuedBlocks[Successor->getBlockID()] = true;
Ted Kremenek610068c2011-01-15 02:58:47 +0000305 }
Chandler Carruth80520502011-07-08 11:19:06 +0000306 if (OldWorklistSize == 0 || OldWorklistSize == worklist.size())
307 return;
308
309 // Rotate the newly added blocks to the start of the worklist so that it forms
310 // a proper queue when we pop off the end of the worklist.
311 std::rotate(worklist.begin(), worklist.begin() + OldWorklistSize,
312 worklist.end());
Ted Kremenek610068c2011-01-15 02:58:47 +0000313}
314
315const CFGBlock *DataflowWorklist::dequeue() {
316 if (worklist.empty())
317 return 0;
318 const CFGBlock *b = worklist.back();
319 worklist.pop_back();
320 enqueuedBlocks[b->getBlockID()] = false;
321 return b;
322}
323
324//------------------------------------------------------------------------====//
325// Transfer function for uninitialized values analysis.
326//====------------------------------------------------------------------------//
327
Ted Kremenek610068c2011-01-15 02:58:47 +0000328namespace {
329class FindVarResult {
330 const VarDecl *vd;
331 const DeclRefExpr *dr;
332public:
333 FindVarResult(VarDecl *vd, DeclRefExpr *dr) : vd(vd), dr(dr) {}
334
335 const DeclRefExpr *getDeclRefExpr() const { return dr; }
336 const VarDecl *getDecl() const { return vd; }
337};
338
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000339class TransferFunctions : public StmtVisitor<TransferFunctions> {
Ted Kremenek610068c2011-01-15 02:58:47 +0000340 CFGBlockValues &vals;
341 const CFG &cfg;
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000342 AnalysisContext &ac;
Ted Kremenek610068c2011-01-15 02:58:47 +0000343 UninitVariablesHandler *handler;
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000344 const bool flagBlockUses;
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000345
346 /// The last DeclRefExpr seen when analyzing a block. Used to
347 /// cheat when detecting cases when the address of a variable is taken.
348 DeclRefExpr *lastDR;
349
350 /// The last lvalue-to-rvalue conversion of a variable whose value
351 /// was uninitialized. Normally this results in a warning, but it is
352 /// possible to either silence the warning in some cases, or we
353 /// propagate the uninitialized value.
354 CastExpr *lastLoad;
Ted Kremenek610068c2011-01-15 02:58:47 +0000355public:
356 TransferFunctions(CFGBlockValues &vals, const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000357 AnalysisContext &ac,
358 UninitVariablesHandler *handler,
359 bool flagBlockUses)
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000360 : vals(vals), cfg(cfg), ac(ac), handler(handler),
361 flagBlockUses(flagBlockUses), lastDR(0), lastLoad(0) {}
Ted Kremenek610068c2011-01-15 02:58:47 +0000362
363 const CFG &getCFG() { return cfg; }
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000364 void reportUninit(const DeclRefExpr *ex, const VarDecl *vd,
365 bool isAlwaysUninit);
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000366
367 void VisitBlockExpr(BlockExpr *be);
Ted Kremenek610068c2011-01-15 02:58:47 +0000368 void VisitDeclStmt(DeclStmt *ds);
Ted Kremenekc21fed32011-01-18 21:18:58 +0000369 void VisitDeclRefExpr(DeclRefExpr *dr);
Ted Kremenek610068c2011-01-15 02:58:47 +0000370 void VisitUnaryOperator(UnaryOperator *uo);
371 void VisitBinaryOperator(BinaryOperator *bo);
372 void VisitCastExpr(CastExpr *ce);
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000373 void VisitObjCForCollectionStmt(ObjCForCollectionStmt *fs);
374 void Visit(Stmt *s);
Ted Kremenek40900ee2011-01-27 02:29:34 +0000375
376 bool isTrackedVar(const VarDecl *vd) {
377 return ::isTrackedVar(vd, cast<DeclContext>(ac.getDecl()));
378 }
379
380 FindVarResult findBlockVarDecl(Expr *ex);
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000381
382 void ProcessUses(Stmt *s = 0);
Ted Kremenek610068c2011-01-15 02:58:47 +0000383};
384}
385
386void TransferFunctions::reportUninit(const DeclRefExpr *ex,
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000387 const VarDecl *vd, bool isAlwaysUnit) {
388 if (handler) handler->handleUseOfUninitVariable(ex, vd, isAlwaysUnit);
Ted Kremenek610068c2011-01-15 02:58:47 +0000389}
390
Ted Kremenek40900ee2011-01-27 02:29:34 +0000391FindVarResult TransferFunctions::findBlockVarDecl(Expr* ex) {
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000392 if (DeclRefExpr* dr = dyn_cast<DeclRefExpr>(ex->IgnoreParenCasts()))
393 if (VarDecl *vd = dyn_cast<VarDecl>(dr->getDecl()))
394 if (isTrackedVar(vd))
Ted Kremenek40900ee2011-01-27 02:29:34 +0000395 return FindVarResult(vd, dr);
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000396 return FindVarResult(0, 0);
397}
398
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000399void TransferFunctions::VisitObjCForCollectionStmt(ObjCForCollectionStmt *fs) {
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000400 // This represents an initialization of the 'element' value.
401 Stmt *element = fs->getElement();
402 const VarDecl* vd = 0;
403
404 if (DeclStmt* ds = dyn_cast<DeclStmt>(element)) {
405 vd = cast<VarDecl>(ds->getSingleDecl());
406 if (!isTrackedVar(vd))
407 vd = 0;
408 }
409 else {
410 // Initialize the value of the reference variable.
411 const FindVarResult &res = findBlockVarDecl(cast<Expr>(element));
412 vd = res.getDecl();
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000413 }
414
415 if (vd)
416 vals[vd] = Initialized;
417}
418
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000419void TransferFunctions::VisitBlockExpr(BlockExpr *be) {
420 if (!flagBlockUses || !handler)
421 return;
Ted Kremenekbc8b44c2011-03-31 22:32:41 +0000422 const BlockDecl *bd = be->getBlockDecl();
423 for (BlockDecl::capture_const_iterator i = bd->capture_begin(),
424 e = bd->capture_end() ; i != e; ++i) {
425 const VarDecl *vd = i->getVariable();
426 if (!vd->hasLocalStorage())
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000427 continue;
Ted Kremenekbc8b44c2011-03-31 22:32:41 +0000428 if (!isTrackedVar(vd))
429 continue;
430 if (i->isByRef()) {
431 vals[vd] = Initialized;
432 continue;
433 }
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000434 Value v = vals[vd];
435 if (isUninitialized(v))
436 handler->handleUseOfUninitVariable(be, vd, isAlwaysUninit(v));
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000437 }
438}
439
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000440void TransferFunctions::VisitDeclRefExpr(DeclRefExpr *dr) {
441 // Record the last DeclRefExpr seen. This is an lvalue computation.
442 // We use this value to later detect if a variable "escapes" the analysis.
443 if (const VarDecl *vd = dyn_cast<VarDecl>(dr->getDecl()))
Ted Kremenekdd4286b2011-07-20 19:49:47 +0000444 if (isTrackedVar(vd)) {
445 ProcessUses();
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000446 lastDR = dr;
Ted Kremenekdd4286b2011-07-20 19:49:47 +0000447 }
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000448}
449
Ted Kremenek610068c2011-01-15 02:58:47 +0000450void TransferFunctions::VisitDeclStmt(DeclStmt *ds) {
451 for (DeclStmt::decl_iterator DI = ds->decl_begin(), DE = ds->decl_end();
452 DI != DE; ++DI) {
453 if (VarDecl *vd = dyn_cast<VarDecl>(*DI)) {
Ted Kremenek4dccb902011-01-18 05:00:42 +0000454 if (isTrackedVar(vd)) {
Chandler Carruthb88fb022011-04-05 21:36:30 +0000455 if (Expr *init = vd->getInit()) {
Chandler Carruthb88fb022011-04-05 21:36:30 +0000456 // If the initializer consists solely of a reference to itself, we
457 // explicitly mark the variable as uninitialized. This allows code
458 // like the following:
459 //
460 // int x = x;
461 //
462 // to deliberately leave a variable uninitialized. Different analysis
463 // clients can detect this pattern and adjust their reporting
464 // appropriately, but we need to continue to analyze subsequent uses
465 // of the variable.
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000466 if (init == lastLoad) {
Ted Kremenekd626ec42011-07-19 20:33:49 +0000467 DeclRefExpr *DR =
468 cast<DeclRefExpr>(lastLoad->getSubExpr()->IgnoreParens());
Ted Kremenek62d126e2011-07-19 21:41:51 +0000469 if (DR->getDecl() == vd) {
470 // int x = x;
471 // Propagate uninitialized value, but don't immediately report
472 // a problem.
473 vals[vd] = Uninitialized;
474 lastLoad = 0;
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000475 lastDR = 0;
Ted Kremenek62d126e2011-07-19 21:41:51 +0000476 return;
477 }
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000478 }
Ted Kremenek62d126e2011-07-19 21:41:51 +0000479
480 // All other cases: treat the new variable as initialized.
481 vals[vd] = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000482 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000483 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000484 }
485 }
486}
487
Ted Kremenek610068c2011-01-15 02:58:47 +0000488void TransferFunctions::VisitBinaryOperator(clang::BinaryOperator *bo) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000489 if (bo->isAssignmentOp()) {
490 const FindVarResult &res = findBlockVarDecl(bo->getLHS());
491 if (const VarDecl* vd = res.getDecl()) {
Ted Kremenek496398d2011-03-15 04:57:32 +0000492 ValueVector::reference val = vals[vd];
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000493 if (isUninitialized(val)) {
Chandler Carruth84350692011-07-16 22:27:02 +0000494 if (bo->getOpcode() != BO_Assign)
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000495 reportUninit(res.getDeclRefExpr(), vd, isAlwaysUninit(val));
Chandler Carruth84350692011-07-16 22:27:02 +0000496 val = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000497 }
498 }
499 }
500}
501
502void TransferFunctions::VisitUnaryOperator(clang::UnaryOperator *uo) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000503 switch (uo->getOpcode()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000504 case clang::UO_PostDec:
505 case clang::UO_PostInc:
506 case clang::UO_PreDec:
507 case clang::UO_PreInc: {
508 const FindVarResult &res = findBlockVarDecl(uo->getSubExpr());
509 if (const VarDecl *vd = res.getDecl()) {
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000510 assert(res.getDeclRefExpr() == lastDR);
511 // We null out lastDR to indicate we have fully processed it
512 // and we don't want the auto-value setting in Visit().
513 lastDR = 0;
Ted Kremenekc21fed32011-01-18 21:18:58 +0000514
Ted Kremenekf7bafc72011-03-15 04:57:38 +0000515 ValueVector::reference val = vals[vd];
516 if (isUninitialized(val)) {
517 reportUninit(res.getDeclRefExpr(), vd, isAlwaysUninit(val));
518 // Don't cascade warnings.
Chandler Carruth84350692011-07-16 22:27:02 +0000519 val = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000520 }
521 }
522 break;
523 }
524 default:
525 break;
526 }
527}
528
529void TransferFunctions::VisitCastExpr(clang::CastExpr *ce) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000530 if (ce->getCastKind() == CK_LValueToRValue) {
531 const FindVarResult &res = findBlockVarDecl(ce->getSubExpr());
Ted Kremenekc21fed32011-01-18 21:18:58 +0000532 if (const VarDecl *vd = res.getDecl()) {
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000533 assert(res.getDeclRefExpr() == lastDR);
534 if (isUninitialized(vals[vd])) {
535 // Record this load of an uninitialized value. Normally this
536 // results in a warning, but we delay reporting the issue
537 // in case it is wrapped in a void cast, etc.
538 lastLoad = ce;
Ted Kremenekc21fed32011-01-18 21:18:58 +0000539 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000540 }
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000541 }
542 else if (CStyleCastExpr *cse = dyn_cast<CStyleCastExpr>(ce)) {
543 if (cse->getType()->isVoidType()) {
544 // e.g. (void) x;
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000545 if (lastLoad == cse->getSubExpr()) {
546 // Squelch any detected load of an uninitialized value if
547 // we cast it to void.
548 lastLoad = 0;
549 lastDR = 0;
550 }
551 }
552 }
553}
554
555void TransferFunctions::Visit(clang::Stmt *s) {
556 StmtVisitor<TransferFunctions>::Visit(s);
557 ProcessUses(s);
558}
559
560void TransferFunctions::ProcessUses(Stmt *s) {
561 // This method is typically called after visiting a CFGElement statement
562 // in the CFG. We delay processing of reporting many loads of uninitialized
563 // values until here.
564 if (lastLoad) {
565 // If we just visited the lvalue-to-rvalue cast, there is nothing
566 // left to do.
567 if (lastLoad == s)
568 return;
569
570 // If we reach here, we have seen a load of an uninitialized value
571 // and it hasn't been casted to void or otherwise handled. In this
572 // situation, report the incident.
Ted Kremenekd626ec42011-07-19 20:33:49 +0000573 DeclRefExpr *DR = cast<DeclRefExpr>(lastLoad->getSubExpr()->IgnoreParens());
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000574 VarDecl *VD = cast<VarDecl>(DR->getDecl());
575 reportUninit(DR, VD, isAlwaysUninit(vals[VD]));
576 lastLoad = 0;
577
578 // Prevent cascade of warnings.
579 vals[VD] = Initialized;
580 if (DR == lastDR) {
581 lastDR = 0;
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000582 return;
583 }
584 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000585
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000586 // Any other uses of 'lastDR' involve taking an lvalue of variable.
587 // In this case, it "escapes" the analysis.
588 if (lastDR && lastDR != s) {
589 vals[cast<VarDecl>(lastDR->getDecl())] = Initialized;
590 lastDR = 0;
Chandler Carruth86684942011-04-13 08:18:42 +0000591 }
592}
593
Ted Kremenek610068c2011-01-15 02:58:47 +0000594//------------------------------------------------------------------------====//
595// High-level "driver" logic for uninitialized values analysis.
596//====------------------------------------------------------------------------//
597
Ted Kremenek13bd4232011-01-20 17:37:17 +0000598static bool runOnBlock(const CFGBlock *block, const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000599 AnalysisContext &ac, CFGBlockValues &vals,
Ted Kremenekf8adeef2011-04-04 20:30:58 +0000600 llvm::BitVector &wasAnalyzed,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000601 UninitVariablesHandler *handler = 0,
602 bool flagBlockUses = false) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000603
Ted Kremenekf8adeef2011-04-04 20:30:58 +0000604 wasAnalyzed[block->getBlockID()] = true;
605
Ted Kremenek13bd4232011-01-20 17:37:17 +0000606 if (const BinaryOperator *b = getLogicalOperatorInChain(block)) {
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000607 CFGBlock::const_pred_iterator itr = block->pred_begin();
Ted Kremenek136f8f22011-03-15 04:57:27 +0000608 BVPair vA = vals.getValueVectors(*itr, false);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000609 ++itr;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000610 BVPair vB = vals.getValueVectors(*itr, false);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000611
612 BVPair valsAB;
613
614 if (b->getOpcode() == BO_LAnd) {
615 // Merge the 'F' bits from the first and second.
616 vals.mergeIntoScratch(*(vA.second ? vA.second : vA.first), true);
617 vals.mergeIntoScratch(*(vB.second ? vB.second : vB.first), false);
618 valsAB.first = vA.first;
Ted Kremenek2d4bed12011-01-20 21:25:31 +0000619 valsAB.second = &vals.getScratch();
Ted Kremenek13bd4232011-01-20 17:37:17 +0000620 }
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000621 else {
622 // Merge the 'T' bits from the first and second.
623 assert(b->getOpcode() == BO_LOr);
624 vals.mergeIntoScratch(*vA.first, true);
625 vals.mergeIntoScratch(*vB.first, false);
626 valsAB.first = &vals.getScratch();
627 valsAB.second = vA.second ? vA.second : vA.first;
628 }
Ted Kremenek136f8f22011-03-15 04:57:27 +0000629 return vals.updateValueVectors(block, valsAB);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000630 }
631
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000632 // Default behavior: merge in values of predecessor blocks.
Ted Kremenek610068c2011-01-15 02:58:47 +0000633 vals.resetScratch();
634 bool isFirst = true;
635 for (CFGBlock::const_pred_iterator I = block->pred_begin(),
636 E = block->pred_end(); I != E; ++I) {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000637 vals.mergeIntoScratch(vals.getValueVector(*I, block), isFirst);
Ted Kremenek610068c2011-01-15 02:58:47 +0000638 isFirst = false;
639 }
640 // Apply the transfer function.
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000641 TransferFunctions tf(vals, cfg, ac, handler, flagBlockUses);
Ted Kremenek610068c2011-01-15 02:58:47 +0000642 for (CFGBlock::const_iterator I = block->begin(), E = block->end();
643 I != E; ++I) {
644 if (const CFGStmt *cs = dyn_cast<CFGStmt>(&*I)) {
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000645 tf.Visit(cs->getStmt());
Ted Kremenek610068c2011-01-15 02:58:47 +0000646 }
647 }
Ted Kremenek0c8e5a02011-07-19 14:18:48 +0000648 tf.ProcessUses();
Ted Kremenek136f8f22011-03-15 04:57:27 +0000649 return vals.updateValueVectorWithScratch(block);
Ted Kremenek610068c2011-01-15 02:58:47 +0000650}
651
Chandler Carruth5d989942011-07-06 16:21:37 +0000652void clang::runUninitializedVariablesAnalysis(
653 const DeclContext &dc,
654 const CFG &cfg,
655 AnalysisContext &ac,
656 UninitVariablesHandler &handler,
657 UninitVariablesAnalysisStats &stats) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000658 CFGBlockValues vals(cfg);
659 vals.computeSetOfDeclarations(dc);
660 if (vals.hasNoDeclarations())
661 return;
Ted Kremenekd40066b2011-04-04 23:29:12 +0000662
Chandler Carruth5d989942011-07-06 16:21:37 +0000663 stats.NumVariablesAnalyzed = vals.getNumEntries();
664
Ted Kremenekd40066b2011-04-04 23:29:12 +0000665 // Mark all variables uninitialized at the entry.
666 const CFGBlock &entry = cfg.getEntry();
667 for (CFGBlock::const_succ_iterator i = entry.succ_begin(),
668 e = entry.succ_end(); i != e; ++i) {
669 if (const CFGBlock *succ = *i) {
670 ValueVector &vec = vals.getValueVector(&entry, succ);
671 const unsigned n = vals.getNumEntries();
672 for (unsigned j = 0; j < n ; ++j) {
673 vec[j] = Uninitialized;
674 }
675 }
676 }
677
678 // Proceed with the workist.
Ted Kremenek610068c2011-01-15 02:58:47 +0000679 DataflowWorklist worklist(cfg);
Ted Kremenek496398d2011-03-15 04:57:32 +0000680 llvm::BitVector previouslyVisited(cfg.getNumBlockIDs());
Ted Kremenek610068c2011-01-15 02:58:47 +0000681 worklist.enqueueSuccessors(&cfg.getEntry());
Ted Kremenekf8adeef2011-04-04 20:30:58 +0000682 llvm::BitVector wasAnalyzed(cfg.getNumBlockIDs(), false);
Ted Kremenek610068c2011-01-15 02:58:47 +0000683
684 while (const CFGBlock *block = worklist.dequeue()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000685 // Did the block change?
Chandler Carruth5d989942011-07-06 16:21:37 +0000686 bool changed = runOnBlock(block, cfg, ac, vals, wasAnalyzed);
687 ++stats.NumBlockVisits;
Ted Kremenek610068c2011-01-15 02:58:47 +0000688 if (changed || !previouslyVisited[block->getBlockID()])
689 worklist.enqueueSuccessors(block);
690 previouslyVisited[block->getBlockID()] = true;
691 }
692
693 // Run through the blocks one more time, and report uninitialized variabes.
694 for (CFG::const_iterator BI = cfg.begin(), BE = cfg.end(); BI != BE; ++BI) {
Chandler Carruth5d989942011-07-06 16:21:37 +0000695 if (wasAnalyzed[(*BI)->getBlockID()]) {
Ted Kremenekf8adeef2011-04-04 20:30:58 +0000696 runOnBlock(*BI, cfg, ac, vals, wasAnalyzed, &handler,
697 /* flagBlockUses */ true);
Chandler Carruth5d989942011-07-06 16:21:37 +0000698 ++stats.NumBlockVisits;
699 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000700 }
701}
702
703UninitVariablesHandler::~UninitVariablesHandler() {}