* revert r7724 so that PresenceChildStatus() starts throwing NRE's again

* apparantly logout code relies on this happening in certain circumstances.  Really, the root issue needs to be investigated.

3 files changed, 12 insertions, 4 deletions

diff --git a/OpenSim/Framework/IScene.cs b/OpenSim/Framework/IScene.cs
index 5978245..d3c3843 100644
--- a/OpenSim/Framework/IScene.cs
+++ b/OpenSim/Framework/IScene.cs
@@ -80,7 +80,10 @@ namespace OpenSim.Framework
         /// Used by ClientView when a 'kick everyone' or 'estate message' occurs
         /// 
         /// <param name="avatarID">AvatarID to lookup</param>
-        /// <returns>true if the presence is a child agent, false if the presence does not exist or is not a child agent</returns>        
+        /// <returns>true if the presence is a child agent, false if the presence is a root exception</returns>
+        /// <exception cref="System.NullReferenceException">
+        /// Thrown if the agent does not exist.
+        /// </exception>
         bool PresenceChildStatus(UUID agentId);
 
         // Diva: get this out of here!!!
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
index e34c8ec..9d6e3af 100644
--- a/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
+++ b/OpenSim/Region/ClientStack/LindenUDP/LLClientView.cs
@@ -333,6 +333,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP
 
             if (m_groupPowers.ContainsKey(groupID))
                 return m_groupPowers[groupID];
+            
             return 0;
         }
 
diff --git a/OpenSim/Region/Environment/Scenes/Scene.cs b/OpenSim/Region/Environment/Scenes/Scene.cs
index 3aa2470..8dcd071 100644
--- a/OpenSim/Region/Environment/Scenes/Scene.cs
+++ b/OpenSim/Region/Environment/Scenes/Scene.cs
@@ -4133,10 +4133,14 @@ namespace OpenSim.Region.Environment.Scenes
         {
             ScenePresence cp = GetScenePresence(avatarID);
             
-            if (cp != null)
-                return cp.IsChildAgent;
+            // FIXME: This is really crap - some logout code is relying on a NullReferenceException to halt its processing
+            // This needs to be fixed properly by cleaning up the logout code.
+            //if (cp != null)
+            //    return cp.IsChildAgent;
             
-            return false;
+            //return false;
+            
+            return cp.IsChildAgent;
         }
 
         /// <summary>