diff --git a/library/jni/art/runtime.hpp b/library/jni/art/runtime.hpp index 6a1be22..403f75f 100644 --- a/library/jni/art/runtime.hpp +++ b/library/jni/art/runtime.hpp @@ -33,7 +33,7 @@ public: if (!RETRIEVE_MEM_FUNC_SYMBOL(SetJavaDebuggable, "_ZN3art7Runtime17SetJavaDebuggableEb")) { return false; } - return false; + return true; } }; diff --git a/library/jni/art/runtime/art_method.hpp b/library/jni/art/runtime/art_method.hpp index 8b079bb..b390ba2 100644 --- a/library/jni/art/runtime/art_method.hpp +++ b/library/jni/art/runtime/art_method.hpp @@ -111,7 +111,7 @@ public: jmethodID get_declared_constructors = JNI_GetMethodID(env, clazz, "getDeclaredConstructors", "()[Ljava/lang/reflect/Constructor;"); auto constructors = JNI_Cast( - JNI_CallObjectMethod(env, clazz, get_declared_constructors)); + JNI_CallObjectMethod(env, throwable, get_declared_constructors)); auto length = JNI_GetArrayLength(env, constructors); if (length < 2) { LOGE("Throwable has less than 2 constructors"); diff --git a/library/jni/lsplant.cc b/library/jni/lsplant.cc index 92876f6..3ac674a 100644 --- a/library/jni/lsplant.cc +++ b/library/jni/lsplant.cc @@ -20,6 +20,7 @@ #pragma clang diagnostic push #pragma clang diagnostic ignored "-Wunknown-pragmas" #pragma ide diagnostic ignored "ConstantConditionsOC" +#pragma ide diagnostic ignored "Simplify" #pragma ide diagnostic ignored "UnreachableCode" namespace lsplant { @@ -139,8 +140,9 @@ inline void UpdateTrampoline(decltype(entry_point_offset) offset) { } bool InitNative(JNIEnv *env, const HookHandler &handler) { - if (!handler.inline_hooker || !handler.inline_unhooker || !handler.art_symbol_resolver) + if (!handler.inline_hooker || !handler.inline_unhooker || !handler.art_symbol_resolver) { return false; + } if (!Runtime::Init(handler)) { LOGE("Failed to init runtime"); return false;