diff options
| author | Ted Kremenek <kremenek@apple.com> | 2010-08-26 22:19:33 +0000 |
|---|---|---|
| committer | Ted Kremenek <kremenek@apple.com> | 2010-08-26 22:19:33 +0000 |
| commit | 34feff654c6304e0a59ceb1376989d28dbc956ff (patch) | |
| tree | 9e8d481dc6ce771979bfd42559f1d68e7f75d588 /lib/Checker/GRExprEngine.cpp | |
| parent | 412711774cf912bac8e81ef86da33acac6856b8a (diff) | |
Fix horrible GRExprEngine bug where switch statements with no 'case:' statements would cause the path to get prematurely aborted. Fixes <rdar://problem/8360854>.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@112233 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Checker/GRExprEngine.cpp')
| -rw-r--r-- | lib/Checker/GRExprEngine.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/lib/Checker/GRExprEngine.cpp b/lib/Checker/GRExprEngine.cpp index 3578483a82..c9173aa92a 100644 --- a/lib/Checker/GRExprEngine.cpp +++ b/lib/Checker/GRExprEngine.cpp @@ -1489,9 +1489,11 @@ void GRExprEngine::ProcessSwitch(GRSwitchNodeBuilder& builder) { DefinedOrUnknownSVal CondV = cast<DefinedOrUnknownSVal>(CondV_untested); const GRState *DefaultSt = state; - bool defaultIsFeasible = false; + + iterator I = builder.begin(), EI = builder.end(); + bool defaultIsFeasible = I == EI; - for (iterator I = builder.begin(), EI = builder.end(); I != EI; ++I) { + for ( ; I != EI; ++I) { const CaseStmt* Case = I.getCase(); // Evaluate the LHS of the case value. |