aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAnna Zaks <ganna@apple.com>2011-12-17 00:26:29 +0000
committerAnna Zaks <ganna@apple.com>2011-12-17 00:26:29 +0000
commit777d706547ebc751d998134774d9d5388fff8e02 (patch)
tree16680ccbf53dad15204f9469ff8fefe884528b28
parentb8786c4dc4d5a4c72f23a2d46cac5f9bc2641926 (diff)
[analyzer] Minor: Simplify & assert.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@146792 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r--include/clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h5
-rw-r--r--lib/StaticAnalyzer/Core/ProgramState.cpp4
2 files changed, 6 insertions, 3 deletions
diff --git a/include/clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h b/include/clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h
index 02c92c9150..b86f4e119d 100644
--- a/include/clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h
+++ b/include/clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h
@@ -34,7 +34,10 @@ public:
: Eng(eng),
Pred(pred),
Location(loc),
- NB(builder) {}
+ NB(builder) {
+ assert(Pred->getState() &&
+ "We should not call the checkers on an empty state.");
+ }
AnalysisManager &getAnalysisManager() {
return Eng.getAnalysisManager();
diff --git a/lib/StaticAnalyzer/Core/ProgramState.cpp b/lib/StaticAnalyzer/Core/ProgramState.cpp
index 76c25f2226..d4b31c1f51 100644
--- a/lib/StaticAnalyzer/Core/ProgramState.cpp
+++ b/lib/StaticAnalyzer/Core/ProgramState.cpp
@@ -692,8 +692,8 @@ bool ProgramState::isTainted(const Stmt *S, TaintTagType Kind) const {
bool ProgramState::isTainted(SVal V, TaintTagType Kind) const {
if (const SymExpr *Sym = V.getAsSymExpr())
return isTainted(Sym, Kind);
- if (loc::MemRegionVal *RegVal = dyn_cast<loc::MemRegionVal>(&V))
- return isTainted(RegVal->getRegion(), Kind);
+ if (const MemRegion *Reg = V.getAsRegion())
+ return isTainted(Reg, Kind);
return false;
}