From c89db34fc4372be7ff94d92472131b2c33de8605 Mon Sep 17 00:00:00 2001
From: Justin Clark-Casey (justincc)
Date: Tue, 12 Jun 2012 02:03:31 +0100
Subject: If the simulator closes a root agent due to ack timeout, then send
 the client a kick message with that reason, in case it is somehow still
 listening.

---
 .../Region/ClientStack/Linden/UDP/LLUDPServer.cs   | 22 ++++++++++++++++++++--
 1 file changed, 20 insertions(+), 2 deletions(-)

(limited to 'OpenSim/Region/ClientStack/Linden/UDP/LLUDPServer.cs')

diff --git a/OpenSim/Region/ClientStack/Linden/UDP/LLUDPServer.cs b/OpenSim/Region/ClientStack/Linden/UDP/LLUDPServer.cs
index 7f86491..2036f61 100644
--- a/OpenSim/Region/ClientStack/Linden/UDP/LLUDPServer.cs
+++ b/OpenSim/Region/ClientStack/Linden/UDP/LLUDPServer.cs
@@ -558,9 +558,12 @@ namespace OpenSim.Region.ClientStack.LindenUDP
             if (!client.IsLoggingOut &&
                 (Environment.TickCount & Int32.MaxValue) - udpClient.TickLastPacketReceived > timeoutTicks)
             {
-                m_log.Warn("[LLUDPSERVER]: Ack timeout, disconnecting " + udpClient.AgentID);
+                m_log.WarnFormat(
+                    "[LLUDPSERVER]: Ack timeout for {0} {1}, disconnecting",
+                    client.Name, client.AgentId);
+
                 StatsManager.SimExtraStats.AddAbnormalClientThreadTermination();
-                RemoveClient(client);
+                LogoutClientDueToTimeout(client);
 
                 return;
             }
@@ -1121,6 +1124,21 @@ namespace OpenSim.Region.ClientStack.LindenUDP
             Util.FireAndForget(o => client.Close());
         }
 
+        private void LogoutClientDueToTimeout(IClientAPI client)
+        {
+            // We must set IsLoggingOut synchronously so that we can stop the packet loop reinvoking this method.
+            client.IsLoggingOut = true;
+
+            // Fire this out on a different thread so that we don't hold up outgoing packet processing for
+            // everybody else if this is being called due to an ack timeout.
+            // This is the same as processing as the async process of a logout request.
+            Util.FireAndForget(
+                o =>
+                    { if (!client.SceneAgent.IsChildAgent)
+                        client.Kick("Simulator logged you out due to connection timeout");
+                      client.Close(); });
+        }
+
         private void IncomingPacketHandler()
         {
             // Set this culture for the thread that incoming packets are received
-- 
cgit v1.1