From a12f65234708c6a0ff9e08193aa65ce15a96326d Mon Sep 17 00:00:00 2001
From: Melanie Thielker
Date: Wed, 27 Aug 2008 17:50:07 +0000
Subject: Make the check on inventory cache clearing more robust. Addresses
 Mantis #1975 in a broader way. This may or may not prop up secure inventory a
 bit better, but I still recommend to disable it

---
 .../Modules/Avatar/Inventory/InventoryModule.cs    | 36 ++++++++++++++++++++++
 1 file changed, 36 insertions(+)

(limited to 'OpenSim/Region/Environment/Modules')

diff --git a/OpenSim/Region/Environment/Modules/Avatar/Inventory/InventoryModule.cs b/OpenSim/Region/Environment/Modules/Avatar/Inventory/InventoryModule.cs
index bf55e7b..d036dbb 100644
--- a/OpenSim/Region/Environment/Modules/Avatar/Inventory/InventoryModule.cs
+++ b/OpenSim/Region/Environment/Modules/Avatar/Inventory/InventoryModule.cs
@@ -233,13 +233,49 @@ namespace OpenSim.Region.Environment.Modules.Avatar.Inventory
         public bool NeedSceneCacheClear(LLUUID agentID, Scene scene)
         {
             if (!m_AgentRegions.ContainsKey(agentID))
+            {
+                // Since we can get here two ways, we need to scan
+                // the scenes here. This is somewhat more expensive
+                // but helps avoid a nasty bug
+                //
+
+                foreach (Scene s in m_Scenelist)
+                {
+                    ScenePresence presence;
+
+                    if (s.TryGetAvatar(agentID, out presence))
+                    {
+                        // If the agent is in this scene, then we
+                        // are being called twice in a single
+                        // teleport. This is wasteful of cycles
+                        // but harmless due to this 2nd level check
+                        //
+                        // If the agent is found in another scene
+                        // then the list wasn't current
+                        //
+                        // If the agent is totally unknown, then what
+                        // are we even doing here??
+                        //
+                        if (s == scene)
+                            return true;
+                        else
+                            return false;
+                    }
+                }
                 return true;
+            }
 
+            // The agent is left in current Scene, so we must be
+            // going to another instance
+            //
             if (m_AgentRegions[agentID] == scene)
             {
                 m_AgentRegions.Remove(agentID);
                 return true;
             }
+
+            // Another region has claimed the agent
+            //
             return false;
         }
 
-- 
cgit v1.1