diff options
author | Jordan Rose <jordan_rose@apple.com> | 2012-11-01 00:18:27 +0000 |
---|---|---|
committer | Jordan Rose <jordan_rose@apple.com> | 2012-11-01 00:18:27 +0000 |
commit | ec8d420d4fa57fc6b5a5a2b1446742e976a7ba00 (patch) | |
tree | d7d3d58808c9f042f0b0295a04fa204fd77cc1f3 /lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp | |
parent | 889b99e6d5b389c0ed99e7ad8470c28b146a42d3 (diff) |
[analyzer] Rename ConditionTruthVal::isTrue to isConstrainedTrue.
(and the same for isFalse)
No functionality change.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@167186 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp')
-rw-r--r-- | lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp b/lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp index d62002c4ab..22176fd5f2 100644 --- a/lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp +++ b/lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp @@ -140,16 +140,19 @@ void SimpleStreamChecker::checkDeadSymbols(SymbolReaper &SymReaper, CheckerContext &C) const { ProgramStateRef State = C.getState(); StreamMapTy TrackedStreams = State->get<StreamMap>(); + SymbolVector LeakedStreams; for (StreamMapTy::iterator I = TrackedStreams.begin(), - E = TrackedStreams.end(); I != E; ++I) { + E = TrackedStreams.end(); I != E; ++I) { SymbolRef Sym = I->first; if (SymReaper.isDead(Sym)) { const StreamState &SS = I->second; if (SS.isOpened()) { - // If a symbolic region is NULL, assume that allocation failed on - // this path and do not report a leak. - if (!State->getConstraintManager().isNull(State, Sym).isTrue()) + // If a symbol is NULL, assume that fopen failed on this path + // and do not report a leak. + ConstraintManager &CMgr = State->getConstraintManager(); + ConditionTruthVal OpenFailed = CMgr.isNull(State, Sym); + if (!OpenFailed.isConstrainedTrue()) LeakedStreams.push_back(Sym); } |