Do not call pending actions under globally shared lock

Signed-off-by: Viktor Svyatokha <[email protected]>

Author: ahydrax

src/MySqlConnector/Utilities/TimerQueue.cs

@@ -67,12 +67,14 @@ private TimerQueue()
 
 	private void Callback(object? obj)
 	{
+		var actionsToBeCalled = new List<Action>();
+
 		lock (m_lock)
 		{
 			// process all timers that have expired or will expire in the granularity of a clock tick
 			while (m_timeoutActions.Count > 0 && unchecked(m_timeoutActions[0].Time - Environment.TickCount) < 15)
 			{
-				m_timeoutActions[0].Action();
+				actionsToBeCalled.Add(m_timeoutActions[0].Action);
 				m_timeoutActions.RemoveAt(0);
 			}
 
@@ -86,6 +88,11 @@ private void Callback(object? obj)
 				UnsafeSetTimer(delay);
 			}
 		}
+
+		foreach (var action in actionsToBeCalled)
+		{
+			action();
+		}
 	}
 
 	// Should be called while holding m_lock.