From dee6ad715412a5adeeb46c404726053ce8310e79 Mon Sep 17 00:00:00 2001 From: diva Date: Sun, 1 Feb 2009 20:36:10 +0000 Subject: [PATCH] Putting the return back in AddCapsHandler upon attempt at adding CAPs twice. The return seems to have been commented in 8038, as an attempt at fixing multiple TP problems later identified to be deadlocks. CAPs should never be overwritten, or the viewer can get confused. Right now this method is erroneously being called twice because of legacy code. I'll fix that later, after further testing. --- .../Modules/Agent/Capabilities/CapabilitiesModule.cs | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/OpenSim/Region/Environment/Modules/Agent/Capabilities/CapabilitiesModule.cs b/OpenSim/Region/Environment/Modules/Agent/Capabilities/CapabilitiesModule.cs index 070736e8f1..5a44902eba 100644 --- a/OpenSim/Region/Environment/Modules/Agent/Capabilities/CapabilitiesModule.cs +++ b/OpenSim/Region/Environment/Modules/Agent/Capabilities/CapabilitiesModule.cs @@ -78,9 +78,12 @@ namespace OpenSim.Region.Environment.Modules.Agent.Capabilities Caps oldCaps = m_capsHandlers[agentId]; m_log.DebugFormat( - "[CAPS]: Reregistering caps for agent {0}. Old caps path {1}, new caps path {2}", + "[CAPS]: Reregistering caps for agent {0}. Old caps path {1}, new caps path {2}. Ignoring. ", agentId, oldCaps.CapsObjectPath, capsObjectPath); - //return; + // This should not happen. The caller code is confused. We need to fix that. + // CAPs can never be reregistered, or the client will be confused. + // Hence this return here. + return; } Caps caps