changing DropSend()'s foreach loop so that it does not copy the

LLQueItem m_NeedAck queue each time.

1 files changed, 11 insertions, 5 deletions

diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLPacketHandler.cs b/OpenSim/Region/ClientStack/LindenUDP/LLPacketHandler.cs
index a1a8e00..7e3d919 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLPacketHandler.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLPacketHandler.cs
@@ -741,16 +741,22 @@ namespace OpenSim.Region.ClientStack.LindenUDP
 
         private void DropResend(Object id)
         {
-            foreach (LLQueItem data in new List<LLQueItem>(m_NeedAck.Values))
+            LLQueItem d = null;
+
+            foreach (LLQueItem data in m_NeedAck.Values)
             {
                 if (data.Identifier != null && data.Identifier == id)
                 {
-                    m_NeedAck.Remove(data.Packet.Header.Sequence);
-                    m_PacketQueue.Cancel(data.Sequence);
-                    PacketPool.Instance.ReturnPacket(data.Packet);
-                    return;
+                    d = data;
+                    break;
                 }
             }
+
+            if (null == d) return;
+
+            m_NeedAck.Remove(d.Packet.Header.Sequence);
+            m_PacketQueue.Cancel(d.Sequence);
+            PacketPool.Instance.ReturnPacket(d.Packet);
         }
 
         private void TriggerOnPacketDrop(Packet packet, Object id)