From 6f4c337a5696ef2e51c10511ceeed873cd0834b3 Mon Sep 17 00:00:00 2001
From: Dean Herbert <pe@ppy.sh>
Date: Mon, 24 Jan 2022 20:52:27 +0900
Subject: [PATCH] Fix a failed `BlockAllOperations` leaving update realm in
 unretrieved state

If the operation timed out on..

```csharp
   throw new TimeoutException(@"Took too long to acquire lock");
```

..from an update thread, it would not restore the update context.

The next call would then fail on the assert that ensures a non-null
context in such cases.

Can add test coverage if required.
---
 osu.Game/Database/RealmAccess.cs | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/osu.Game/Database/RealmAccess.cs b/osu.Game/Database/RealmAccess.cs
index 66b4edbe84..64063664eb 100644
--- a/osu.Game/Database/RealmAccess.cs
+++ b/osu.Game/Database/RealmAccess.cs
@@ -595,8 +595,8 @@ namespace osu.Game.Database
                 {
                     if (updateRealm == null)
                     {
-                        // null realm means the update thread has not yet retrieved its instance.
-                        // we don't need to worry about reviving the update instance in this case, so don't bother with the SynchronizationContext.
+                        // null context means the update thread has not yet retrieved its context.
+                        // we don't need to worry about reviving the update context in this case, so don't bother with the SynchronizationContext.
                         Debug.Assert(!ThreadSafety.IsUpdateThread);
                     }
                     else
@@ -639,18 +639,19 @@ namespace osu.Game.Database
             }
             catch
             {
-                realmCreationLock.Release();
+                restoreOperation();
                 throw;
             }
 
-            return new InvokeOnDisposal<RealmAccess>(this, factory =>
-            {
-                factory.realmCreationLock.Release();
-                Logger.Log(@"Restoring realm operations.", LoggingTarget.Database);
+            return new InvokeOnDisposal(restoreOperation);
 
+            void restoreOperation()
+            {
+                Logger.Log(@"Restoring realm operations.", LoggingTarget.Database);
+                realmCreationLock.Release();
                 // Post back to the update thread to revive any subscriptions.
                 syncContext?.Post(_ => ensureUpdateRealm(), null);
-            });
+            }
         }
 
         // https://github.com/realm/realm-dotnet/blob/32f4ebcc88b3e80a3b254412665340cd9f3bd6b5/Realm/Realm/Extensions/ReflectionExtensions.cs#L46