From 6f6131815e2cedac66bf63d918d36da38526ccbe Mon Sep 17 00:00:00 2001 From: Benjamin Kramer Date: Mon, 12 Jul 2021 17:55:49 +0200 Subject: [PATCH] sanitizer_common: Suppress another thread safety warning Another follow-up to 0da172b1766e1559ef677aa836dce4f1995dfef3 compiler-rt/lib/scudo/scudo_tsd_shared.cpp:103:1: error: mutex 'CandidateTSD->Mutex' is not held on every path through here [-Werror,-Wthread-safety-analysis] } ^ compiler-rt/lib/scudo/scudo_tsd_shared.cpp:95:21: note: mutex acquired here CandidateTSD->lock(); ^ compiler-rt/lib/scudo/scudo_tsd_shared.cpp:103:1: error: mutex 'TSD->Mutex' is not held on every path through here [-Werror,-Wthread-safety-analysis] } ^ compiler-rt/lib/scudo/scudo_tsd_shared.cpp:101:8: note: mutex acquired here TSD->lock(); ^ compiler-rt/lib/scudo/scudo_tsd_shared.cpp:103:1: error: mutex 'TSDs[Index].Mutex' is not held on every path through here [-Werror,-Wthread-safety-analysis] } ^ compiler-rt/lib/scudo/scudo_tsd_shared.cpp:80:23: note: mutex acquired here if (TSDs[Index].tryLock()) { ^ --- compiler-rt/lib/scudo/scudo_tsd_shared.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/compiler-rt/lib/scudo/scudo_tsd_shared.cpp b/compiler-rt/lib/scudo/scudo_tsd_shared.cpp index 59ad254..fd85a7c 100644 --- a/compiler-rt/lib/scudo/scudo_tsd_shared.cpp +++ b/compiler-rt/lib/scudo/scudo_tsd_shared.cpp @@ -64,7 +64,7 @@ void initThread(bool MinimalInit) { setCurrentTSD(&TSDs[Index % NumberOfTSDs]); } -ScudoTSD *getTSDAndLockSlow(ScudoTSD *TSD) { +ScudoTSD *getTSDAndLockSlow(ScudoTSD *TSD) NO_THREAD_SAFETY_ANALYSIS { if (NumberOfTSDs > 1) { // Use the Precedence of the current TSD as our random seed. Since we are in // the slow path, it means that tryLock failed, and as a result it's very -- 2.7.4