based on recent unit test output, put some extra checking in the

RunAssetCache error code

1 files changed, 9 insertions, 1 deletions

diff --git a/OpenSim/Framework/Communications/Cache/AssetCache.cs b/OpenSim/Framework/Communications/Cache/AssetCache.cs
index ad8d625..f214e19 100644
--- a/OpenSim/Framework/Communications/Cache/AssetCache.cs
+++ b/OpenSim/Framework/Communications/Cache/AssetCache.cs
@@ -185,7 +185,15 @@ namespace OpenSim.Framework.Communications.Cache
                 }
                 catch (Exception e)
                 {
-                    m_log.Error("[ASSET CACHE]: " + e.ToString());
+                    if (e != null) 
+                    {
+                        m_log.ErrorFormat("[ASSET CACHE]: {0}", e);
+                    } 
+                    else 
+                    {
+                        // this looks weird, but we've seen this show up as an issue in unit tests, so leave it here until we know why
+                        m_log.Error("[ASSET CACHE]: an exception was thrown in RunAssetManager, but is now null.  Something is very wrong.");
+                    }
                 }
             }
         }