cleanup last patch

1 files changed, 23 insertions, 18 deletions

diff --git a/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/OSSL_Api.cs b/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/OSSL_Api.cs
index 7c43b6a..3c824eb 100644
--- a/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/OSSL_Api.cs
+++ b/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/OSSL_Api.cs
@@ -3164,33 +3164,38 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api
                 module.Say(npcId, World, message, channel);
             }
         }
-                
-        public void osNpcSayTo(LSL_Key npc, string target, int channel, string msg)
+
+        public void osNpcSayTo(LSL_Key npc, LSL_Key target, int channel, string msg)
         {
             CheckThreatLevel(ThreatLevel.High, "osNpcSayTo");
 
             INPCModule module = World.RequestModuleInterface<INPCModule>();
-            if (module != null)
-            {
-                UUID npcId = new UUID(npc.m_string);
+            if (module == null)
+                return;
+
+            UUID npcId;
+            if (!UUID.TryParse(npc.m_string, out npcId))
+                return;
 
-               if (!module.CheckPermissions(npcId, m_host.OwnerID))
-                   return;
-                        
-                             ScenePresence NPCpresence = World.GetScenePresence(npcId);
-                if (NPCpresence == null || NPCpresence.IsDeleted)
-                    return;
-        
-            Vector3 npcPOS = NPCpresence.AbsolutePosition;
-                        string npcNAME = NPCpresence.Name;
             UUID TargetID;
-            UUID.TryParse(target, out TargetID);
-            
-                IWorldComm wComm = m_ScriptEngine.World.RequestModuleInterface<IWorldComm>();
+            if (!UUID.TryParse(target.m_string, out TargetID))
+                return;
+
+            if (!module.CheckPermissions(npcId, m_host.OwnerID))
+                return;
+
+            ScenePresence NPCpresence = World.GetScenePresence(npcId);
+            if (NPCpresence == null || NPCpresence.IsDeleted || !NPCpresence.IsNPC)
+                return;
+
+            Vector3 npcPOS = NPCpresence.AbsolutePosition;
+            string npcNAME = NPCpresence.Name;
+
+            IWorldComm wComm = m_ScriptEngine.World.RequestModuleInterface<IWorldComm>();
             if (wComm != null)
                 wComm.DeliverMessageTo(TargetID, channel, npcPOS, npcNAME, npcId, msg);
-            }
         }
+
         public void osNpcShout(LSL_Key npc, int channel, string message)
         {
             CheckThreatLevel(ThreatLevel.High, "osNpcShout");