fix: Fixed invalid ownership assignment

alexkar2030 · Nov 5, 2019 · 32a59c5 · 32a59c5
1 parent fa53010
commit 32a59c5
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/MLAPI/Spawning/SpawnManager.cs b/MLAPI/Spawning/SpawnManager.cs
@@ -212,10 +212,13 @@ internal static void ChangeOwnership(NetworkedObject netObject, ulong clientId)
                 throw new SpawnStateException("Object is not spawned");
             }
 
-            for (int i = NetworkingManager.Singleton.ConnectedClients[netObject.OwnerClientId].OwnedObjects.Count - 1; i > -1; i--)
+            if (NetworkingManager.Singleton.ConnectedClients.ContainsKey(netObject.OwnerClientId))
             {
-                if (NetworkingManager.Singleton.ConnectedClients[netObject.OwnerClientId].OwnedObjects[i] == netObject)
-                    NetworkingManager.Singleton.ConnectedClients[netObject.OwnerClientId].OwnedObjects.RemoveAt(i);
+                for (int i = NetworkingManager.Singleton.ConnectedClients[netObject.OwnerClientId].OwnedObjects.Count - 1; i >= 0; i--)
+                {
+                    if (NetworkingManager.Singleton.ConnectedClients[netObject.OwnerClientId].OwnedObjects[i] == netObject)
+                        NetworkingManager.Singleton.ConnectedClients[netObject.OwnerClientId].OwnedObjects.RemoveAt(i);
+                }
             }
 
             NetworkingManager.Singleton.ConnectedClients[clientId].OwnedObjects.Add(netObject);