From f489313054d81b2a80109d6aefda15e5d909e1e2 Mon Sep 17 00:00:00 2001
From: fankesyooni <qzmmcn@163.com>
Date: Sat, 8 Apr 2023 00:55:05 +0800
Subject: [PATCH] Modify remove checkingInternal function in ReflectionFactory,
 YukiMemberHookCreator

---
 .../hook/core/YukiMemberHookCreator.kt        |  2 +-
 .../hook/factory/ReflectionFactory.kt         | 28 +++----------------
 2 files changed, 5 insertions(+), 25 deletions(-)

diff --git a/yukihookapi/src/api/kotlin/com/highcapable/yukihookapi/hook/core/YukiMemberHookCreator.kt b/yukihookapi/src/api/kotlin/com/highcapable/yukihookapi/hook/core/YukiMemberHookCreator.kt
index b6f49a7e..f4cd3fc2 100644
--- a/yukihookapi/src/api/kotlin/com/highcapable/yukihookapi/hook/core/YukiMemberHookCreator.kt
+++ b/yukihookapi/src/api/kotlin/com/highcapable/yukihookapi/hook/core/YukiMemberHookCreator.kt
@@ -158,7 +158,7 @@ class YukiMemberHookCreator @PublishedApi internal constructor(
         else -> Result().await {
             when {
                 isDisableCreatorRunHook.not() && hookClass.instance != null -> runCatching {
-                    hookClass.instance?.apply { checkingInternal(); checkingDangerous() }
+                    hookClass.instance?.checkingDangerous()
                     it.onPrepareHook?.invoke()
                     preHookMembers.forEach { (_, m) -> m.hook() }
                 }.onFailure {
diff --git a/yukihookapi/src/api/kotlin/com/highcapable/yukihookapi/hook/factory/ReflectionFactory.kt b/yukihookapi/src/api/kotlin/com/highcapable/yukihookapi/hook/factory/ReflectionFactory.kt
index a96e29ae..c4734d96 100644
--- a/yukihookapi/src/api/kotlin/com/highcapable/yukihookapi/hook/factory/ReflectionFactory.kt
+++ b/yukihookapi/src/api/kotlin/com/highcapable/yukihookapi/hook/factory/ReflectionFactory.kt
@@ -29,7 +29,6 @@
 
 package com.highcapable.yukihookapi.hook.factory
 
-import com.highcapable.yukihookapi.YukiHookAPI
 import com.highcapable.yukihookapi.hook.bean.CurrentClass
 import com.highcapable.yukihookapi.hook.bean.GenericClass
 import com.highcapable.yukihookapi.hook.core.finder.base.rules.ModifierRules
@@ -40,7 +39,6 @@ import com.highcapable.yukihookapi.hook.core.finder.members.MethodFinder
 import com.highcapable.yukihookapi.hook.core.finder.tools.ReflectionTool
 import com.highcapable.yukihookapi.hook.core.finder.type.factory.*
 import com.highcapable.yukihookapi.hook.type.java.*
-import com.highcapable.yukihookapi.hook.xposed.bridge.status.YukiXposedModuleStatus
 import com.highcapable.yukihookapi.hook.xposed.parasitic.AppParasitics
 import dalvik.system.BaseDexClassLoader
 import java.lang.reflect.*
@@ -326,10 +324,7 @@ inline fun Class<*>.constructor(initiate: ConstructorConditions = { emptyParam()
  * 如果当前实例不存在泛型将返回 null
  * @return [GenericClass] or null
  */
-fun Class<*>.generic(): GenericClass? {
-    checkingInternal()
-    return genericSuperclass?.let { (it as? ParameterizedType?)?.let { e -> GenericClass(e) } }
-}
+fun Class<*>.generic() = genericSuperclass?.let { (it as? ParameterizedType?)?.let { e -> GenericClass(e) } }
 
 /**
  * 获得当前 [Class] 的泛型父类
@@ -345,10 +340,8 @@ inline fun Class<*>.generic(initiate: GenericClass.() -> Unit) = generic()?.appl
  * @param ignored 是否开启忽略错误警告功能 - 默认否
  * @return [CurrentClass]
  */
-inline fun <reified T : Any> T.current(ignored: Boolean = false): CurrentClass {
-    javaClass.checkingInternal()
-    return CurrentClass(javaClass, instance = this).apply { isShutErrorPrinting = ignored }
-}
+inline fun <reified T : Any> T.current(ignored: Boolean = false) =
+    CurrentClass(javaClass, instance = this).apply { isShutErrorPrinting = ignored }
 
 /**
  * 获得当前实例的类操作对象
@@ -413,17 +406,4 @@ inline fun Class<*>.allConstructors(isAccessible: Boolean = true, result: (index
  * @param result 回调 - ([Int] 下标,[Field] 实例)
  */
 inline fun Class<*>.allFields(isAccessible: Boolean = true, result: (index: Int, field: Field) -> Unit) =
-    declaredFields.forEachIndexed { p, it -> result(p, it.also { e -> e.isAccessible = isAccessible }) }
-
-/**
- * 检查内部类调用
- * @throws RuntimeException 如果遇到非法调用
- */
-@PublishedApi
-internal fun Class<*>.checkingInternal() {
-    if (name == YukiXposedModuleStatus.IMPL_CLASS_NAME) return
-    if (name == classOf<YukiHookAPI>().name || name.startsWith("com.highcapable.yukihookapi.hook")) throw RuntimeException(
-        "!!!DO NOT ALLOWED!!! You cannot hook or reflection to call the internal class of the YukiHookAPI itself, " +
-                "The called class is [$this]"
-    )
-}
\ No newline at end of file
+    declaredFields.forEachIndexed { p, it -> result(p, it.also { e -> e.isAccessible = isAccessible }) }
\ No newline at end of file