Check if length has been initialized in ArrayInstance.GetLength (#761)

sebastienros · Jul 19, 2020 · 11b0173 · 11b0173
1 parent 5d1114d
commit 11b0173
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/Jint.Tests/Runtime/ArrayTests.cs b/Jint.Tests/Runtime/ArrayTests.cs
@@ -1,4 +1,5 @@
 using System;
+using Jint.Native.Array;
 using Xunit;
 
 namespace Jint.Tests.Runtime
@@ -64,5 +65,14 @@ public void LargeArraySize()
             var engine = new Engine();
             engine.Execute(code);
         }
+
+        [Fact]
+        public void ArrayLengthFromInitialState()
+        {
+            var engine = new Engine();
+            var array = new ArrayInstance(engine, 0);
+            var length = (int) array.Length;
+            Assert.Equal(0, length);
+        }
     }
 }
diff --git a/Jint/Native/Array/ArrayInstance.cs b/Jint/Native/Array/ArrayInstance.cs
@@ -222,6 +222,11 @@ public override bool DefineOwnProperty(JsValue property, PropertyDescriptor desc
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         internal uint GetLength()
         {
+            if (_length is null)
+            {
+                return 0;
+            }
+
             return (uint) ((JsNumber) _length._value)._value;
         }