* Fix http://opensimulator.org/mantis/view.php?id=2517

* Don't save attachments on saving oar, which stops them coming back as ghost prims

2 files changed, 7 insertions, 12 deletions

diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
index 07af65d..5b3bc98 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
@@ -4129,6 +4129,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP
                             handlerApproveFriendRequest(this, agentID, transactionID, callingCardFolders);
                         }
                         break;
+                    
                     case PacketType.TerminateFriendship:
                         TerminateFriendshipPacket tfriendpack = (TerminateFriendshipPacket)Pack;
                         UUID listOwnerAgentID = tfriendpack.AgentData.AgentID;
@@ -4140,22 +4141,13 @@ namespace OpenSim.Region.ClientStack.LindenUDP
                             handlerTerminateFriendship(this, listOwnerAgentID, exFriendID);
                         }
                         break;
+                    
                     case PacketType.RezObject:
                         RezObjectPacket rezPacket = (RezObjectPacket)Pack;
 
                         handlerRezObject = OnRezObject;
                         if (handlerRezObject != null)
                         {
-                            //rezPacket.RezData.BypassRaycast;
-                            //rezPacket.RezData.RayEnd;
-                            //rezPacket.RezData.RayEndIsIntersection;
-                            //rezPacket.RezData.RayStart;
-                            //rezPacket.RezData.RayTargetID;
-                            //rezPacket.RezData.RemoveItem;
-                            //rezPacket.RezData.RezSelected;
-                            //rezPacket.RezData.FromTaskID;
-                            //m_log.Info("[REZData]: " + rezPacket.ToString());
-
                             handlerRezObject(this, rezPacket.InventoryData.ItemID, rezPacket.RezData.RayEnd,
                                              rezPacket.RezData.RayStart, rezPacket.RezData.RayTargetID,
                                              rezPacket.RezData.BypassRaycast, rezPacket.RezData.RayEndIsIntersection,
@@ -4163,6 +4155,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP
                                              rezPacket.RezData.FromTaskID);
                         }
                         break;
+                    
                     case PacketType.DeRezObject:
                         DeRezObjectPacket DeRezPacket = (DeRezObjectPacket) Pack;
                         handlerDeRezObject = OnDeRezObject;
@@ -4178,6 +4171,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP
                             }
                         }
                         break;
+                    
                     case PacketType.ModifyLand:
                         ModifyLandPacket modify = (ModifyLandPacket)Pack;
                         //m_log.Info("[LAND]: LAND:" + modify.ToString());
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs b/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs
index 52a2467..ba4e18b 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLUDPServer.cs
@@ -293,9 +293,10 @@ namespace OpenSim.Region.ClientStack.LindenUDP
             {
                 CloseCircuit(reusedEpSender, e);
             }
-            catch (Exception a)
+            catch (Exception e2)
             {
-                m_log.Error("[UDPSERVER]: " + a);
+                m_log.ErrorFormat(
+                    "[CLIENT]: Exception thrown when trying to close the circuit for {0} - {1}", reusedEpSender, e2);
             }
 
             // ENDLESS LOOP ON PURPOSE!