aboutsummaryrefslogtreecommitdiff
path: root/lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp
diff options
context:
space:
mode:
authorJordan Rose <jordan_rose@apple.com>2012-11-01 00:18:27 +0000
committerJordan Rose <jordan_rose@apple.com>2012-11-01 00:18:27 +0000
commitec8d420d4fa57fc6b5a5a2b1446742e976a7ba00 (patch)
treed7d3d58808c9f042f0b0295a04fa204fd77cc1f3 /lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp
parent889b99e6d5b389c0ed99e7ad8470c28b146a42d3 (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.cpp11
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);
}