Add validation when calling array element accessors

Author: vmutafov

test-app/app/src/main/assets/app/tests/testReleaseNativeCounterpart.js

@@ -25,6 +25,21 @@ describe("Test native counterpart release", function () {
 		expect(errorMessage).toBe("Failed calling toString on a java/lang/Object instance. The JavaScript instance no longer has available Java instance counterpart.");
 	});
 
+	it("Calling the indexer operator on a released native array should throw an exception", function(){
+	    var errorMessage = "";
+
+	    try{
+	        var arr = new java.lang.reflect.Array.newInstance(java.lang.Object.class, 10);
+            global.__releaseNativeCounterpart(arr);
+            arr[1];
+	    } catch(e){
+	        errorMessage = e.message;
+	    }
+
+	    expect(errorMessage).toBe("Failed calling indexer operator on native array. The JavaScript instance no longer has available Java instance counterpart.");
+
+	});
+
     it("Calling release on a non native object should throw exception", function () {
 
         var errorMessage = "";

test-app/runtime/src/main/cpp/ArrayElementAccessor.cpp

@@ -18,6 +18,8 @@ Local<Value> ArrayElementAccessor::GetArrayElement(Isolate* isolate, const Local
 
     auto arr = objectManager->GetJavaObjectByJsObject(array);
 
+    assertNonNullNativeArray(arr);
+
     Local<Value> value;
     jsize startIndex = index;
     const jsize length = 1;
@@ -85,7 +87,9 @@ void ArrayElementAccessor::SetArrayElement(Isolate* isolate, const Local<Object>
     auto objectManager = runtime->GetObjectManager();
     auto context = isolate->GetCurrentContext();
 
-    auto arr = objectManager->GetJavaObjectByJsObject(array);
+    tns::JniLocalRef arr = objectManager->GetJavaObjectByJsObject(array);
+
+    assertNonNullNativeArray(arr);
 
     const string elementSignature = arraySignature.substr(1);
     jboolean isCopy = false;
@@ -198,3 +202,9 @@ Local<Value> ArrayElementAccessor::ConvertToJsValue(Isolate* isolate, ObjectMana
 
     return jsValue;
 }
+
+void ArrayElementAccessor::assertNonNullNativeArray(tns::JniLocalRef& arrayReference) {
+    if(arrayReference.IsNull()){
+        throw NativeScriptException("Failed calling indexer operator on native array. The JavaScript instance no longer has available Java instance counterpart.");
+    }
+}

test-app/runtime/src/main/cpp/ArrayElementAccessor.h

@@ -15,6 +15,7 @@ class ArrayElementAccessor {
 
     private:
         v8::Local<v8::Value> ConvertToJsValue(v8::Isolate* isolate, ObjectManager* objectManager, JEnv& env, const std::string& elementSignature, const void* value);
+        void assertNonNullNativeArray(tns::JniLocalRef& arrayReference);
 };
 }