-
-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
KSU report "not installed" on installed system, but "installed" on Live boot when using BlissOS #1783
Comments
@tiann latest versions are even worse when in Live Boot, even though it said working, granting Termux can't be able to use |
I've downgraded to 0.9.2 and it's actually recognized on installed system, starting to build on 0.9.3 is when the bug start happenening |
In order to update to higher version, this bug need to be fixed: tiann/KernelSU#1783 Signed-off-by: Huy Minh <buingoc67@gmail.com>
Did you tried 0.9.5? |
yes. Even 1.0 |
Which dentry is NULL? old_dentry or new_dentry? |
It's been a while so I forgot, I'll check back when I got time |
Update: I pulled KSU to latest commit currently (9bcdff1) and boot in debug mode, it still not recognize the manager. There's some output on the console though
|
The same log does show up on Live boot, but it actually continue find the manager
|
So with the help of ChatGPT, I made a patch to print some more info to debug:
It seems like KSU can be able to look at the actual dir instead of just |
On Android-x86 (or BlissOS) it initialize Android by using switch_root or chroot, when checking a path with dentry_path_raw() it will show the whole real path instead of the path that we want. Relax the checking requirement by using strstr to look for "/system/packages.list" in the string instead of requiring the path to be "/system/packages.list" This fixes tiann#1783 Signed-off-by: hmtheboy154 <buingoc67@gmail.com>
On Android-x86 (or BlissOS) it initialize Android by using switch_root or chroot, when checking a path with dentry_path_raw() it will show the whole real path instead of the path that we want. Relax the checking requirement by using strstr to look for "/system/packages.list" in the string instead of requiring the path to be "/system/packages.list" This fixes tiann#1783 Signed-off-by: hmtheboy154 <buingoc67@gmail.com>
The same thing happened with LKM in lineage-21.0-20240923-nightly-oriole pixel 6. |
On Android-x86 (or BlissOS) it initialize Android by using switch_root or chroot, when checking a path with dentry_path_raw() it will show the whole real path instead of the path that we want. Relax the checking requirement by using strstr to look for "/system/packages.list" in the string instead of requiring the path to be "/system/packages.list" This fixes tiann#1783 Signed-off-by: hmtheboy154 <buingoc67@gmail.com>
Glad to see this is fixed. Can we get an updated release? |
It's already available on all public BlissOS builds |
Hmm. I'm using what I thought was the latest build: BlissOS 16.9.7. Am I missing something? |
I had tried just now, it's still not working on B.OS 16.9.7 |
Give me the iso name & what show in KSU Manager |
you too |
On Android-x86 (or BlissOS) it initialize Android by using switch_root or chroot, when checking a path with dentry_path_raw() it will show the whole real path instead of the path that we want. Relax the checking requirement by using strstr to look for "/system/packages.list" in the string instead of requiring the path to be "/system/packages.list" This fixes tiann#1783 Signed-off-by: hmtheboy154 <buingoc67@gmail.com>
I tested them yesterday and today and only could get a shell root activating adb shell in kernelsu, and doing this hack in the adb shell
This gives me a su shell using bashsudo I hope you get it fixed! I love your project! Really awesome! |
Yea I will have to create a new Github Issue here, I do notice that on Termux I can only get |
If I try
|
I have just found something else that might help you to solve the problem, but maybe you know it already: |
I didn't know about this...... interesting though |
It may help you. Maybe there is a way to trace how the toybox executable calls: xargs -0 -n1 su -c and then reproducing the syscall somehow. It seems like through all this piping stuff and toybox, an environment is created where this: |
Made new issue #2113 . If you have any new finding please go there |
On Android-x86 (or BlissOS) it initialize Android by using switch_root or chroot, when checking a path with dentry_path_raw() it will show the whole real path instead of the path that we want. Relax the checking requirement by using strstr to look for "/system/packages.list" in the string instead of requiring the path to be "/system/packages.list" This fixes tiann#1783 Signed-off-by: hmtheboy154 <buingoc67@gmail.com>
On Android-x86 (or BlissOS) it initialize Android by using switch_root or chroot, when checking a path with dentry_path_raw() it will show the whole real path instead of the path that we want. Relax the checking requirement by using strstr to look for "/system/packages.list" in the string instead of requiring the path to be "/system/packages.list" This fixes tiann#1783 Signed-off-by: hmtheboy154 <buingoc67@gmail.com>
On Android-x86 (or BlissOS) it initialize Android by using switch_root or chroot, when checking a path with dentry_path_raw() it will show the whole real path instead of the path that we want. Relax the checking requirement by using strstr to look for "/system/packages.list" in the string instead of requiring the path to be "/system/packages.list" This fixes tiann#1783 Signed-off-by: hmtheboy154 <buingoc67@gmail.com>
On Android-x86 (or BlissOS) it initialize Android by using switch_root or chroot, when checking a path with dentry_path_raw() it will show the whole real path instead of the path that we want. Relax the checking requirement by using strstr to look for "/system/packages.list" in the string instead of requiring the path to be "/system/packages.list" This fixes tiann#1783 Signed-off-by: hmtheboy154 <buingoc67@gmail.com>
On Android-x86 (or BlissOS) it initialize Android by using switch_root or chroot, when checking a path with dentry_path_raw() it will show the whole real path instead of the path that we want. Relax the checking requirement by using strstr to look for "/system/packages.list" in the string instead of requiring the path to be "/system/packages.list" This fixes tiann#1783 Signed-off-by: hmtheboy154 <buingoc67@gmail.com>
On Android-x86 (or BlissOS) it initialize Android by using switch_root or chroot, when checking a path with dentry_path_raw() it will show the whole real path instead of the path that we want. Relax the checking requirement by using strstr to look for "/system/packages.list" in the string instead of requiring the path to be "/system/packages.list" This fixes tiann#1783 Signed-off-by: hmtheboy154 <buingoc67@gmail.com>
On Android-x86 (or BlissOS) it initialize Android by using switch_root or chroot, when checking a path with dentry_path_raw() it will show the whole real path instead of the path that we want. Relax the checking requirement by using strstr to look for "/system/packages.list" in the string instead of requiring the path to be "/system/packages.list" This fixes tiann#1783 Signed-off-by: hmtheboy154 <buingoc67@gmail.com>
Please check before submitting an issue
Describe the bug
Recent KernelSU version report "not installed" on BlissOS when the OS is installed. On Live Boot, it works normally and report the version on the KernelSU Manager app.
To Reproduce
Get a BlissOS build with recent KernelSU in the kernel & Manager
https://sourceforge.net/projects/blissos-x86/files/Official/BlissOS16/FOSS/Go/Bliss-Go-v16.9.5-x86_64-OFFICIAL-foss-20240529.iso/download
https://drive.google.com/file/d/1tQAVbIygV4DqEyDpmGxsssF2whASzgK0/view?usp=sharing
https://drive.google.com/file/d/19m0UOp70ZiNp74MHNswfjGsyqPdexDWP/view?usp=sharing
Install it on a VM or baremetal
Check KernelSU Manager
Expected behavior
It should work & report the version
Screenshots
report "not installed" on installed system
report "installed" on Live boot
currently where it keep looping
Logs
none, logs on the Manager can not dump anything
Device info
Additional context
According to the screenshot, it's being looped here:
https://github.com/tiann/KernelSU/blob/main/kernel/core_hook.c#L189
The text was updated successfully, but these errors were encountered: