blob: ffb8cf20207b3f605bb56e4172613ca327123015 [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"
Chandler Carruth55fc8732012-12-04 09:13:33 +000016#include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
Argyrios Kyrtzidisec8605f2011-03-01 01:16:21 +000017#include "clang/StaticAnalyzer/Core/Checker.h"
Argyrios Kyrtzidis695fb502011-02-17 21:39:17 +000018#include "clang/StaticAnalyzer/Core/CheckerManager.h"
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +000019#include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.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"
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> > {
Dylan Noblesmith6f42b622012-02-05 02:12:40 +000028 mutable OwningPtr<BugType> BT_doublelock;
29 mutable OwningPtr<BugType> BT_lor;
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000030 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.
Jordan Rose166d5022012-11-02 01:54:06 +000046REGISTER_LIST_WITH_PROGRAMSTATE(LockSet, const MemRegion *)
Ted Kremenekac9bea82009-11-12 06:17:47 +000047
Ted Kremenekac9bea82009-11-12 06:17:47 +000048
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +000049void PthreadLockChecker::checkPostStmt(const CallExpr *CE,
50 CheckerContext &C) const {
Ted Kremenek8bef8232012-01-26 21:29:00 +000051 ProgramStateRef state = C.getState();
Ted Kremenek5eca4822012-01-06 22:09:28 +000052 const LocationContext *LCtx = C.getLocationContext();
Anna Zaksb805c8f2011-12-01 05:57:37 +000053 StringRef FName = C.getCalleeName(CE);
54 if (FName.empty())
Ted Kremenekac9bea82009-11-12 06:17:47 +000055 return;
Jordy Rose4cc11872011-07-19 20:31:42 +000056
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000057 if (CE->getNumArgs() != 1)
58 return;
Jordy Rose4cc11872011-07-19 20:31:42 +000059
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000060 if (FName == "pthread_mutex_lock" ||
61 FName == "pthread_rwlock_rdlock" ||
62 FName == "pthread_rwlock_wrlock")
Ted Kremenek5eca4822012-01-06 22:09:28 +000063 AcquireLock(C, CE, state->getSVal(CE->getArg(0), LCtx),
64 false, PthreadSemantics);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000065 else if (FName == "lck_mtx_lock" ||
66 FName == "lck_rw_lock_exclusive" ||
67 FName == "lck_rw_lock_shared")
Ted Kremenek5eca4822012-01-06 22:09:28 +000068 AcquireLock(C, CE, state->getSVal(CE->getArg(0), LCtx),
69 false, XNUSemantics);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000070 else if (FName == "pthread_mutex_trylock" ||
71 FName == "pthread_rwlock_tryrdlock" ||
72 FName == "pthread_rwlock_tryrwlock")
Ted Kremenek5eca4822012-01-06 22:09:28 +000073 AcquireLock(C, CE, state->getSVal(CE->getArg(0), LCtx),
74 true, PthreadSemantics);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000075 else if (FName == "lck_mtx_try_lock" ||
76 FName == "lck_rw_try_lock_exclusive" ||
77 FName == "lck_rw_try_lock_shared")
Ted Kremenek5eca4822012-01-06 22:09:28 +000078 AcquireLock(C, CE, state->getSVal(CE->getArg(0), LCtx),
79 true, XNUSemantics);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000080 else if (FName == "pthread_mutex_unlock" ||
81 FName == "pthread_rwlock_unlock" ||
82 FName == "lck_mtx_unlock" ||
83 FName == "lck_rw_done")
Ted Kremenek5eca4822012-01-06 22:09:28 +000084 ReleaseLock(C, CE, state->getSVal(CE->getArg(0), LCtx));
Ted Kremenekac9bea82009-11-12 06:17:47 +000085}
86
87void PthreadLockChecker::AcquireLock(CheckerContext &C, const CallExpr *CE,
Jordy Rosedcb1d5d2011-07-19 20:21:41 +000088 SVal lock, bool isTryLock,
89 enum LockingSemantics semantics) const {
Ted Kremenekac9bea82009-11-12 06:17:47 +000090
91 const MemRegion *lockR = lock.getAsRegion();
92 if (!lockR)
93 return;
94
Ted Kremenek8bef8232012-01-26 21:29:00 +000095 ProgramStateRef state = C.getState();
Ted Kremenekac9bea82009-11-12 06:17:47 +000096
Ted Kremenek5eca4822012-01-06 22:09:28 +000097 SVal X = state->getSVal(CE, C.getLocationContext());
Ted Kremenekac9bea82009-11-12 06:17:47 +000098 if (X.isUnknownOrUndef())
99 return;
100
David Blaikie5251abe2013-02-20 05:52:05 +0000101 DefinedSVal retVal = X.castAs<DefinedSVal>();
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000102
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000103 if (state->contains<LockSet>(lockR)) {
104 if (!BT_doublelock)
105 BT_doublelock.reset(new BugType("Double locking", "Lock checker"));
106 ExplodedNode *N = C.generateSink();
107 if (!N)
108 return;
Anna Zakse172e8b2011-08-17 23:00:25 +0000109 BugReport *report = new BugReport(*BT_doublelock,
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000110 "This lock has already "
111 "been acquired", N);
112 report->addRange(CE->getArg(0)->getSourceRange());
Jordan Rose785950e2012-11-02 01:53:40 +0000113 C.emitReport(report);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000114 return;
Ted Kremenekac9bea82009-11-12 06:17:47 +0000115 }
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000116
Ted Kremenek8bef8232012-01-26 21:29:00 +0000117 ProgramStateRef lockSucc = state;
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000118 if (isTryLock) {
119 // Bifurcate the state, and allow a mode where the lock acquisition fails.
Ted Kremenek8bef8232012-01-26 21:29:00 +0000120 ProgramStateRef lockFail;
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000121 switch (semantics) {
122 case PthreadSemantics:
123 llvm::tie(lockFail, lockSucc) = state->assume(retVal);
124 break;
125 case XNUSemantics:
126 llvm::tie(lockSucc, lockFail) = state->assume(retVal);
127 break;
128 default:
129 llvm_unreachable("Unknown tryLock locking semantics");
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000130 }
131 assert(lockFail && lockSucc);
Anna Zaks0bd6b112011-10-26 21:06:34 +0000132 C.addTransition(lockFail);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000133
134 } else if (semantics == PthreadSemantics) {
135 // Assume that the return value was 0.
Ted Kremenek28f47b92010-12-01 22:16:56 +0000136 lockSucc = state->assume(retVal, false);
Ted Kremenekac9bea82009-11-12 06:17:47 +0000137 assert(lockSucc);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000138
139 } else {
140 // XNU locking semantics return void on non-try locks
141 assert((semantics == XNUSemantics) && "Unknown locking semantics");
142 lockSucc = state;
Ted Kremenekac9bea82009-11-12 06:17:47 +0000143 }
144
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000145 // Record that the lock was acquired.
Ted Kremenekac9bea82009-11-12 06:17:47 +0000146 lockSucc = lockSucc->add<LockSet>(lockR);
Anna Zaks0bd6b112011-10-26 21:06:34 +0000147 C.addTransition(lockSucc);
Ted Kremenekac9bea82009-11-12 06:17:47 +0000148}
149
150void PthreadLockChecker::ReleaseLock(CheckerContext &C, const CallExpr *CE,
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +0000151 SVal lock) const {
Ted Kremenekac9bea82009-11-12 06:17:47 +0000152
153 const MemRegion *lockR = lock.getAsRegion();
154 if (!lockR)
155 return;
156
Ted Kremenek8bef8232012-01-26 21:29:00 +0000157 ProgramStateRef state = C.getState();
Jordan Rose166d5022012-11-02 01:54:06 +0000158 LockSetTy LS = state->get<LockSet>();
Ted Kremenekac9bea82009-11-12 06:17:47 +0000159
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000160 // FIXME: Better analysis requires IPA for wrappers.
161 // FIXME: check for double unlocks
162 if (LS.isEmpty())
Ted Kremenekac9bea82009-11-12 06:17:47 +0000163 return;
164
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000165 const MemRegion *firstLockR = LS.getHead();
166 if (firstLockR != lockR) {
167 if (!BT_lor)
168 BT_lor.reset(new BugType("Lock order reversal", "Lock checker"));
169 ExplodedNode *N = C.generateSink();
170 if (!N)
171 return;
Anna Zakse172e8b2011-08-17 23:00:25 +0000172 BugReport *report = new BugReport(*BT_lor,
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000173 "This was not the most "
174 "recently acquired lock. "
175 "Possible lock order "
176 "reversal", N);
177 report->addRange(CE->getArg(0)->getSourceRange());
Jordan Rose785950e2012-11-02 01:53:40 +0000178 C.emitReport(report);
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000179 return;
180 }
181
182 // Record that the lock was released.
183 state = state->set<LockSet>(LS.getTail());
Anna Zaks0bd6b112011-10-26 21:06:34 +0000184 C.addTransition(state);
Ted Kremenekac9bea82009-11-12 06:17:47 +0000185}
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +0000186
Jordy Rosedcb1d5d2011-07-19 20:21:41 +0000187
Argyrios Kyrtzidis983326f2011-02-23 01:05:36 +0000188void ento::registerPthreadLockChecker(CheckerManager &mgr) {
189 mgr.registerChecker<PthreadLockChecker>();
190}