android/SukiSU-Ultra
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

kernel: no need to remark process on post-fs-data and boot-completed

Browse Source 
- Remark on post-fs-data may unmark zygote unexpectedly, and there is no
necessity to remark on these stages, so simply remove them.
This commit is contained in:
5ec1cff
2025-11-15 17:19:45 +08:00
committed by ShirkNeko
parent 011a740217
commit 70f722e614
2 changed files with 4 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
kernel/ksud.c
View File

@@ -33,9 +33,6 @@
#include "klog.h" // IWYU pragma: keep
#include "ksud.h"
#include "selinux/selinux.h"
#ifndef CONFIG_KSU_SUSFS
#include "syscall_hook_manager.h"
#endif // #ifndef CONFIG_KSU_SUSFS
#include "throne_tracker.h"
bool ksu_module_mounted __read_mostly = false;
@@ -93,10 +90,6 @@ void on_post_fs_data(void)
done = true;
pr_info("on_post_fs_data!\n");
ksu_load_allow_list();
#ifndef CONFIG_KSU_SUSFS
pr_info("mark tif for running process\n");
ksu_mark_running_process();
#endif // #ifndef CONFIG_KSU_SUSFS
ksu_observer_init();
// sanity check, this may influence the performance
stop_input_hook();
@@ -142,12 +135,6 @@ void on_boot_completed(void){
ksu_boot_completed = true;
pr_info("on_boot_completed!\n");
track_throne(true);
#ifndef CONFIG_KSU_SUSFS
// remark process, we don't want to mark other init
// forked process excepte zygote and adbd
ksu_unmark_all_process();
ksu_mark_running_process();
#endif // #ifndef CONFIG_KSU_SUSFS
}
#ifndef CONFIG_KSU_SUSFS
@@ -346,9 +333,6 @@ int ksu_handle_execveat_ksud(int *fd, struct filename **filename_ptr,
task_work_add(init_task, &on_post_fs_data_cb, TWA_RESUME);
}
rcu_read_unlock();
#ifndef CONFIG_KSU_SUSFS
ksu_set_task_tracepoint_flag(current); // we are zygote!
#endif
stop_execve_hook();
}
@@ -685,4 +669,4 @@ void ksu_ksud_exit(void)
#endif
is_boot_phase = false;
volumedown_pressed_count = 0;
}
}

5
kernel/syscall_hook_manager.c
View File

@@ -242,7 +242,7 @@ static inline bool check_syscall_fastpath(int nr)
}
}
// Unmark init's child that are not zygote or adbd
// Unmark init's child that are not zygote, adbd or ksud
int ksu_handle_init_mark_tracker(const char __user **filename_user)
{
char path[64];
@@ -253,7 +253,8 @@ int ksu_handle_init_mark_tracker(const char __user **filename_user)
memset(path, 0, sizeof(path));
strncpy_from_user_nofault(path, *filename_user, sizeof(path));
if (likely(strstr(path, "/app_process") == NULL && strstr(path, "/adbd") == NULL)) {
if (likely(strstr(path, "/app_process") == NULL && strstr(path, "/adbd") == NULL && strstr(path, "/ksud") == NULL)) {
pr_info("hook_manager: unmark %d exec %s", current->pid, path);
ksu_clear_task_tracepoint_flag_if_needed(current);
}