From c713ec8223c95c9b2503f61a5ef8e55d463dbea3 Mon Sep 17 00:00:00 2001 From: Justin Clarke Casey Date: Sat, 30 Aug 2008 21:33:55 +0000 Subject: * Committed patch in mantis 623 * Unpatched code certainly looks bizarre - attempts to add a new client if we encountered a failure in processing a packet. No apparant ill effects on a sniff test. * Thanks openlifegrid --- OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs b/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs index 7c21df2..8dec185 100644 --- a/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs +++ b/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs @@ -158,7 +158,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP try { numBytes = m_socket.EndReceiveFrom(result, ref epSender); - ok = true; + ok = true; } catch (SocketException e) { @@ -250,20 +250,9 @@ namespace OpenSim.Region.ClientStack.LindenUDP SendPacketTo(ack_it.ToBytes(),ack_it.ToBytes().Length,SocketFlags.None,p.CircuitCode.Code); } } - catch (Exception) + catch (Exception e) { - m_log.Error("[UDPSERVER]: Exception in processing packet."); - m_log.Debug("[UDPSERVER]: Adding New Client"); - try - { - AddNewClient(packet); - } - catch (Exception e3) - { - m_log.Error("[UDPSERVER]: Adding New Client threw exception " + e3.ToString()); - m_socket.BeginReceiveFrom(RecvBuffer, 0, RecvBuffer.Length, SocketFlags.None, ref epSender, - ReceivedData, null); - } + m_log.Error("[UDPSERVER]: Exception in processing packet - ignoring: ", e); } } } -- cgit v1.1