ensure it does not skip outside of the buffer

Author: riccardobl

jme3-plugins/src/gltf/java/com/jme3/scene/plugins/gltf/GltfUtils.java

@@ -328,6 +328,11 @@ public static void populateBuffer(Object store, ByteBuffer source, int count, in
             populateMatrix4fArray((Matrix4f[]) store, source, count, byteOffset, byteStride, numComponents, format);
         }
     }
+
+    private static void skip(ByteBuffer buff, int n) {
+        buff.position(Math.min(buff.position() + n, buff.limit()));
+    }
+
     private static void populateByteBuffer(ByteBuffer buffer, ByteBuffer source, int count, int byteOffset, int byteStride, int numComponents, VertexBuffer.Format format) {
         int componentSize = format.getComponentSize();
         int index = byteOffset;
@@ -355,7 +360,7 @@ private static void populateShortBuffer(ShortBuffer buffer, ByteBuffer source, i
             }
 
             if (dataLength < stride) {
-                source.position(source.position() + (stride - dataLength));
+                skip(source, stride - dataLength);
             }
             index += stride;
         }
@@ -374,7 +379,7 @@ private static void populateIntBuffer(IntBuffer buffer, ByteBuffer source, int c
                 buffer.put(source.getInt());
             }
             if (dataLength < stride) {
-                source.position(source.position() + (stride - dataLength));
+                skip(source, stride - dataLength);
             }
             index += stride;
         }
@@ -392,7 +397,7 @@ private static void populateFloatBuffer(FloatBuffer buffer, ByteBuffer source, i
                 buffer.put(readAsFloat(source, format));
             }
             if (dataLength < stride) {
-                source.position(source.position() + (stride - dataLength));
+                skip(source, stride - dataLength);
             }
             index += stride;
         }
@@ -448,7 +453,7 @@ private static void populateByteArray(byte[] array, ByteBuffer source, int count
             System.arraycopy(buffer, 0, array, arrayIndex, numComponents);
             arrayIndex += numComponents;
             if (dataLength < stride) {
-                source.position(source.position() + (stride - dataLength));
+                skip(source, stride - dataLength);
             }
             index += stride;
         }
@@ -484,7 +489,7 @@ private static void populateShortArray(short[] array, ByteBuffer source, int cou
                 arrayIndex++;
             }
             if (dataLength < stride) {
-                source.position(source.position() + (stride - dataLength));
+                skip(source, stride - dataLength);
             }
             index += stride;
         }
@@ -582,7 +587,7 @@ private static void populateFloatArray(float[] array, ByteBuffer source, int cou
                 arrayIndex++;
             }
             if (dataLength < stride) {
-                source.position(source.position() + (stride - dataLength));
+                skip(source, stride - dataLength);
             }
             index += stride;
         }
@@ -605,7 +610,7 @@ private static void populateVector3fArray(Vector3f[] array, ByteBuffer source, i
 
             arrayIndex++;
             if (dataLength < stride) {
-                source.position(source.position() + (stride - dataLength));
+                skip(source, stride - dataLength);
             }
             index += stride;
         }
@@ -629,7 +634,7 @@ private static void populateQuaternionArray(Quaternion[] array, ByteBuffer sourc
 
             arrayIndex++;
             if (dataLength < stride) {
-                source.position(source.position() + (stride - dataLength));
+                skip(source, stride - dataLength);
             }
             index += stride;
         }
@@ -667,7 +672,7 @@ private static void populateMatrix4fArray(Matrix4f[] array, ByteBuffer source, i
 
             arrayIndex++;
             if (dataLength < stride) {
-                source.position(source.position() + (stride - dataLength));
+                skip(source, stride - dataLength);
             }
 
             index += stride;