1 files changed, 18 insertions, 12 deletions

diff --git a/linden/indra/newview/llviewerobjectlist.cpp b/linden/indra/newview/llviewerobjectlist.cpp
index 78ce247..2d6c9f4 100644
--- a/linden/indra/newview/llviewerobjectlist.cpp
+++ b/linden/indra/newview/llviewerobjectlist.cpp
@@ -166,12 +166,12 @@ U64 LLViewerObjectList::getIndex(const U32 local_id,
         return (((U64)index) << 32) | (U64)local_id;
 }
 
-BOOL LLViewerObjectList::removeFromLocalIDTable(const LLViewerObject &object)
+BOOL LLViewerObjectList::removeFromLocalIDTable(const LLViewerObject* objectp)
 {
-        if(object.getRegion())
+        if (objectp && objectp->getRegion())
         {
-                U32 local_id = object.mLocalID;
-                LLHost region_host = object.getRegion()->getHost();
+                U32 local_id = objectp->mLocalID;
+                LLHost region_host = objectp->getRegion()->getHost();
                 U32 ip = region_host.getAddress();
                 U32 port = region_host.getPort();
                 U64 ipport = (((U64)ip) << 32) | (U64)port;
@@ -186,7 +186,7 @@ BOOL LLViewerObjectList::removeFromLocalIDTable(const LLViewerObject &object)
                 }
                 
                 // Found existing entry
-                if (iter->second == object.getID())
+                if (iter->second == objectp->getID())
                 {   // Full UUIDs match, so remove the entry
                         sIndexAndLocalIDToUUID.erase(iter);
                         return TRUE;
@@ -451,7 +451,7 @@ void LLViewerObjectList::processObjectUpdate(LLMessageSystem *mesgsys,
                         ((objectp->mLocalID != local_id) ||
                          (objectp->getRegion() != regionp)))
                 {
-                        removeFromLocalIDTable(*objectp);
+                        removeFromLocalIDTable(objectp);
                         setUUIDAndLocal(fullid,
                                                         local_id,
                                                         gMessageSystem->getSenderIP(),
@@ -813,20 +813,26 @@ void LLViewerObjectList::clearDebugText()
 
 void LLViewerObjectList::cleanupReferences(LLViewerObject *objectp)
 {
+        if (!objectp)
+        {
+                llwarns << "NULL object pointer passed." << llendl;
+                return;
+        }
         LLMemType mt(LLMemType::MTYPE_OBJECT);
-        if (mDeadObjects.count(objectp->mID))
+        if (mDeadObjects.find(objectp->mID) != mDeadObjects.end())
         {
                 llinfos << "Object " << objectp->mID << " already on dead list, ignoring cleanup!" << llendl;   
-                return;
         }
-
-        mDeadObjects.insert(std::pair<LLUUID, LLPointer<LLViewerObject> >(objectp->mID, objectp));
+        else
+        {
+                mDeadObjects.insert(objectp->mID);
+        }
 
         // Cleanup any references we have to this object
         // Remove from object map so noone can look it up.
 
         mUUIDObjectMap.erase(objectp->mID);
-        removeFromLocalIDTable(*objectp);
+        removeFromLocalIDTable(objectp);
 
         if (objectp->onActiveList())
         {
@@ -1060,7 +1066,7 @@ void LLViewerObjectList::renderObjectsForMap(LLNetMap &netmap)
         for (S32 i = 0; i < mMapObjects.count(); i++)
         {
                 LLViewerObject* objectp = mMapObjects[i];
-                if (!objectp->getRegion() || objectp->isOrphaned() || objectp->isAttachment())
+                if (objectp->isDead() || !objectp->getRegion() || objectp->isOrphaned() || objectp->isAttachment())
                 {
                         continue;
                 }