1 files changed, 66 insertions, 1 deletions

diff --git a/indra/newview/gltf/buffer_util.h b/indra/newview/gltf/buffer_util.h
index ef9bba8128..53dee98cd1 100644
--- a/indra/newview/gltf/buffer_util.h
+++ b/indra/newview/gltf/buffer_util.h
@@ -141,12 +141,54 @@ namespace LL
         }
 
         template<>
+        inline void copyScalar<U32, LLVector4a>(U32* src, LLVector4a& dst)
+        {
+            dst.set((F32)*src, 0.f, 0.f, 0.f);
+        }
+
+        template<>
+        inline void copyScalar<U16, LLVector4a>(U16* src, LLVector4a& dst)
+        {
+            dst.set((F32)*src, 0.f, 0.f, 0.f);
+        }
+
+        template<>
+        inline void copyScalar<U8, LLVector4a>(U8* src, LLVector4a& dst)
+        {
+            dst.set((F32)*src, 0.f, 0.f, 0.f);
+        }
+
+        template<>
+        inline void copyScalar<U32, LLVector2>(U32* src, LLVector2& dst)
+        {
+            dst.set((F32)*src, 0.f);
+        }
+
+        template<>
+        inline void copyScalar<U16, LLVector2>(U16* src, LLVector2& dst)
+        {
+            dst.set((F32)*src, 0.f);
+        }
+
+        template<>
+        inline void copyScalar<U8, LLVector2>(U8* src, LLVector2& dst)
+        {
+            dst.set((F32)*src, 0.f);
+        }
+
+        template<>
         inline void copyVec2<F32, LLVector2>(F32* src, LLVector2& dst)
         {
             dst.set(src[0], src[1]);
         }
 
         template<>
+        inline void copyVec3<F32, LLVector2>(F32* src, LLVector2& dst)
+        {
+            dst.set(src[0], src[1]);
+        }
+
+        template<>
         inline void copyVec3<F32, vec3>(F32* src, vec3& dst)
         {
             dst = vec3(src[0], src[1], src[2]);
@@ -159,6 +201,12 @@ namespace LL
         }
 
         template<>
+        inline void copyVec3<F32, LLColor4U>(F32* src, LLColor4U& dst)
+        {
+            dst.set((U8)(src[0] * 255.f), (U8)(src[1] * 255.f), (U8)(src[2] * 255.f), 255);
+        }
+
+        template<>
         inline void copyVec3<U16, LLColor4U>(U16* src, LLColor4U& dst)
         {
             dst.set((U8)(src[0]), (U8)(src[1]), (U8)(src[2]), 255);
@@ -209,6 +257,12 @@ namespace LL
         }
 
         template<>
+        inline void copyVec4<U32, LLVector4a>(U32* src, LLVector4a& dst)
+        {
+            dst.set((F32)src[0], (F32)src[1], (F32)src[2], (F32)src[3]);
+        }
+
+        template<>
         inline void copyVec4<U16, LLVector4a>(U16* src, LLVector4a& dst)
         {
             dst.set(src[0], src[1], src[2], src[3]);
@@ -361,7 +415,7 @@ namespace LL
             }
             else
             {
-                LL_ERRS("GLTF") << "Unsupported accessor type" << LL_ENDL;
+                LL_ERRS("GLTF") << "Unsupported accessor type " << (S32)accessor.mType << LL_ENDL;
             }
         }
 
@@ -369,7 +423,18 @@ namespace LL
         template<class T>
         inline void copy(Asset& asset, Accessor& accessor, LLStrider<T>& dst)
         {
+            if (accessor.mBufferView == INVALID_INDEX
+                || accessor.mBufferView >= asset.mBufferViews.size())
+            {
+                LL_WARNS("GLTF") << "Invalid buffer" << LL_ENDL;
+                return;
+            }
             const BufferView& bufferView = asset.mBufferViews[accessor.mBufferView];
+            if (bufferView.mBuffer >= asset.mBuffers.size())
+            {
+                LL_WARNS("GLTF") << "Invalid buffer view" << LL_ENDL;
+                return;
+            }
             const Buffer& buffer = asset.mBuffers[bufferView.mBuffer];
             const U8* src = buffer.mData.data() + bufferView.mByteOffset + accessor.mByteOffset;