nick/godot
1
0
Fork 0
mirror of https://github.com/godotengine/godot.git synced 2026-03-24 21:27:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #115800 from m4gr3d/fix_javaclasswrapper_test_crashes

Browse Source 
Android: Fix JavaClassWrapper test crashes on API 26 and lower
This commit is contained in:
Thaddeus Crews
2026-02-04 11:04:19 -06:00
parent 72aeb54f9f 8e5c352b48
commit f438a4a12e
2 changed files with 16 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
platform/android/java_class_wrapper.cpp
View File

@@ -44,6 +44,11 @@ bool JavaClass::_call_method(JavaObject *p_instance, const StringName &p_method,
ERR_FAIL_NULL_V(env, false);
env->PushLocalFrame(p_argcount);
if (env->ExceptionCheck()) {
env->ExceptionDescribe();
env->ExceptionClear();
return false;
}
MethodInfo *method = nullptr;
for (MethodInfo &E : M->value) {

13
platform/android/jni_utils.cpp
View File

@@ -32,6 +32,9 @@
#include "api/java_class_wrapper.h"
// Maximum recursion depth allowed when converting variants to jobjects or vice-versa.
#define RECURSION_DEPTH_LIMIT 64
static jobject android_class_loader = nullptr;
static jmethodID load_class_method = nullptr;
@@ -93,12 +96,18 @@ String charsequence_to_string(JNIEnv *p_env, jobject p_charsequence) {
jobject _variant_to_jobject(JNIEnv *env, Variant::Type p_type, const Variant *p_arg, int p_depth) {
jobject ret = nullptr;
if (p_depth > Variant::MAX_RECURSION_DEPTH) {
if (p_depth > RECURSION_DEPTH_LIMIT) {
ERR_PRINT("Variant is too deep! Bailing.");
return ret;
}
env->PushLocalFrame(2);
if (env->ExceptionCheck()) {
env->ExceptionDescribe();
env->ExceptionClear();
return ret;
}
switch (p_type) {
case Variant::BOOL: {
jclass bclass = jni_find_class(env, "java/lang/Boolean");
@@ -271,7 +280,7 @@ String _get_class_name(JNIEnv *env, jclass cls, bool *array) {
}
Variant _jobject_to_variant(JNIEnv *env, jobject obj, int p_depth) {
ERR_FAIL_COND_V_MSG(p_depth > Variant::MAX_RECURSION_DEPTH, Variant(), "Variant is too deep! Bailing.");
ERR_FAIL_COND_V_MSG(p_depth > RECURSION_DEPTH_LIMIT, Variant(), "Variant is too deep! Bailing.");
if (obj == nullptr) {
return Variant();