diff options
author | DeLesley Hutchins <delesley@google.com> | 2013-11-06 18:40:01 +0000 |
---|---|---|
committer | DeLesley Hutchins <delesley@google.com> | 2013-11-06 18:40:01 +0000 |
commit | 05b7b37021b3601ff890e3697bfb9ba905a765e0 (patch) | |
tree | 1796ee48b89f2b1f20c0966a841f6da1e3aea34f /clang/lib | |
parent | f682f5301212c58ca501cb4092a1dd59c43b80ea (diff) | |
download | bcm5719-llvm-05b7b37021b3601ff890e3697bfb9ba905a765e0.tar.gz bcm5719-llvm-05b7b37021b3601ff890e3697bfb9ba905a765e0.zip |
Thread safety analysis: minor bugfix to smart pointer handling, and expanded
test case.
llvm-svn: 194157
Diffstat (limited to 'clang/lib')
-rw-r--r-- | clang/lib/Analysis/ThreadSafety.cpp | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/clang/lib/Analysis/ThreadSafety.cpp b/clang/lib/Analysis/ThreadSafety.cpp index 607526db3e5..df163aaf6d6 100644 --- a/clang/lib/Analysis/ThreadSafety.cpp +++ b/clang/lib/Analysis/ThreadSafety.cpp @@ -2137,14 +2137,15 @@ void BuildLockset::VisitCallExpr(CallExpr *Exp) { case OO_Star: case OO_Arrow: { if (Analyzer->Handler.issueBetaWarnings()) { - const Expr *Target = OE->getArg(0); - checkPtAccess(Target, AK_Read); + const Expr *Obj = OE->getArg(0); + checkAccess(Obj, AK_Read); + checkPtAccess(Obj, AK_Read); } break; } default: { - const Expr *Source = OE->getArg(0); - checkAccess(Source, AK_Read); + const Expr *Obj = OE->getArg(0); + checkAccess(Obj, AK_Read); break; } } |