From a5d6978b32569f10cdde9d21584b525e5887f60d Mon Sep 17 00:00:00 2001
From: ShirkNeko <109797057+ShirkNeko@users.noreply.github.com>
Date: Sat, 17 May 2025 18:30:19 +0800
Subject: [PATCH] kernel: sucompat: sucompat toggle support for non-kp
 (tiann#2506) This is done like how vfs_read_hook, input_hook and execve_hook
 is disabled. While this is not exactly the same thing, this CAN achieve the
 same results. The complete disabling of all KernelSU hooks.

While this is likely unneeded, It keeps feature parity to non-kprobe builds.

adapted from upstream:
	kernel: Allow to re-enable sucompat - tiann/KernelSU@4593ae8

Rejected: tiann/KernelSU#2506

Signed-off-by: backslashxx <118538522+backslashxx@users.noreply.github.com>
---
 kernel/sucompat.c | 52 +++++++----------------------------------------
 1 file changed, 7 insertions(+), 45 deletions(-)

diff --git a/kernel/sucompat.c b/kernel/sucompat.c
index a4e914e7..735a23e1 100644
--- a/kernel/sucompat.c
+++ b/kernel/sucompat.c
@@ -27,20 +27,16 @@
 #define SU_PATH "/system/bin/su"
 #define SH_PATH "/system/bin/sh"
 
-bool ksu_faccessat_hook __read_mostly = true;
-bool ksu_stat_hook __read_mostly = true;
-bool ksu_execve_sucompat_hook __read_mostly = true;
-bool ksu_execveat_sucompat_hook __read_mostly = true;
+#ifndef CONFIG_KSU_HOOK_KPROBES
+static bool ksu_sucompat_non_kp __read_mostly = true;
+#endif
+
 #ifndef CONFIG_KSU_SUSFS_SUS_SU
 bool ksu_devpts_hook __read_mostly = true;
 #endif
 
 extern void ksu_escape_to_root();
 
-#ifndef CONFIG_KSU_HOOK_KPROBES
-static bool ksu_sucompat_non_kp __read_mostly = true;
-#endif
-
 static void __user *userspace_stack_buffer(const void *d, size_t len)
 {
 	/* To avoid having to mmap a page in userspace, just write below the stack
@@ -73,12 +69,6 @@ int ksu_handle_faccessat(int *dfd, const char __user **filename_user, int *mode,
  	}
 #endif
 
-#ifndef CONFIG_KSU_HOOK_KPROBES
-	if (!ksu_faccessat_hook) {
-		return 0;
-	}
-#endif
-
 	if (!ksu_is_allow_uid(current_uid().val)) {
 		return 0;
 	}
@@ -130,12 +120,6 @@ int ksu_handle_stat(int *dfd, const char __user **filename_user, int *flags)
  	}
 #endif
 
-#ifndef CONFIG_KSU_HOOK_KPROBES
-	if (!ksu_stat_hook){
-		return 0;
-	}
-#endif
-
 	if (!ksu_is_allow_uid(current_uid().val)) {
 		return 0;
 	}
@@ -183,12 +167,6 @@ int ksu_handle_execveat_sucompat(int *fd, struct filename **filename_ptr, void *
  	}
 #endif
 
-#ifndef CONFIG_KSU_HOOK_KPROBES
-	if (!ksu_execveat_sucompat_hook) {
-		return 0;
-	}
-#endif
-
 	if (unlikely(!filename_ptr)) {
     return 0;
 	}
@@ -225,12 +203,6 @@ int ksu_handle_execve_sucompat(int *fd, const char __user **filename_user, void
  	}
 #endif
 
-#ifndef CONFIG_KSU_HOOK_KPROBES
-	if (!ksu_execve_sucompat_hook) {
-		return 0;
-	}
-#endif
-
 	if (unlikely(!filename_user))
 		return 0;
 
@@ -254,7 +226,7 @@ int ksu_handle_execve_sucompat(int *fd, const char __user **filename_user, void
 int ksu_handle_devpts(struct inode *inode)
 {
 #ifndef CONFIG_KSU_HOOK_KPROBES
-	if (!ksu_devpts_hook) {
+	if (!ksu_sucompat_non_kp) {
 		return 0;
 	}
 #endif
@@ -375,13 +347,8 @@ void ksu_sucompat_init()
 	su_kps[4] = init_kprobe(SYS_FSTATAT64_SYMBOL, newfstatat_handler_pre);
 	su_kps[5] = init_kprobe("pts_unix98_lookup", pts_unix98_lookup_pre);
 #else
-	ksu_faccessat_hook = true;
-	ksu_stat_hook = true;
-	ksu_execve_sucompat_hook = true;
-	ksu_execveat_sucompat_hook = true;
-	ksu_devpts_hook = true;
 	ksu_sucompat_non_kp = true;
-	pr_info("ksu_sucompat_init: hooks enabled: execve/execveat_su, faccessat, stat, devpts\n");
+	pr_info("ksu_sucompat_init: hooks enabled: execve/execveat_su, faccessat, stat\n");
 #endif
 }
 
@@ -393,13 +360,8 @@ void ksu_sucompat_exit()
 		destroy_kprobe(&su_kps[i]);
 	}
 #else
-	ksu_faccessat_hook = false;
-	ksu_stat_hook = false;
-	ksu_execve_sucompat_hook = false;
-	ksu_execveat_sucompat_hook = false;
-	ksu_devpts_hook = false;
 	ksu_sucompat_non_kp = false;
-	pr_info("ksu_sucompat_exit: hooks disabled: execve/execveat_su, faccessat, stat, devpts\n");
+	pr_info("ksu_sucompat_exit: hooks disabled: execve/execveat_su, faccessat, stat\n");
 #endif
 }