blob: f6f01b81a25d40099d79609fa7745eb7b16bcaaa [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 Kremenek18c66fd2011-08-15 22:09:50 +000020#include "clang/StaticAnalyzer/Core/PathSensitive/ProgramStateTrait.h"
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000021#include "llvm/ADT/ImmutableList.h"
Benjamin Kramer00bd44d2012-02-04 12:31:12 +000022#include "llvm/ADT/STLExtras.h"
Ted Kremenekac9bea82009-11-12 06:17:47 +000023
24using namespace clang;
Ted Kremenek9ef65372010-12-23 07:20:52 +000025using namespace ento;
Ted Kremenekac9bea82009-11-12 06:17:47 +000026
27namespace {
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000028class PthreadLockChecker : public Checker< check::PostStmt<CallExpr> > {
Dylan Noblesmith6f42b622012-02-05 02:12:40 +000029 mutable OwningPtr<BugType> BT_doublelock;
30 mutable OwningPtr<BugType> BT_lor;
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000031 enum LockingSemantics {
32 NotApplicable = 0,
33 PthreadSemantics,
34 XNUSemantics
35 };
Ted Kremenekac9bea82009-11-12 06:17:47 +000036public:
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +000037 void checkPostStmt(const CallExpr *CE, CheckerContext &C) const;
Ted Kremenekac9bea82009-11-12 06:17:47 +000038
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000039 void AcquireLock(CheckerContext &C, const CallExpr *CE, SVal lock,
40 bool isTryLock, enum LockingSemantics semantics) const;
Ted Kremenekac9bea82009-11-12 06:17:47 +000041
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000042 void ReleaseLock(CheckerContext &C, const CallExpr *CE, SVal lock) const;
Ted Kremenekac9bea82009-11-12 06:17:47 +000043};
44} // end anonymous namespace
45
46// GDM Entry for tracking lock state.
Kovarththanan Rajaratnamba5fb5a2009-11-28 06:07:30 +000047namespace { class LockSet {}; }
Ted Kremenekac9bea82009-11-12 06:17:47 +000048namespace clang {
Ted Kremenek9ef65372010-12-23 07:20:52 +000049namespace ento {
Ted Kremenek18c66fd2011-08-15 22:09:50 +000050template <> struct ProgramStateTrait<LockSet> :
51 public ProgramStatePartialTrait<llvm::ImmutableList<const MemRegion*> > {
Ted Kremenek9c378f72011-08-12 23:37:29 +000052 static void *GDMIndex() { static int x = 0; return &x; }
Ted Kremenekac9bea82009-11-12 06:17:47 +000053};
Benjamin Kramerd25be9f2011-09-24 03:04:03 +000054} // end of ento (ProgramState) namespace
Ted Kremenekac9bea82009-11-12 06:17:47 +000055} // end clang namespace
56
Ted Kremenekac9bea82009-11-12 06:17:47 +000057
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +000058void PthreadLockChecker::checkPostStmt(const CallExpr *CE,
59 CheckerContext &C) const {
Ted Kremenek8bef8232012-01-26 21:29:00 +000060 ProgramStateRef state = C.getState();
Ted Kremenek5eca4822012-01-06 22:09:28 +000061 const LocationContext *LCtx = C.getLocationContext();
Anna Zaksb805c8f2011-12-01 05:57:37 +000062 StringRef FName = C.getCalleeName(CE);
63 if (FName.empty())
Ted Kremenekac9bea82009-11-12 06:17:47 +000064 return;
Jordy Rose4cc11872011-07-19 20:31:42 +000065
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000066 if (CE->getNumArgs() != 1)
67 return;
Jordy Rose4cc11872011-07-19 20:31:42 +000068
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000069 if (FName == "pthread_mutex_lock" ||
70 FName == "pthread_rwlock_rdlock" ||
71 FName == "pthread_rwlock_wrlock")
Ted Kremenek5eca4822012-01-06 22:09:28 +000072 AcquireLock(C, CE, state->getSVal(CE->getArg(0), LCtx),
73 false, PthreadSemantics);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000074 else if (FName == "lck_mtx_lock" ||
75 FName == "lck_rw_lock_exclusive" ||
76 FName == "lck_rw_lock_shared")
Ted Kremenek5eca4822012-01-06 22:09:28 +000077 AcquireLock(C, CE, state->getSVal(CE->getArg(0), LCtx),
78 false, XNUSemantics);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000079 else if (FName == "pthread_mutex_trylock" ||
80 FName == "pthread_rwlock_tryrdlock" ||
81 FName == "pthread_rwlock_tryrwlock")
Ted Kremenek5eca4822012-01-06 22:09:28 +000082 AcquireLock(C, CE, state->getSVal(CE->getArg(0), LCtx),
83 true, PthreadSemantics);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000084 else if (FName == "lck_mtx_try_lock" ||
85 FName == "lck_rw_try_lock_exclusive" ||
86 FName == "lck_rw_try_lock_shared")
Ted Kremenek5eca4822012-01-06 22:09:28 +000087 AcquireLock(C, CE, state->getSVal(CE->getArg(0), LCtx),
88 true, XNUSemantics);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000089 else if (FName == "pthread_mutex_unlock" ||
90 FName == "pthread_rwlock_unlock" ||
91 FName == "lck_mtx_unlock" ||
92 FName == "lck_rw_done")
Ted Kremenek5eca4822012-01-06 22:09:28 +000093 ReleaseLock(C, CE, state->getSVal(CE->getArg(0), LCtx));
Ted Kremenekac9bea82009-11-12 06:17:47 +000094}
95
96void PthreadLockChecker::AcquireLock(CheckerContext &C, const CallExpr *CE,
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000097 SVal lock, bool isTryLock,
98 enum LockingSemantics semantics) const {
Ted Kremenekac9bea82009-11-12 06:17:47 +000099
100 const MemRegion *lockR = lock.getAsRegion();
101 if (!lockR)
102 return;
103
Ted Kremenek8bef8232012-01-26 21:29:00 +0000104 ProgramStateRef state = C.getState();
Ted Kremenekac9bea82009-11-12 06:17:47 +0000105
Ted Kremenek5eca4822012-01-06 22:09:28 +0000106 SVal X = state->getSVal(CE, C.getLocationContext());
Ted Kremenekac9bea82009-11-12 06:17:47 +0000107 if (X.isUnknownOrUndef())
108 return;
109
110 DefinedSVal retVal = cast<DefinedSVal>(X);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000111
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000112 if (state->contains<LockSet>(lockR)) {
113 if (!BT_doublelock)
114 BT_doublelock.reset(new BugType("Double locking", "Lock checker"));
115 ExplodedNode *N = C.generateSink();
116 if (!N)
117 return;
Anna Zakse172e8b2011-08-17 23:00:25 +0000118 BugReport *report = new BugReport(*BT_doublelock,
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000119 "This lock has already "
120 "been acquired", N);
121 report->addRange(CE->getArg(0)->getSourceRange());
122 C.EmitReport(report);
123 return;
Ted Kremenekac9bea82009-11-12 06:17:47 +0000124 }
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000125
Ted Kremenek8bef8232012-01-26 21:29:00 +0000126 ProgramStateRef lockSucc = state;
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000127 if (isTryLock) {
128 // Bifurcate the state, and allow a mode where the lock acquisition fails.
Ted Kremenek8bef8232012-01-26 21:29:00 +0000129 ProgramStateRef lockFail;
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000130 switch (semantics) {
131 case PthreadSemantics:
132 llvm::tie(lockFail, lockSucc) = state->assume(retVal);
133 break;
134 case XNUSemantics:
135 llvm::tie(lockSucc, lockFail) = state->assume(retVal);
136 break;
137 default:
138 llvm_unreachable("Unknown tryLock locking semantics");
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000139 }
140 assert(lockFail && lockSucc);
Anna Zaks0bd6b112011-10-26 21:06:34 +0000141 C.addTransition(lockFail);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000142
143 } else if (semantics == PthreadSemantics) {
144 // Assume that the return value was 0.
Ted Kremenek28f47b92010-12-01 22:16:56 +0000145 lockSucc = state->assume(retVal, false);
Ted Kremenekac9bea82009-11-12 06:17:47 +0000146 assert(lockSucc);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000147
148 } else {
149 // XNU locking semantics return void on non-try locks
150 assert((semantics == XNUSemantics) && "Unknown locking semantics");
151 lockSucc = state;
Ted Kremenekac9bea82009-11-12 06:17:47 +0000152 }
153
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000154 // Record that the lock was acquired.
Ted Kremenekac9bea82009-11-12 06:17:47 +0000155 lockSucc = lockSucc->add<LockSet>(lockR);
Anna Zaks0bd6b112011-10-26 21:06:34 +0000156 C.addTransition(lockSucc);
Ted Kremenekac9bea82009-11-12 06:17:47 +0000157}
158
159void PthreadLockChecker::ReleaseLock(CheckerContext &C, const CallExpr *CE,
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +0000160 SVal lock) const {
Ted Kremenekac9bea82009-11-12 06:17:47 +0000161
162 const MemRegion *lockR = lock.getAsRegion();
163 if (!lockR)
164 return;
165
Ted Kremenek8bef8232012-01-26 21:29:00 +0000166 ProgramStateRef state = C.getState();
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000167 llvm::ImmutableList<const MemRegion*> LS = state->get<LockSet>();
Ted Kremenekac9bea82009-11-12 06:17:47 +0000168
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000169 // FIXME: Better analysis requires IPA for wrappers.
170 // FIXME: check for double unlocks
171 if (LS.isEmpty())
Ted Kremenekac9bea82009-11-12 06:17:47 +0000172 return;
173
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000174 const MemRegion *firstLockR = LS.getHead();
175 if (firstLockR != lockR) {
176 if (!BT_lor)
177 BT_lor.reset(new BugType("Lock order reversal", "Lock checker"));
178 ExplodedNode *N = C.generateSink();
179 if (!N)
180 return;
Anna Zakse172e8b2011-08-17 23:00:25 +0000181 BugReport *report = new BugReport(*BT_lor,
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000182 "This was not the most "
183 "recently acquired lock. "
184 "Possible lock order "
185 "reversal", N);
186 report->addRange(CE->getArg(0)->getSourceRange());
187 C.EmitReport(report);
188 return;
189 }
190
191 // Record that the lock was released.
192 state = state->set<LockSet>(LS.getTail());
Anna Zaks0bd6b112011-10-26 21:06:34 +0000193 C.addTransition(state);
Ted Kremenekac9bea82009-11-12 06:17:47 +0000194}
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +0000195
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000196
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +0000197void ento::registerPthreadLockChecker(CheckerManager &mgr) {
198 mgr.registerChecker<PthreadLockChecker>();
199}