From 855eda844ffa12b01fccfd9aeb831390b02687bf Mon Sep 17 00:00:00 2001
From: Justin Clark-Casey (justincc)
Date: Mon, 19 Apr 2010 20:06:55 +0100
Subject: log problems when an asset request through the UDP TransferRequest
 channel goes wrong

---
 .../Region/ClientStack/LindenUDP/LLClientView.cs   | 47 +++++++++++++++++++++-
 1 file changed, 45 insertions(+), 2 deletions(-)

(limited to 'OpenSim/Region/ClientStack')

diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
index b3f5f09..54c312c 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
@@ -7091,32 +7091,75 @@ namespace OpenSim.Region.ClientStack.LindenUDP
                 taskID = new UUID(transfer.TransferInfo.Params, 48);
                 UUID itemID = new UUID(transfer.TransferInfo.Params, 64);
                 UUID requestID = new UUID(transfer.TransferInfo.Params, 80);
+
+//                m_log.DebugFormat(
+//                    "[LLCLIENTVIEW]: Got request for asset {0} from item {1} in prim {2} by {3}", 
+//                    requestID, itemID, taskID, Name);
+                
                 if (!(((Scene)m_scene).Permissions.BypassPermissions()))
                 {
                     if (taskID != UUID.Zero) // Prim
                     {
                         SceneObjectPart part = ((Scene)m_scene).GetSceneObjectPart(taskID);
+                        
                         if (part == null)
+                        {
+                            m_log.WarnFormat(
+                                "[LLCLIENTVIEW]: {0} requested asset {1} from item {2} in prim {3} but prim does not exist", 
+                                Name, requestID, itemID, taskID);
                             return true;
+                        }
 
                         if (part.OwnerID != AgentId)
+                        {
+                            m_log.WarnFormat(
+                                "[LLCLIENTVIEW]: {0} requested asset {1} from item {2} in prim {3} but the prim is owned by {4}",
+                                Name, requestID, itemID, taskID, part.OwnerID);                            
                             return true;
+                        }
 
                         if ((part.OwnerMask & (uint)PermissionMask.Modify) == 0)
+                        {
+                            m_log.WarnFormat(
+                                "[LLCLIENTVIEW]: {0} requested asset {1} from item {2} in prim {3} but modify permissions are not set", 
+                                Name, requestID, itemID, taskID);                            
                             return true;
+                        }
 
                         TaskInventoryItem ti = part.Inventory.GetInventoryItem(itemID);
                         if (ti == null)
+                        {
+                            m_log.WarnFormat(
+                                "[LLCLIENTVIEW]: {0} requested asset {1} from item {2} in prim {3} but item does not exist", 
+                                Name, requestID, itemID, taskID);                            
                             return true;
+                        }
 
                         if (ti.OwnerID != AgentId)
+                        {
+                            m_log.WarnFormat(
+                                "[LLCLIENTVIEW]: {0} requested asset {1} from item {2} in prim {3} but the item is owned by {4}", 
+                                Name, requestID, itemID, taskID, ti.OwnerID);                            
                             return true;
+                        }
 
-                        if ((ti.CurrentPermissions & ((uint)PermissionMask.Modify | (uint)PermissionMask.Copy | (uint)PermissionMask.Transfer)) != ((uint)PermissionMask.Modify | (uint)PermissionMask.Copy | (uint)PermissionMask.Transfer))
+                        if ((
+                            ti.CurrentPermissions & ((uint)PermissionMask.Modify | (uint)PermissionMask.Copy | (uint)PermissionMask.Transfer)) 
+                                != ((uint)PermissionMask.Modify | (uint)PermissionMask.Copy | (uint)PermissionMask.Transfer))
+                        {
+                            m_log.WarnFormat(
+                                "[LLCLIENTVIEW]: {0} requested asset {1} from item {2} in prim {3} but item permissions are not modify/copy/transfer", 
+                                Name, requestID, itemID, taskID);                            
                             return true;
+                        }
 
                         if (ti.AssetID != requestID)
+                        {
+                            m_log.WarnFormat(
+                                "[LLCLIENTVIEW]: {0} requested asset {1} from item {2} in prim {3} but this does not match item's asset {4}", 
+                                Name, requestID, itemID, taskID, ti.AssetID);                            
                             return true;
+                        }
                     }
                     else // Agent
                     {
@@ -11389,7 +11432,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP
                 //    }
             }
 
-            //m_log.DebugFormat("[LLCLIENTVIEW]: {0} requesting asset {1}", Name, requestID);
+//            m_log.DebugFormat("[LLCLIENTVIEW]: {0} requesting asset {1}", Name, requestID);
 
             m_assetService.Get(requestID.ToString(), transferRequest, AssetReceived);
         }
-- 
cgit v1.1