From 2641f78a205733dd2730302b84f46d01262bc2ec Mon Sep 17 00:00:00 2001
From: Teravus Ovares
Date: Thu, 20 Dec 2007 14:44:59 +0000
Subject: * Fixed a null NewPack error in ClientView.InPacket (object pool
 returning null objects?)

---
 OpenSim/Region/ClientStack/ClientView.cs | 60 +++++++++++++++++---------------
 1 file changed, 32 insertions(+), 28 deletions(-)

(limited to 'OpenSim/Region/ClientStack')

diff --git a/OpenSim/Region/ClientStack/ClientView.cs b/OpenSim/Region/ClientStack/ClientView.cs
index 58cb9a2..7161942 100644
--- a/OpenSim/Region/ClientStack/ClientView.cs
+++ b/OpenSim/Region/ClientStack/ClientView.cs
@@ -2217,44 +2217,48 @@ namespace OpenSim.Region.ClientStack
         public virtual void InPacket(Packet NewPack)
         {
             // Handle appended ACKs
-            if (NewPack.Header.AppendedAcks)
+            if (NewPack != null)
             {
-                lock (m_needAck)
+                if (NewPack.Header.AppendedAcks)
                 {
-                    foreach (uint ack in NewPack.Header.AckList)
+                    lock (m_needAck)
                     {
-                        m_needAck.Remove(ack);
+                        foreach (uint ack in NewPack.Header.AckList)
+                        {
+                            m_needAck.Remove(ack);
+                        }
                     }
                 }
-            }
-
-            // Handle PacketAck packets
-            if (NewPack.Type == PacketType.PacketAck)
-            {
-                PacketAckPacket ackPacket = (PacketAckPacket) NewPack;
+            
 
-                lock (m_needAck)
+                // Handle PacketAck packets
+                if (NewPack.Type == PacketType.PacketAck)
                 {
-                    foreach (PacketAckPacket.PacketsBlock block in ackPacket.Packets)
+                    PacketAckPacket ackPacket = (PacketAckPacket) NewPack;
+
+                    lock (m_needAck)
                     {
-                        m_needAck.Remove(block.ID);
+                        foreach (PacketAckPacket.PacketsBlock block in ackPacket.Packets)
+                        {
+                            m_needAck.Remove(block.ID);
+                        }
                     }
                 }
-            }
-            else if ((NewPack.Type == PacketType.StartPingCheck))
-            {
-                //reply to pingcheck
-                StartPingCheckPacket startPing = (StartPingCheckPacket) NewPack;
-                CompletePingCheckPacket endPing = new CompletePingCheckPacket();
-                endPing.PingID.PingID = startPing.PingID.PingID;
-                OutPacket(endPing, ThrottleOutPacketType.Task);
-            }
-            else
-            {
-                QueItem item = new QueItem();
-                item.Packet = NewPack;
-                item.Incoming = true;
-                m_packetQueue.Enqueue(item);
+                else if ((NewPack.Type == PacketType.StartPingCheck))
+                {
+                    //reply to pingcheck
+                    StartPingCheckPacket startPing = (StartPingCheckPacket) NewPack;
+                    CompletePingCheckPacket endPing = new CompletePingCheckPacket();
+                    endPing.PingID.PingID = startPing.PingID.PingID;
+                    OutPacket(endPing, ThrottleOutPacketType.Task);
+                }
+                else
+                {
+                    QueItem item = new QueItem();
+                    item.Packet = NewPack;
+                    item.Incoming = true;
+                    m_packetQueue.Enqueue(item);
+                }
             }
         }
 
-- 
cgit v1.1