br0kenpixel/ReZygisk
1
0
Fork 0
mirror of https://github.com/PerformanC/ReZygisk.git synced 2025-09-06 06:37:01 +00:00
Code Issues Packages Projects Releases 1 Wiki Activity

improve: update_mnt_ns logging; add: comment about "rooted denylisted" apps

Browse Source 
This commit improves logging for "update_mnt_ns" function, which now specifies which state it will update the mns to, for easier debugging. It also adds a note about the possibility of having apps with rooted permissions and also be denylisted in Magisk, causing weird behavior.
This commit is contained in:
ThePedroo
2025-04-24 01:33:53 -03:00
parent b24c74ac0d
commit a460c54d08
1 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
loader/src/injector/hook.cpp
View File

@@ -156,7 +156,13 @@ bool update_mnt_ns(enum mount_namespace_state mns_state, bool dry_run) {
return false;
}
LOGD("set mount namespace to [%s] fd=[%d]\n", ns_path, updated_ns);
const char *mns_state_str = NULL;
if (mns_state == Clean) mns_state_str = "clean";
else if (mns_state == Module) mns_state_str = "module";
else if (mns_state == Rooted) mns_state_str = "rooted";
else mns_state_str = "unknown";
LOGD("set mount namespace to [%s] fd=[%d]: %s", ns_path, updated_ns, mns_state_str);
if (setns(updated_ns, CLONE_NEWNS) == -1) {
PLOGE("Failed to set mount namespace [%s]", ns_path);
close(updated_ns);
@@ -183,6 +189,11 @@ DCL_HOOK_FUNC(int, unshare, int flags) {
update_mnt_ns(Module, false);
}
/* INFO: There might be cases, specifically in Magisk, where the app is in
DenyList but also has root privileges. For those, it is up to the
user remove it, and the weird behavior is expected, as the weird
user behavior. */
old_unshare(CLONE_NEWNS);
}