#4109 Add validation for non-triangulated geometry in GLTF loader

Author: marchcat

indra/newview/gltf/llgltfloader.cpp

@@ -461,6 +461,16 @@ bool LLGLTFLoader::populateModelFromMesh(LLModel* pModel, const LL::GLTF::Mesh&
                 vertices.push_back(vert);
             }
 
+            if (prim.getIndexCount() % 3 != 0)
+            {
+                LL_WARNS("GLTF_IMPORT") << "Invalid primitive: index count " << prim.getIndexCount()
+                                       << " is not divisible by 3. GLTF files must contain triangulated geometry." << LL_ENDL;
+                LLSD args;
+                args["Message"] = "InvalidGeometryNonTriangulated";
+                mWarningsArray.append(args);
+                continue; // Skip this primitive
+            }
+
             // When processing indices, flip winding order if needed
             for (U32 i = 0; i < prim.getIndexCount(); i += 3)
             {

indra/newview/skins/default/xui/en/floater_model_preview.xml

@@ -61,6 +61,7 @@
   <string name="ParsingErrorNoRoot">Document has no root</string>
   <string name="ParsingErrorNoScene">Document has no visual_scene</string>
   <string name="ParsingErrorPositionInvalidModel">Unable to process mesh without position data. Invalid model.</string>
+  <string name="InvalidGeometryNonTriangulated">Invalid geometry: GLTF files must contain triangulated meshes only.</string>
 
   <panel
     follows="top|left"