diff options
author | Ted Kremenek <kremenek@apple.com> | 2011-10-06 20:53:28 +0000 |
---|---|---|
committer | Ted Kremenek <kremenek@apple.com> | 2011-10-06 20:53:28 +0000 |
commit | b7531d622daea80369988084231f746fc0d18466 (patch) | |
tree | 014e51e575b009854564c9109d26f256011a1b3e /clang/lib/Analysis/LiveVariables.cpp | |
parent | 89c11337ba461243f4cdb8eddc7237f9f0bccbc2 (diff) | |
download | bcm5719-llvm-b7531d622daea80369988084231f746fc0d18466.tar.gz bcm5719-llvm-b7531d622daea80369988084231f746fc0d18466.zip |
[static analyzer] Fix crash in LiveVariables and Environment::getSVal() when analyzing C++ pointer-to-member calls. Fixes <rdar://problem/10243398>.
llvm-svn: 141312
Diffstat (limited to 'clang/lib/Analysis/LiveVariables.cpp')
-rw-r--r-- | clang/lib/Analysis/LiveVariables.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/clang/lib/Analysis/LiveVariables.cpp b/clang/lib/Analysis/LiveVariables.cpp index be6e659bd99..c800c70a38f 100644 --- a/clang/lib/Analysis/LiveVariables.cpp +++ b/clang/lib/Analysis/LiveVariables.cpp @@ -352,9 +352,10 @@ void TransferFunctions::Visit(Stmt *S) { case Stmt::CXXMemberCallExprClass: { // Include the implicit "this" pointer as being live. CXXMemberCallExpr *CE = cast<CXXMemberCallExpr>(S); - val.liveStmts = - LV.SSetFact.add(val.liveStmts, - CE->getImplicitObjectArgument()->IgnoreParens()); + if (Expr *ImplicitObj = CE->getImplicitObjectArgument()) { + ImplicitObj = ImplicitObj->IgnoreParens(); + val.liveStmts = LV.SSetFact.add(val.liveStmts, ImplicitObj); + } break; } case Stmt::DeclStmtClass: { |