From 5043be13fb07a8e0a37c1f496e9bb6127b163dfd Mon Sep 17 00:00:00 2001
From: Justin Clark-Casey (justincc)
Date: Fri, 29 Jul 2011 21:49:23 +0100
Subject: Return null from CreateMeshFromPrimMesher if OpenJPEG decoding of the
 sculpt data fails.

This is to address http://opensimulator.org/mantis/view.php?id=5612
---
 OpenSim/Region/Physics/Meshing/Meshmerizer.cs | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/OpenSim/Region/Physics/Meshing/Meshmerizer.cs b/OpenSim/Region/Physics/Meshing/Meshmerizer.cs
index 5413aa8..8a9260c 100644
--- a/OpenSim/Region/Physics/Meshing/Meshmerizer.cs
+++ b/OpenSim/Region/Physics/Meshing/Meshmerizer.cs
@@ -456,11 +456,21 @@ namespace OpenSim.Region.Physics.Meshing
                         {
                             OpenMetaverse.Imaging.ManagedImage unusedData;
                             OpenMetaverse.Imaging.OpenJPEG.DecodeToImage(primShape.SculptData, out unusedData, out idata);
+
+                            if (idata == null)
+                            {
+                                // In some cases it seems that the decode can return a null bitmap without throwing
+                                // an exception
+                                m_log.WarnFormat("[PHYSICS]: OpenJPEG decoded sculpt data for {0} to a null bitmap.  Ignoring.", primName);
+
+                                return null;
+                            }
+
                             unusedData = null;
 
                             //idata = CSJ2K.J2kImage.FromBytes(primShape.SculptData);
 
-                            if (cacheSculptMaps && idata != null)
+                            if (cacheSculptMaps)
                             {
                                 try { idata.Save(decodedSculptFileName, ImageFormat.MemoryBmp); }
                                 catch (Exception e) { m_log.Error("[SCULPT]: unable to cache sculpt map " + decodedSculptFileName + " " + e.Message); }
-- 
cgit v1.1