Fixed deadlock scenario caused by unsubscribing during an event.

MapleWheels · MapleWheels · commit 8bc5b585344d · 2026-03-21T23:59:19.000-04:00
diff --git a/Barotrauma/BarotraumaShared/SharedSource/LuaCs/_Services/EventService.cs b/Barotrauma/BarotraumaShared/SharedSource/LuaCs/_Services/EventService.cs
@@ -264,7 +264,7 @@ public FluentResults.Result Subscribe<T>(T subscriber) where T : class, IEvent<T
     public void Unsubscribe<T>(T subscriber) where T : class, IEvent
     {
         Guard.IsNotNull(subscriber, nameof(subscriber));
-        using var lck = _operationsLock.AcquireWriterLock().ConfigureAwait(false).GetAwaiter().GetResult();
+        using var lck = _operationsLock.AcquireReaderLock().ConfigureAwait(false).GetAwaiter().GetResult();
         IService.CheckDisposed(this);
 
         if (!_subscribers.TryGetValue(typeof(T), out var evtSubscribers))

Original file line number	Diff line number	Diff line change
`@@ -264,7 +264,7 @@ public FluentResults.Result Subscribe<T>(T subscriber) where T : class, IEvent<T`
`264`	`264`	`public void Unsubscribe<T>(T subscriber) where T : class, IEvent`
`265`	`265`	`{`
`266`	`266`	`Guard.IsNotNull(subscriber, nameof(subscriber));`
`267`		`- using var lck = _operationsLock.AcquireWriterLock().ConfigureAwait(false).GetAwaiter().GetResult();`
	`267`	`+ using var lck = _operationsLock.AcquireReaderLock().ConfigureAwait(false).GetAwaiter().GetResult();`
`268`	`268`	`IService.CheckDisposed(this);`
`269`	`269`
`270`	`270`	`if (!_subscribers.TryGetValue(typeof(T), out var evtSubscribers))`