-
Notifications
You must be signed in to change notification settings - Fork 12.4k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add a testcase to prevent regressions
- Loading branch information
Showing
1 changed file
with
31 additions
and
0 deletions.
There are no files selected for viewing
31 changes: 31 additions & 0 deletions
31
clang/test/Analysis/block-in-critical-section-inheritance.cpp
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
// RUN: %clang_analyze_cc1 \ | ||
// RUN: -analyzer-checker=unix.BlockInCriticalSection \ | ||
// RUN: -std=c++11 \ | ||
// RUN: -analyzer-output text \ | ||
// RUN: -verify %s | ||
|
||
unsigned int sleep(unsigned int seconds) {return 0;} | ||
namespace std { | ||
// There are some standard library implementations where some mutex methods | ||
// come from an implementation detail base class. We need to ensure that these | ||
// are matched correctly. | ||
class __mutex_base { | ||
public: | ||
void lock(); | ||
}; | ||
class mutex : public __mutex_base{ | ||
public: | ||
void unlock(); | ||
bool try_lock(); | ||
}; | ||
} // namespace std | ||
|
||
void gh_99628() { | ||
std::mutex m; | ||
m.lock(); | ||
// expected-note@-1 {{Entering critical section here}} | ||
sleep(10); | ||
// expected-warning@-1 {{Call to blocking function 'sleep' inside of critical section}} | ||
// expected-note@-2 {{Call to blocking function 'sleep' inside of critical section}} | ||
m.unlock(); | ||
} |