diff options
author | Ted Kremenek <kremenek@apple.com> | 2009-10-06 01:39:48 +0000 |
---|---|---|
committer | Ted Kremenek <kremenek@apple.com> | 2009-10-06 01:39:48 +0000 |
commit | cd8f6ac9b613e1fe962ebf9c87d822ce765275e6 (patch) | |
tree | 9571f28a057b3d9bc021e9d6af2ab420fe7da2e6 /lib/Analysis/GRExprEngine.cpp | |
parent | 2e9f652d53346bf7e64c8a12a9ff06b004a3e489 (diff) |
Fix: <rdar://problem/7275774> Static analyzer warns about NULL pointer when
adding assert
This fix required a few changes:
SimpleSValuator:
- Eagerly replace a symbolic value with its constant value in EvalBinOpNN
when it is constrained to a constant. This allows us to better constant fold
values along a path.
- Handle trivial case of '<', '>' comparison of pointers when the two pointers
are exactly the same.
RegionStoreManager:
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@83358 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Analysis/GRExprEngine.cpp')
-rw-r--r-- | lib/Analysis/GRExprEngine.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/Analysis/GRExprEngine.cpp b/lib/Analysis/GRExprEngine.cpp index dc39d8b041..8de200cb1e 100644 --- a/lib/Analysis/GRExprEngine.cpp +++ b/lib/Analysis/GRExprEngine.cpp @@ -2545,7 +2545,7 @@ void GRExprEngine::VisitUnaryOperator(UnaryOperator* U, ExplodedNode* Pred, } else { nonloc::ConcreteInt X(getBasicVals().getValue(0, Ex->getType())); - Result = EvalBinOp(BinaryOperator::EQ, cast<NonLoc>(V), X, + Result = EvalBinOp(state, BinaryOperator::EQ, cast<NonLoc>(V), X, U->getType()); } |