blob: 4aa62254da081c80738311d00e3d27bafe7de093 [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"
17#include "llvm/ADT/BitVector.h"
18#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 Kremenekc104e532011-01-18 04:53:25 +000029 return vd->isLocalVarDecl() && !vd->hasGlobalStorage() &&
Ted Kremenek40900ee2011-01-27 02:29:34 +000030 vd->getType()->isScalarType() &&
31 vd->getDeclContext() == dc;
Ted Kremenekc104e532011-01-18 04:53:25 +000032}
33
Ted Kremenek610068c2011-01-15 02:58:47 +000034//------------------------------------------------------------------------====//
Ted Kremenek136f8f22011-03-15 04:57:27 +000035// DeclToIndex: a mapping from Decls we track to value indices.
Ted Kremenek610068c2011-01-15 02:58:47 +000036//====------------------------------------------------------------------------//
37
38namespace {
Ted Kremenek136f8f22011-03-15 04:57:27 +000039class DeclToIndex {
Ted Kremenek610068c2011-01-15 02:58:47 +000040 llvm::DenseMap<const VarDecl *, unsigned> map;
41public:
Ted Kremenek136f8f22011-03-15 04:57:27 +000042 DeclToIndex() {}
Ted Kremenek610068c2011-01-15 02:58:47 +000043
44 /// Compute the actual mapping from declarations to bits.
45 void computeMap(const DeclContext &dc);
46
47 /// Return the number of declarations in the map.
48 unsigned size() const { return map.size(); }
49
50 /// Returns the bit vector index for a given declaration.
Ted Kremenek136f8f22011-03-15 04:57:27 +000051 llvm::Optional<unsigned> getValueIndex(const VarDecl *d);
Ted Kremenek610068c2011-01-15 02:58:47 +000052};
53}
54
Ted Kremenek136f8f22011-03-15 04:57:27 +000055void DeclToIndex::computeMap(const DeclContext &dc) {
Ted Kremenek610068c2011-01-15 02:58:47 +000056 unsigned count = 0;
57 DeclContext::specific_decl_iterator<VarDecl> I(dc.decls_begin()),
58 E(dc.decls_end());
59 for ( ; I != E; ++I) {
60 const VarDecl *vd = *I;
Ted Kremenek40900ee2011-01-27 02:29:34 +000061 if (isTrackedVar(vd, &dc))
Ted Kremenek610068c2011-01-15 02:58:47 +000062 map[vd] = count++;
63 }
64}
65
Ted Kremenek136f8f22011-03-15 04:57:27 +000066llvm::Optional<unsigned> DeclToIndex::getValueIndex(const VarDecl *d) {
Ted Kremenek610068c2011-01-15 02:58:47 +000067 llvm::DenseMap<const VarDecl *, unsigned>::iterator I = map.find(d);
68 if (I == map.end())
69 return llvm::Optional<unsigned>();
70 return I->second;
71}
72
73//------------------------------------------------------------------------====//
74// CFGBlockValues: dataflow values for CFG blocks.
75//====------------------------------------------------------------------------//
76
Ted Kremenek496398d2011-03-15 04:57:32 +000077enum Value { Initialized = 0, Uninitialized = 1 };
Ted Kremenekafb10c42011-03-15 04:57:29 +000078
79class ValueVector {
80 llvm::BitVector vec;
81public:
82 ValueVector() {}
83 ValueVector(unsigned size) : vec(size) {}
Ted Kremenekafb10c42011-03-15 04:57:29 +000084 void resize(unsigned n) { vec.resize(n); }
85 void merge(const ValueVector &rhs) { vec |= rhs.vec; }
86 bool operator!=(const ValueVector &rhs) const { return vec != rhs.vec; }
Ted Kremenekafb10c42011-03-15 04:57:29 +000087 void reset() { vec.reset(); }
Ted Kremenek496398d2011-03-15 04:57:32 +000088
89 class reference {
90 ValueVector &vv;
91 const unsigned idx;
92
93 reference(); // Undefined
94 public:
95 reference(ValueVector &vv, unsigned idx) : vv(vv), idx(idx) {}
96 ~reference() {}
97
98 reference &operator=(Value v) {
99 vv.vec[idx] = (v == Initialized ? false : true);
100 return *this;
101 }
102 bool operator==(Value v) {
103 return vv.vec[idx] == (v == Initialized ? false : true);
104 }
105 };
106
107 reference operator[](unsigned idx) { return reference(*this, idx); }
Ted Kremenekafb10c42011-03-15 04:57:29 +0000108};
109
Ted Kremenek136f8f22011-03-15 04:57:27 +0000110typedef std::pair<ValueVector *, ValueVector *> BVPair;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000111
Ted Kremenek610068c2011-01-15 02:58:47 +0000112namespace {
113class CFGBlockValues {
114 const CFG &cfg;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000115 BVPair *vals;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000116 ValueVector scratch;
117 DeclToIndex DeclToIndex;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000118
Ted Kremenek136f8f22011-03-15 04:57:27 +0000119 ValueVector &lazyCreate(ValueVector *&bv);
Ted Kremenek610068c2011-01-15 02:58:47 +0000120public:
121 CFGBlockValues(const CFG &cfg);
122 ~CFGBlockValues();
123
124 void computeSetOfDeclarations(const DeclContext &dc);
Ted Kremenek136f8f22011-03-15 04:57:27 +0000125 ValueVector &getValueVector(const CFGBlock *block,
Ted Kremenek13bd4232011-01-20 17:37:17 +0000126 const CFGBlock *dstBlock);
127
Ted Kremenek136f8f22011-03-15 04:57:27 +0000128 BVPair &getValueVectors(const CFGBlock *block, bool shouldLazyCreate);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000129
Ted Kremenek136f8f22011-03-15 04:57:27 +0000130 void mergeIntoScratch(ValueVector const &source, bool isFirst);
131 bool updateValueVectorWithScratch(const CFGBlock *block);
132 bool updateValueVectors(const CFGBlock *block, const BVPair &newVals);
Ted Kremenek610068c2011-01-15 02:58:47 +0000133
134 bool hasNoDeclarations() const {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000135 return DeclToIndex.size() == 0;
Ted Kremenek610068c2011-01-15 02:58:47 +0000136 }
137
138 void resetScratch();
Ted Kremenek136f8f22011-03-15 04:57:27 +0000139 ValueVector &getScratch() { return scratch; }
Ted Kremenek13bd4232011-01-20 17:37:17 +0000140
Ted Kremenek136f8f22011-03-15 04:57:27 +0000141 ValueVector::reference operator[](const VarDecl *vd);
Ted Kremenek610068c2011-01-15 02:58:47 +0000142};
143}
144
145CFGBlockValues::CFGBlockValues(const CFG &c) : cfg(c), vals(0) {
146 unsigned n = cfg.getNumBlockIDs();
147 if (!n)
148 return;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000149 vals = new std::pair<ValueVector*, ValueVector*>[n];
Francois Pichet2d78c372011-01-15 13:27:47 +0000150 memset(vals, 0, sizeof(*vals) * n);
Ted Kremenek610068c2011-01-15 02:58:47 +0000151}
152
153CFGBlockValues::~CFGBlockValues() {
154 unsigned n = cfg.getNumBlockIDs();
155 if (n == 0)
156 return;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000157 for (unsigned i = 0; i < n; ++i) {
158 delete vals[i].first;
159 delete vals[i].second;
160 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000161 delete [] vals;
162}
163
164void CFGBlockValues::computeSetOfDeclarations(const DeclContext &dc) {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000165 DeclToIndex.computeMap(dc);
166 scratch.resize(DeclToIndex.size());
Ted Kremenek610068c2011-01-15 02:58:47 +0000167}
168
Ted Kremenek136f8f22011-03-15 04:57:27 +0000169ValueVector &CFGBlockValues::lazyCreate(ValueVector *&bv) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000170 if (!bv)
Ted Kremenek136f8f22011-03-15 04:57:27 +0000171 bv = new ValueVector(DeclToIndex.size());
Ted Kremenek610068c2011-01-15 02:58:47 +0000172 return *bv;
173}
174
Ted Kremenek13bd4232011-01-20 17:37:17 +0000175/// This function pattern matches for a '&&' or '||' that appears at
176/// the beginning of a CFGBlock that also (1) has a terminator and
177/// (2) has no other elements. If such an expression is found, it is returned.
178static BinaryOperator *getLogicalOperatorInChain(const CFGBlock *block) {
179 if (block->empty())
180 return 0;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000181
Ted Kremenek3c0349e2011-03-01 03:15:10 +0000182 const CFGStmt *cstmt = block->front().getAs<CFGStmt>();
183 BinaryOperator *b = llvm::dyn_cast_or_null<BinaryOperator>(cstmt->getStmt());
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000184
185 if (!b || !b->isLogicalOp())
Ted Kremenek13bd4232011-01-20 17:37:17 +0000186 return 0;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000187
188 if (block->pred_size() == 2 &&
189 ((block->succ_size() == 2 && block->getTerminatorCondition() == b) ||
190 block->size() == 1))
191 return b;
192
193 return 0;
Ted Kremenek13bd4232011-01-20 17:37:17 +0000194}
195
Ted Kremenek136f8f22011-03-15 04:57:27 +0000196ValueVector &CFGBlockValues::getValueVector(const CFGBlock *block,
197 const CFGBlock *dstBlock) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000198 unsigned idx = block->getBlockID();
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000199 if (dstBlock && getLogicalOperatorInChain(block)) {
200 if (*block->succ_begin() == dstBlock)
201 return lazyCreate(vals[idx].first);
202 assert(*(block->succ_begin()+1) == dstBlock);
203 return lazyCreate(vals[idx].second);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000204 }
205
206 assert(vals[idx].second == 0);
207 return lazyCreate(vals[idx].first);
208}
209
Ted Kremenek136f8f22011-03-15 04:57:27 +0000210BVPair &CFGBlockValues::getValueVectors(const clang::CFGBlock *block,
211 bool shouldLazyCreate) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000212 unsigned idx = block->getBlockID();
213 lazyCreate(vals[idx].first);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000214 if (shouldLazyCreate)
215 lazyCreate(vals[idx].second);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000216 return vals[idx];
217}
218
Ted Kremenek136f8f22011-03-15 04:57:27 +0000219void CFGBlockValues::mergeIntoScratch(ValueVector const &source,
Ted Kremenek610068c2011-01-15 02:58:47 +0000220 bool isFirst) {
221 if (isFirst)
222 scratch = source;
223 else
Ted Kremenekafb10c42011-03-15 04:57:29 +0000224 scratch.merge(source);
Ted Kremenek610068c2011-01-15 02:58:47 +0000225}
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000226#if 0
Ted Kremenek136f8f22011-03-15 04:57:27 +0000227static void printVector(const CFGBlock *block, ValueVector &bv,
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000228 unsigned num) {
229
230 llvm::errs() << block->getBlockID() << " :";
231 for (unsigned i = 0; i < bv.size(); ++i) {
232 llvm::errs() << ' ' << bv[i];
233 }
234 llvm::errs() << " : " << num << '\n';
235}
236#endif
Ted Kremenek610068c2011-01-15 02:58:47 +0000237
Ted Kremenek136f8f22011-03-15 04:57:27 +0000238bool CFGBlockValues::updateValueVectorWithScratch(const CFGBlock *block) {
239 ValueVector &dst = getValueVector(block, 0);
Ted Kremenek610068c2011-01-15 02:58:47 +0000240 bool changed = (dst != scratch);
241 if (changed)
242 dst = scratch;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000243#if 0
244 printVector(block, scratch, 0);
245#endif
Ted Kremenek13bd4232011-01-20 17:37:17 +0000246 return changed;
247}
248
Ted Kremenek136f8f22011-03-15 04:57:27 +0000249bool CFGBlockValues::updateValueVectors(const CFGBlock *block,
Ted Kremenek13bd4232011-01-20 17:37:17 +0000250 const BVPair &newVals) {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000251 BVPair &vals = getValueVectors(block, true);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000252 bool changed = *newVals.first != *vals.first ||
253 *newVals.second != *vals.second;
254 *vals.first = *newVals.first;
255 *vals.second = *newVals.second;
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000256#if 0
257 printVector(block, *vals.first, 1);
258 printVector(block, *vals.second, 2);
259#endif
Ted Kremenek610068c2011-01-15 02:58:47 +0000260 return changed;
261}
262
263void CFGBlockValues::resetScratch() {
264 scratch.reset();
265}
266
Ted Kremenek136f8f22011-03-15 04:57:27 +0000267ValueVector::reference CFGBlockValues::operator[](const VarDecl *vd) {
268 const llvm::Optional<unsigned> &idx = DeclToIndex.getValueIndex(vd);
Ted Kremenek610068c2011-01-15 02:58:47 +0000269 assert(idx.hasValue());
270 return scratch[idx.getValue()];
271}
272
273//------------------------------------------------------------------------====//
274// Worklist: worklist for dataflow analysis.
275//====------------------------------------------------------------------------//
276
277namespace {
278class DataflowWorklist {
279 llvm::SmallVector<const CFGBlock *, 20> worklist;
Ted Kremenek496398d2011-03-15 04:57:32 +0000280 llvm::BitVector enqueuedBlocks;
Ted Kremenek610068c2011-01-15 02:58:47 +0000281public:
282 DataflowWorklist(const CFG &cfg) : enqueuedBlocks(cfg.getNumBlockIDs()) {}
283
284 void enqueue(const CFGBlock *block);
285 void enqueueSuccessors(const CFGBlock *block);
286 const CFGBlock *dequeue();
287
288};
289}
290
291void DataflowWorklist::enqueue(const CFGBlock *block) {
Ted Kremenekc104e532011-01-18 04:53:25 +0000292 if (!block)
293 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000294 unsigned idx = block->getBlockID();
295 if (enqueuedBlocks[idx])
296 return;
297 worklist.push_back(block);
298 enqueuedBlocks[idx] = true;
299}
300
301void DataflowWorklist::enqueueSuccessors(const clang::CFGBlock *block) {
302 for (CFGBlock::const_succ_iterator I = block->succ_begin(),
303 E = block->succ_end(); I != E; ++I) {
304 enqueue(*I);
305 }
306}
307
308const CFGBlock *DataflowWorklist::dequeue() {
309 if (worklist.empty())
310 return 0;
311 const CFGBlock *b = worklist.back();
312 worklist.pop_back();
313 enqueuedBlocks[b->getBlockID()] = false;
314 return b;
315}
316
317//------------------------------------------------------------------------====//
318// Transfer function for uninitialized values analysis.
319//====------------------------------------------------------------------------//
320
Ted Kremenek610068c2011-01-15 02:58:47 +0000321namespace {
322class FindVarResult {
323 const VarDecl *vd;
324 const DeclRefExpr *dr;
325public:
326 FindVarResult(VarDecl *vd, DeclRefExpr *dr) : vd(vd), dr(dr) {}
327
328 const DeclRefExpr *getDeclRefExpr() const { return dr; }
329 const VarDecl *getDecl() const { return vd; }
330};
331
332class TransferFunctions : public CFGRecStmtVisitor<TransferFunctions> {
333 CFGBlockValues &vals;
334 const CFG &cfg;
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000335 AnalysisContext &ac;
Ted Kremenek610068c2011-01-15 02:58:47 +0000336 UninitVariablesHandler *handler;
Ted Kremenekc21fed32011-01-18 21:18:58 +0000337 const DeclRefExpr *currentDR;
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000338 const Expr *currentVoidCast;
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000339 const bool flagBlockUses;
Ted Kremenek610068c2011-01-15 02:58:47 +0000340public:
341 TransferFunctions(CFGBlockValues &vals, const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000342 AnalysisContext &ac,
343 UninitVariablesHandler *handler,
344 bool flagBlockUses)
345 : vals(vals), cfg(cfg), ac(ac), handler(handler), currentDR(0),
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000346 currentVoidCast(0), flagBlockUses(flagBlockUses) {}
Ted Kremenek610068c2011-01-15 02:58:47 +0000347
348 const CFG &getCFG() { return cfg; }
349 void reportUninit(const DeclRefExpr *ex, const VarDecl *vd);
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000350
351 void VisitBlockExpr(BlockExpr *be);
Ted Kremenek610068c2011-01-15 02:58:47 +0000352 void VisitDeclStmt(DeclStmt *ds);
Ted Kremenekc21fed32011-01-18 21:18:58 +0000353 void VisitDeclRefExpr(DeclRefExpr *dr);
Ted Kremenek610068c2011-01-15 02:58:47 +0000354 void VisitUnaryOperator(UnaryOperator *uo);
355 void VisitBinaryOperator(BinaryOperator *bo);
356 void VisitCastExpr(CastExpr *ce);
Peter Collingbournef4e3cfb2011-03-11 19:24:49 +0000357 void VisitUnaryExprOrTypeTraitExpr(UnaryExprOrTypeTraitExpr *se);
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000358 void BlockStmt_VisitObjCForCollectionStmt(ObjCForCollectionStmt *fs);
Ted Kremenek40900ee2011-01-27 02:29:34 +0000359
360 bool isTrackedVar(const VarDecl *vd) {
361 return ::isTrackedVar(vd, cast<DeclContext>(ac.getDecl()));
362 }
363
364 FindVarResult findBlockVarDecl(Expr *ex);
Ted Kremenek610068c2011-01-15 02:58:47 +0000365};
366}
367
368void TransferFunctions::reportUninit(const DeclRefExpr *ex,
369 const VarDecl *vd) {
370 if (handler) handler->handleUseOfUninitVariable(ex, vd);
371}
372
Ted Kremenek40900ee2011-01-27 02:29:34 +0000373FindVarResult TransferFunctions::findBlockVarDecl(Expr* ex) {
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000374 if (DeclRefExpr* dr = dyn_cast<DeclRefExpr>(ex->IgnoreParenCasts()))
375 if (VarDecl *vd = dyn_cast<VarDecl>(dr->getDecl()))
376 if (isTrackedVar(vd))
Ted Kremenek40900ee2011-01-27 02:29:34 +0000377 return FindVarResult(vd, dr);
Ted Kremenek1ea800c2011-01-27 02:01:31 +0000378 return FindVarResult(0, 0);
379}
380
381void TransferFunctions::BlockStmt_VisitObjCForCollectionStmt(
382 ObjCForCollectionStmt *fs) {
383
384 Visit(fs->getCollection());
385
386 // This represents an initialization of the 'element' value.
387 Stmt *element = fs->getElement();
388 const VarDecl* vd = 0;
389
390 if (DeclStmt* ds = dyn_cast<DeclStmt>(element)) {
391 vd = cast<VarDecl>(ds->getSingleDecl());
392 if (!isTrackedVar(vd))
393 vd = 0;
394 }
395 else {
396 // Initialize the value of the reference variable.
397 const FindVarResult &res = findBlockVarDecl(cast<Expr>(element));
398 vd = res.getDecl();
399 if (!vd) {
400 Visit(element);
401 return;
402 }
403 }
404
405 if (vd)
406 vals[vd] = Initialized;
407}
408
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000409void TransferFunctions::VisitBlockExpr(BlockExpr *be) {
410 if (!flagBlockUses || !handler)
411 return;
412 AnalysisContext::referenced_decls_iterator i, e;
413 llvm::tie(i, e) = ac.getReferencedBlockVars(be->getBlockDecl());
414 for ( ; i != e; ++i) {
415 const VarDecl *vd = *i;
Ted Kremenek40900ee2011-01-27 02:29:34 +0000416 if (vd->getAttr<BlocksAttr>() || !vd->hasLocalStorage() ||
417 !isTrackedVar(vd))
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000418 continue;
419 if (vals[vd] == Uninitialized)
Ted Kremenek40900ee2011-01-27 02:29:34 +0000420 handler->handleUseOfUninitVariable(be, vd);
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000421 }
422}
423
Ted Kremenek610068c2011-01-15 02:58:47 +0000424void TransferFunctions::VisitDeclStmt(DeclStmt *ds) {
425 for (DeclStmt::decl_iterator DI = ds->decl_begin(), DE = ds->decl_end();
426 DI != DE; ++DI) {
427 if (VarDecl *vd = dyn_cast<VarDecl>(*DI)) {
Ted Kremenek4dccb902011-01-18 05:00:42 +0000428 if (isTrackedVar(vd)) {
429 vals[vd] = Uninitialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000430 if (Stmt *init = vd->getInit()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000431 Visit(init);
Ted Kremenekc104e532011-01-18 04:53:25 +0000432 vals[vd] = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000433 }
Ted Kremenek4dccb902011-01-18 05:00:42 +0000434 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000435 else if (Stmt *init = vd->getInit()) {
436 Visit(init);
437 }
Ted Kremenek610068c2011-01-15 02:58:47 +0000438 }
439 }
440}
441
Ted Kremenekc21fed32011-01-18 21:18:58 +0000442void TransferFunctions::VisitDeclRefExpr(DeclRefExpr *dr) {
443 // We assume that DeclRefExprs wrapped in an lvalue-to-rvalue cast
444 // cannot be block-level expressions. Therefore, we determine if
445 // a DeclRefExpr is involved in a "load" by comparing it to the current
446 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
447 // If a DeclRefExpr is not involved in a load, we are essentially computing
448 // its address, either for assignment to a reference or via the '&' operator.
449 // In such cases, treat the variable as being initialized, since this
450 // analysis isn't powerful enough to do alias tracking.
451 if (dr != currentDR)
452 if (const VarDecl *vd = dyn_cast<VarDecl>(dr->getDecl()))
453 if (isTrackedVar(vd))
454 vals[vd] = Initialized;
455}
456
Ted Kremenek610068c2011-01-15 02:58:47 +0000457void TransferFunctions::VisitBinaryOperator(clang::BinaryOperator *bo) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000458 if (bo->isAssignmentOp()) {
459 const FindVarResult &res = findBlockVarDecl(bo->getLHS());
460 if (const VarDecl* vd = res.getDecl()) {
Ted Kremenekc21fed32011-01-18 21:18:58 +0000461 // We assume that DeclRefExprs wrapped in a BinaryOperator "assignment"
462 // cannot be block-level expressions. Therefore, we determine if
463 // a DeclRefExpr is involved in a "load" by comparing it to the current
464 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
465 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
466 res.getDeclRefExpr());
467 Visit(bo->getRHS());
468 Visit(bo->getLHS());
469
Ted Kremenek496398d2011-03-15 04:57:32 +0000470 ValueVector::reference val = vals[vd];
471 if (val == Uninitialized) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000472 if (bo->getOpcode() != BO_Assign)
473 reportUninit(res.getDeclRefExpr(), vd);
Ted Kremenek496398d2011-03-15 04:57:32 +0000474 val = Initialized;
Ted Kremenek610068c2011-01-15 02:58:47 +0000475 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000476 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000477 }
478 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000479 Visit(bo->getRHS());
480 Visit(bo->getLHS());
Ted Kremenek610068c2011-01-15 02:58:47 +0000481}
482
483void TransferFunctions::VisitUnaryOperator(clang::UnaryOperator *uo) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000484 switch (uo->getOpcode()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000485 case clang::UO_PostDec:
486 case clang::UO_PostInc:
487 case clang::UO_PreDec:
488 case clang::UO_PreInc: {
489 const FindVarResult &res = findBlockVarDecl(uo->getSubExpr());
490 if (const VarDecl *vd = res.getDecl()) {
Ted Kremenekc21fed32011-01-18 21:18:58 +0000491 // We assume that DeclRefExprs wrapped in a unary operator ++/--
492 // cannot be block-level expressions. Therefore, we determine if
493 // a DeclRefExpr is involved in a "load" by comparing it to the current
494 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
495 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
496 res.getDeclRefExpr());
497 Visit(uo->getSubExpr());
498
Ted Kremenek136f8f22011-03-15 04:57:27 +0000499 ValueVector::reference bit = vals[vd];
Ted Kremenek610068c2011-01-15 02:58:47 +0000500 if (bit == Uninitialized) {
501 reportUninit(res.getDeclRefExpr(), vd);
502 bit = Initialized;
503 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000504 return;
Ted Kremenek610068c2011-01-15 02:58:47 +0000505 }
506 break;
507 }
508 default:
509 break;
510 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000511 Visit(uo->getSubExpr());
Ted Kremenek610068c2011-01-15 02:58:47 +0000512}
513
514void TransferFunctions::VisitCastExpr(clang::CastExpr *ce) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000515 if (ce->getCastKind() == CK_LValueToRValue) {
516 const FindVarResult &res = findBlockVarDecl(ce->getSubExpr());
Ted Kremenekc21fed32011-01-18 21:18:58 +0000517 if (const VarDecl *vd = res.getDecl()) {
518 // We assume that DeclRefExprs wrapped in an lvalue-to-rvalue cast
519 // cannot be block-level expressions. Therefore, we determine if
520 // a DeclRefExpr is involved in a "load" by comparing it to the current
521 // DeclRefExpr found when analyzing the last lvalue-to-rvalue CastExpr.
522 // Here we update 'currentDR' to be the one associated with this
523 // lvalue-to-rvalue cast. Then, when we analyze the DeclRefExpr, we
524 // will know that we are not computing its lvalue for other purposes
525 // than to perform a load.
526 SaveAndRestore<const DeclRefExpr*> lastDR(currentDR,
527 res.getDeclRefExpr());
528 Visit(ce->getSubExpr());
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000529 if (currentVoidCast != ce && vals[vd] == Uninitialized) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000530 reportUninit(res.getDeclRefExpr(), vd);
Ted Kremenekc21fed32011-01-18 21:18:58 +0000531 // Don't cascade warnings.
532 vals[vd] = Initialized;
533 }
534 return;
535 }
Ted Kremenekdd0f7942011-01-26 04:49:43 +0000536 }
537 else if (CStyleCastExpr *cse = dyn_cast<CStyleCastExpr>(ce)) {
538 if (cse->getType()->isVoidType()) {
539 // e.g. (void) x;
540 SaveAndRestore<const Expr *>
541 lastVoidCast(currentVoidCast, cse->getSubExpr()->IgnoreParens());
542 Visit(cse->getSubExpr());
543 return;
544 }
545 }
Ted Kremenekc21fed32011-01-18 21:18:58 +0000546 Visit(ce->getSubExpr());
Ted Kremenek610068c2011-01-15 02:58:47 +0000547}
548
Peter Collingbournef4e3cfb2011-03-11 19:24:49 +0000549void TransferFunctions::VisitUnaryExprOrTypeTraitExpr(
550 UnaryExprOrTypeTraitExpr *se) {
551 if (se->getKind() == UETT_SizeOf) {
Ted Kremenek96608032011-01-23 17:53:04 +0000552 if (se->getType()->isConstantSizeType())
553 return;
554 // Handle VLAs.
555 Visit(se->getArgumentExpr());
556 }
557}
558
Ted Kremenek610068c2011-01-15 02:58:47 +0000559//------------------------------------------------------------------------====//
560// High-level "driver" logic for uninitialized values analysis.
561//====------------------------------------------------------------------------//
562
Ted Kremenek13bd4232011-01-20 17:37:17 +0000563static bool runOnBlock(const CFGBlock *block, const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000564 AnalysisContext &ac, CFGBlockValues &vals,
565 UninitVariablesHandler *handler = 0,
566 bool flagBlockUses = false) {
Ted Kremenek13bd4232011-01-20 17:37:17 +0000567
568 if (const BinaryOperator *b = getLogicalOperatorInChain(block)) {
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000569 CFGBlock::const_pred_iterator itr = block->pred_begin();
Ted Kremenek136f8f22011-03-15 04:57:27 +0000570 BVPair vA = vals.getValueVectors(*itr, false);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000571 ++itr;
Ted Kremenek136f8f22011-03-15 04:57:27 +0000572 BVPair vB = vals.getValueVectors(*itr, false);
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000573
574 BVPair valsAB;
575
576 if (b->getOpcode() == BO_LAnd) {
577 // Merge the 'F' bits from the first and second.
578 vals.mergeIntoScratch(*(vA.second ? vA.second : vA.first), true);
579 vals.mergeIntoScratch(*(vB.second ? vB.second : vB.first), false);
580 valsAB.first = vA.first;
Ted Kremenek2d4bed12011-01-20 21:25:31 +0000581 valsAB.second = &vals.getScratch();
Ted Kremenek13bd4232011-01-20 17:37:17 +0000582 }
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000583 else {
584 // Merge the 'T' bits from the first and second.
585 assert(b->getOpcode() == BO_LOr);
586 vals.mergeIntoScratch(*vA.first, true);
587 vals.mergeIntoScratch(*vB.first, false);
588 valsAB.first = &vals.getScratch();
589 valsAB.second = vA.second ? vA.second : vA.first;
590 }
Ted Kremenek136f8f22011-03-15 04:57:27 +0000591 return vals.updateValueVectors(block, valsAB);
Ted Kremenek13bd4232011-01-20 17:37:17 +0000592 }
593
Ted Kremenek9fcbcee2011-02-01 17:43:18 +0000594 // Default behavior: merge in values of predecessor blocks.
Ted Kremenek610068c2011-01-15 02:58:47 +0000595 vals.resetScratch();
596 bool isFirst = true;
597 for (CFGBlock::const_pred_iterator I = block->pred_begin(),
598 E = block->pred_end(); I != E; ++I) {
Ted Kremenek136f8f22011-03-15 04:57:27 +0000599 vals.mergeIntoScratch(vals.getValueVector(*I, block), isFirst);
Ted Kremenek610068c2011-01-15 02:58:47 +0000600 isFirst = false;
601 }
602 // Apply the transfer function.
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000603 TransferFunctions tf(vals, cfg, ac, handler, flagBlockUses);
Ted Kremenek610068c2011-01-15 02:58:47 +0000604 for (CFGBlock::const_iterator I = block->begin(), E = block->end();
605 I != E; ++I) {
606 if (const CFGStmt *cs = dyn_cast<CFGStmt>(&*I)) {
607 tf.BlockStmt_Visit(cs->getStmt());
608 }
609 }
Ted Kremenek136f8f22011-03-15 04:57:27 +0000610 return vals.updateValueVectorWithScratch(block);
Ted Kremenek610068c2011-01-15 02:58:47 +0000611}
612
613void clang::runUninitializedVariablesAnalysis(const DeclContext &dc,
614 const CFG &cfg,
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000615 AnalysisContext &ac,
Ted Kremenek610068c2011-01-15 02:58:47 +0000616 UninitVariablesHandler &handler) {
617 CFGBlockValues vals(cfg);
618 vals.computeSetOfDeclarations(dc);
619 if (vals.hasNoDeclarations())
620 return;
621 DataflowWorklist worklist(cfg);
Ted Kremenek496398d2011-03-15 04:57:32 +0000622 llvm::BitVector previouslyVisited(cfg.getNumBlockIDs());
Ted Kremenek610068c2011-01-15 02:58:47 +0000623
624 worklist.enqueueSuccessors(&cfg.getEntry());
625
626 while (const CFGBlock *block = worklist.dequeue()) {
Ted Kremenek610068c2011-01-15 02:58:47 +0000627 // Did the block change?
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000628 bool changed = runOnBlock(block, cfg, ac, vals);
Ted Kremenek610068c2011-01-15 02:58:47 +0000629 if (changed || !previouslyVisited[block->getBlockID()])
630 worklist.enqueueSuccessors(block);
631 previouslyVisited[block->getBlockID()] = true;
632 }
633
634 // Run through the blocks one more time, and report uninitialized variabes.
635 for (CFG::const_iterator BI = cfg.begin(), BE = cfg.end(); BI != BE; ++BI) {
Ted Kremeneka8c17a52011-01-25 19:13:48 +0000636 runOnBlock(*BI, cfg, ac, vals, &handler, /* flagBlockUses */ true);
Ted Kremenek610068c2011-01-15 02:58:47 +0000637 }
638}
639
640UninitVariablesHandler::~UninitVariablesHandler() {}
641