Java array check should check isHostObject.

woess · woess · commit f0e65540ab60 · 2022-06-07T15:42:09.000+02:00
diff --git a/graal-js/src/com.oracle.truffle.js/src/com/oracle/truffle/js/nodes/cast/OrdinaryToPrimitiveNode.java b/graal-js/src/com.oracle.truffle.js/src/com/oracle/truffle/js/nodes/cast/OrdinaryToPrimitiveNode.java
@@ -58,6 +58,7 @@
 import com.oracle.truffle.js.runtime.Errors;
 import com.oracle.truffle.js.runtime.JSArguments;
 import com.oracle.truffle.js.runtime.JSConfig;
+import com.oracle.truffle.js.runtime.JSRealm;
 import com.oracle.truffle.js.runtime.JSRuntime;
 import com.oracle.truffle.js.runtime.Strings;
 import com.oracle.truffle.js.runtime.objects.JSDynamicObject;
@@ -244,7 +245,7 @@ private Object tryInvokeForeignMethod(Object object, InteropLibrary interop, Str
     }
 
     public static boolean isJavaArray(Object object, InteropLibrary interop) {
-        return interop.hasArrayElements(object) && interop.isMemberReadable(object, "length");
+        return JSRealm.get(interop).getEnv().isHostObject(object) && interop.hasArrayElements(object) && interop.isMemberReadable(object, "length");
     }
 
     private PropertyGetNode getToString() {

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,7 @@`
`58`	`58`	`import com.oracle.truffle.js.runtime.Errors;`
`59`	`59`	`import com.oracle.truffle.js.runtime.JSArguments;`
`60`	`60`	`import com.oracle.truffle.js.runtime.JSConfig;`
	`61`	`+import com.oracle.truffle.js.runtime.JSRealm;`
`61`	`62`	`import com.oracle.truffle.js.runtime.JSRuntime;`
`62`	`63`	`import com.oracle.truffle.js.runtime.Strings;`
`63`	`64`	`import com.oracle.truffle.js.runtime.objects.JSDynamicObject;`
`@@ -244,7 +245,7 @@ private Object tryInvokeForeignMethod(Object object, InteropLibrary interop, Str`
`244`	`245`	`}`
`245`	`246`
`246`	`247`	`public static boolean isJavaArray(Object object, InteropLibrary interop) {`
`247`		`- return interop.hasArrayElements(object) && interop.isMemberReadable(object, "length");`
	`248`	`+ return JSRealm.get(interop).getEnv().isHostObject(object) && interop.hasArrayElements(object) && interop.isMemberReadable(object, "length");`
`248`	`249`	`}`
`249`	`250`
`250`	`251`	`private PropertyGetNode getToString() {`