blob: a722c4938525343d8cf29291f72a765703b02b46 [file] [log] [blame]
Jordy Rosedcb1d5d2011-07-19 20:21:41 +00001//===--- PthreadLockChecker.cpp - Check for locking problems ---*- C++ -*--===//
Ted Kremenekac9bea82009-11-12 06:17:47 +00002//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
Jordy Rose4cc11872011-07-19 20:31:42 +000010// This defines PthreadLockChecker, a simple lock -> unlock checker.
11// Also handles XNU locks, which behave similarly enough to share code.
Ted Kremenekac9bea82009-11-12 06:17:47 +000012//
13//===----------------------------------------------------------------------===//
14
Argyrios Kyrtzidisa0decc92011-02-15 21:25:03 +000015#include "ClangSACheckers.h"
Argyrios Kyrtzidisec8605f2011-03-01 01:16:21 +000016#include "clang/StaticAnalyzer/Core/Checker.h"
Argyrios Kyrtzidis695fb502011-02-17 21:39:17 +000017#include "clang/StaticAnalyzer/Core/CheckerManager.h"
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +000018#include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000019#include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
Ted Kremenek9b663712011-02-10 01:03:03 +000020#include "clang/StaticAnalyzer/Core/PathSensitive/GRStateTrait.h"
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000021#include "llvm/ADT/ImmutableList.h"
Ted Kremenekac9bea82009-11-12 06:17:47 +000022
23using namespace clang;
Ted Kremenek9ef65372010-12-23 07:20:52 +000024using namespace ento;
Ted Kremenekac9bea82009-11-12 06:17:47 +000025
26namespace {
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000027class PthreadLockChecker : public Checker< check::PostStmt<CallExpr> > {
28 mutable llvm::OwningPtr<BugType> BT_doublelock;
29 mutable llvm::OwningPtr<BugType> BT_lor;
30 enum LockingSemantics {
31 NotApplicable = 0,
32 PthreadSemantics,
33 XNUSemantics
34 };
Ted Kremenekac9bea82009-11-12 06:17:47 +000035public:
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +000036 void checkPostStmt(const CallExpr *CE, CheckerContext &C) const;
Ted Kremenekac9bea82009-11-12 06:17:47 +000037
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000038 void AcquireLock(CheckerContext &C, const CallExpr *CE, SVal lock,
39 bool isTryLock, enum LockingSemantics semantics) const;
Ted Kremenekac9bea82009-11-12 06:17:47 +000040
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000041 void ReleaseLock(CheckerContext &C, const CallExpr *CE, SVal lock) const;
Ted Kremenekac9bea82009-11-12 06:17:47 +000042};
43} // end anonymous namespace
44
45// GDM Entry for tracking lock state.
Kovarththanan Rajaratnamba5fb5a2009-11-28 06:07:30 +000046namespace { class LockSet {}; }
Ted Kremenekac9bea82009-11-12 06:17:47 +000047namespace clang {
Ted Kremenek9ef65372010-12-23 07:20:52 +000048namespace ento {
Ted Kremenekac9bea82009-11-12 06:17:47 +000049template <> struct GRStateTrait<LockSet> :
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000050 public GRStatePartialTrait<llvm::ImmutableList<const MemRegion*> > {
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +000051 static void* GDMIndex() { static int x = 0; return &x; }
Ted Kremenekac9bea82009-11-12 06:17:47 +000052};
Argyrios Kyrtzidis5a4f98f2010-12-22 18:53:20 +000053} // end GR namespace
Ted Kremenekac9bea82009-11-12 06:17:47 +000054} // end clang namespace
55
Ted Kremenekac9bea82009-11-12 06:17:47 +000056
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +000057void PthreadLockChecker::checkPostStmt(const CallExpr *CE,
58 CheckerContext &C) const {
Ted Kremenekac9bea82009-11-12 06:17:47 +000059 const GRState *state = C.getState();
60 const Expr *Callee = CE->getCallee();
Jordy Rose4cc11872011-07-19 20:31:42 +000061 const FunctionDecl *FD = state->getSVal(Callee).getAsFunctionDecl();
62
63 if (!FD)
Ted Kremenekac9bea82009-11-12 06:17:47 +000064 return;
Jordy Rose4cc11872011-07-19 20:31:42 +000065
66 // Get the name of the callee.
67 IdentifierInfo *II = FD->getIdentifier();
Douglas Gregor90d26a42010-11-01 23:16:05 +000068 if (!II) // if no identifier, not a simple C function
69 return;
70 llvm::StringRef FName = II->getName();
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000071
72 if (CE->getNumArgs() != 1)
73 return;
Jordy Rose4cc11872011-07-19 20:31:42 +000074
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000075 if (FName == "pthread_mutex_lock" ||
76 FName == "pthread_rwlock_rdlock" ||
77 FName == "pthread_rwlock_wrlock")
78 AcquireLock(C, CE, state->getSVal(CE->getArg(0)), false, PthreadSemantics);
79 else if (FName == "lck_mtx_lock" ||
80 FName == "lck_rw_lock_exclusive" ||
81 FName == "lck_rw_lock_shared")
82 AcquireLock(C, CE, state->getSVal(CE->getArg(0)), false, XNUSemantics);
83 else if (FName == "pthread_mutex_trylock" ||
84 FName == "pthread_rwlock_tryrdlock" ||
85 FName == "pthread_rwlock_tryrwlock")
86 AcquireLock(C, CE, state->getSVal(CE->getArg(0)), true, PthreadSemantics);
87 else if (FName == "lck_mtx_try_lock" ||
88 FName == "lck_rw_try_lock_exclusive" ||
89 FName == "lck_rw_try_lock_shared")
90 AcquireLock(C, CE, state->getSVal(CE->getArg(0)), true, XNUSemantics);
91 else if (FName == "pthread_mutex_unlock" ||
92 FName == "pthread_rwlock_unlock" ||
93 FName == "lck_mtx_unlock" ||
94 FName == "lck_rw_done")
Ted Kremenek13976632010-02-08 16:18:51 +000095 ReleaseLock(C, CE, state->getSVal(CE->getArg(0)));
Ted Kremenekac9bea82009-11-12 06:17:47 +000096}
97
98void PthreadLockChecker::AcquireLock(CheckerContext &C, const CallExpr *CE,
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000099 SVal lock, bool isTryLock,
100 enum LockingSemantics semantics) const {
Ted Kremenekac9bea82009-11-12 06:17:47 +0000101
102 const MemRegion *lockR = lock.getAsRegion();
103 if (!lockR)
104 return;
105
106 const GRState *state = C.getState();
107
Ted Kremenek13976632010-02-08 16:18:51 +0000108 SVal X = state->getSVal(CE);
Ted Kremenekac9bea82009-11-12 06:17:47 +0000109 if (X.isUnknownOrUndef())
110 return;
111
112 DefinedSVal retVal = cast<DefinedSVal>(X);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000113
114 llvm::ImmutableList<const MemRegion*> LS = state->get<LockSet>();
115
116 if (state->contains<LockSet>(lockR)) {
117 if (!BT_doublelock)
118 BT_doublelock.reset(new BugType("Double locking", "Lock checker"));
119 ExplodedNode *N = C.generateSink();
120 if (!N)
121 return;
122 EnhancedBugReport *report = new EnhancedBugReport(*BT_doublelock,
123 "This lock has already "
124 "been acquired", N);
125 report->addRange(CE->getArg(0)->getSourceRange());
126 C.EmitReport(report);
127 return;
Ted Kremenekac9bea82009-11-12 06:17:47 +0000128 }
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000129
130 const GRState *lockSucc = state;
131 if (isTryLock) {
132 // Bifurcate the state, and allow a mode where the lock acquisition fails.
133 const GRState *lockFail;
134 switch (semantics) {
135 case PthreadSemantics:
136 llvm::tie(lockFail, lockSucc) = state->assume(retVal);
137 break;
138 case XNUSemantics:
139 llvm::tie(lockSucc, lockFail) = state->assume(retVal);
140 break;
141 default:
142 llvm_unreachable("Unknown tryLock locking semantics");
143 break;
144 }
145 assert(lockFail && lockSucc);
146 C.addTransition(lockFail);
147
148 } else if (semantics == PthreadSemantics) {
149 // Assume that the return value was 0.
Ted Kremenek28f47b92010-12-01 22:16:56 +0000150 lockSucc = state->assume(retVal, false);
Ted Kremenekac9bea82009-11-12 06:17:47 +0000151 assert(lockSucc);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000152
153 } else {
154 // XNU locking semantics return void on non-try locks
155 assert((semantics == XNUSemantics) && "Unknown locking semantics");
156 lockSucc = state;
Ted Kremenekac9bea82009-11-12 06:17:47 +0000157 }
158
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000159 // Record that the lock was acquired.
Ted Kremenekac9bea82009-11-12 06:17:47 +0000160 lockSucc = lockSucc->add<LockSet>(lockR);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000161 C.addTransition(lockSucc);
Ted Kremenekac9bea82009-11-12 06:17:47 +0000162}
163
164void PthreadLockChecker::ReleaseLock(CheckerContext &C, const CallExpr *CE,
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +0000165 SVal lock) const {
Ted Kremenekac9bea82009-11-12 06:17:47 +0000166
167 const MemRegion *lockR = lock.getAsRegion();
168 if (!lockR)
169 return;
170
171 const GRState *state = C.getState();
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000172 llvm::ImmutableList<const MemRegion*> LS = state->get<LockSet>();
Ted Kremenekac9bea82009-11-12 06:17:47 +0000173
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000174 // FIXME: Better analysis requires IPA for wrappers.
175 // FIXME: check for double unlocks
176 if (LS.isEmpty())
Ted Kremenekac9bea82009-11-12 06:17:47 +0000177 return;
178
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000179 const MemRegion *firstLockR = LS.getHead();
180 if (firstLockR != lockR) {
181 if (!BT_lor)
182 BT_lor.reset(new BugType("Lock order reversal", "Lock checker"));
183 ExplodedNode *N = C.generateSink();
184 if (!N)
185 return;
186 EnhancedBugReport *report = new EnhancedBugReport(*BT_lor,
187 "This was not the most "
188 "recently acquired lock. "
189 "Possible lock order "
190 "reversal", N);
191 report->addRange(CE->getArg(0)->getSourceRange());
192 C.EmitReport(report);
193 return;
194 }
195
196 // Record that the lock was released.
197 state = state->set<LockSet>(LS.getTail());
198 C.addTransition(state);
Ted Kremenekac9bea82009-11-12 06:17:47 +0000199}
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +0000200
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000201
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +0000202void ento::registerPthreadLockChecker(CheckerManager &mgr) {
203 mgr.registerChecker<PthreadLockChecker>();
204}