From 7dec345d9a9da40d67072eb9b2b2194887d972ea Mon Sep 17 00:00:00 2001
From: Justin Clarke Casey
Date: Sun, 13 Apr 2008 22:25:23 +0000
Subject: * Temporarily, stop the exception of mantis #951 from killing the
 entire client session.

---
 OpenSim/Region/ClientStack/ClientView.cs | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

(limited to 'OpenSim/Region/ClientStack')

diff --git a/OpenSim/Region/ClientStack/ClientView.cs b/OpenSim/Region/ClientStack/ClientView.cs
index 939bd8f..9fe2144 100644
--- a/OpenSim/Region/ClientStack/ClientView.cs
+++ b/OpenSim/Region/ClientStack/ClientView.cs
@@ -3265,7 +3265,19 @@ namespace OpenSim.Region.ClientStack
                         handlerSetAppearance = OnSetAppearance;
                         if (handlerSetAppearance != null)
                         {
-                            handlerSetAppearance(appear.ObjectData.TextureEntry, appear.VisualParam);
+                            // Temporarily protect ourselves from the mantis #951 failure.
+                            // However, we could do this for several other handlers where a failure isn't terminal
+                            // for the client session anyway, in order to protect ourselves against bad code in plugins 
+                            try
+                            {
+                                handlerSetAppearance(appear.ObjectData.TextureEntry, appear.VisualParam);
+                            }
+                            catch (Exception e)
+                            {
+                                m_log.ErrorFormat(
+                                    "[CLIENT VIEW]: AgentSetApperance packet handler threw an exception, {0}", 
+                                    e);
+                            }
                         }
 
                         break;
-- 
cgit v1.1