tree 554239d76d4ebd30ffba6609b0f0a63e51704dfb
parent e4a0a67c425eec93c924d83edb446572f9ab4277
author Wyatt Hepler <hepler@google.com> 1641536426 -0800
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> 1641923508 +0000

pw_{rpc, sync_stl}: Assert if STL mutex is misused

- Update the pw_sync_stl mutex to track whether it is locked and assert
  if it's used improperly. The C++ standard states that unlocking an
  unlocked mutex is undefined behavior. Neither libc++ nor libstdc++
  crash when a mutex is unlocked multiple times (at least on Linux).
- Annotate the stub pw::rpc::internal::LockGuard class with thread
  safety annotations so that thread safety analysis applies when
  compiling without RPC locking.

Change-Id: Ibcd77e90aedafaa5aa05016e9657f2b6983104a2
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/77504
Pigweed-Auto-Submit: Wyatt Hepler <hepler@google.com>
Reviewed-by: Keir Mierle <keir@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed.google.com.iam.gserviceaccount.com>
Commit-Queue: Wyatt Hepler <hepler@google.com>