================
@@ -1573,6 +1576,8 @@ static uint64_t bits(uint64_t value, uint32_t msbit, 
uint32_t lsbit) {
 }
 
 bool MachProcess::Interrupt() {
+  PTHREAD_MUTEX_LOCKER(locker, m_exception_and_signal_mutex);
+
----------------
JDevlieghere wrote:

I was worried about lock order inversion between the exception mutex and the 
state mutex. That said, how we're protecting GetState is already questionable 
(i.e. nothing guarantees the state hasn't changed between the function returned 
you read the value) so I can tighten the scope and we can fix the issue if it 
comes up when we try to fix the state locking. 

https://github.com/llvm/llvm-project/pull/131073
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to