-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 bug in granting root for apps on BlissOS #2113
Comments
Same problem here. It seems like something in the shell is "cloaking" the su call Maybe it is releated to:
|
exec |
/system/bin/strace su says |
It seems that this kprobe failed: https://github.com/tiann/KernelSU/blob/main/kernel/sucompat.c#L265 |
Looks like it has succeeded
|
this indicate it failed to unregister ? |
@tiann I just remember something: When I test #2041 , I test it in kernel 6.1.84. And it work perfectly fine from granting apps to using modules. After the PR got merged I upstream the kernel and then I have reports about KSU bugging. I am currently narrowing down to see up until which version does the bug happen, currently it's around 6.1.84 ~ 6.1.89 |
6.1.85 is when it happen |
I just thought, it could be that ksud init broke because modules also don't work, modules.img does not get mounted. |
6.1.84 works perfectly fine with 1.0.1, here's an image for you to test |
I have tested 6.1.84 and can confirm everything works there, but I meant in the builds with newer kernel, modules don't work or even get mounted. |
try to get log, maybe there are more clue than faccessat() is broken |
I tried even with loglevel=7 and checked in logcat but no clue yet. The dmesg doesn't differ much from that of the working version (6.1.84). But I saw that KernelSU has CONFIG_KSU_DEBUG, it could possibly help in getting more logs.
|
@tiann found the commit: Revert this and it KSU work as expected now. And I even test this on kernel 6.11.2 |
so this seems affected all x86 platform not just Android-x86 based like us |
please check |
when KSU broke or when it's working ? |
broken, and the kernel image |
|
So as a workaround, I saw this patch in LKML: I applied it and test on my kernel 6.11 and it's working so far However, I still think that we should fix KSU itself. |
Build this on 6.1 requires disabling BHI mitigations by setting |
@tiann I found this post : https://blog.hakuya.moe/post/13 |
Finalize the temp fix: android-generic/kernel-zenith@5648396 Because I don't want to fully revert, I put the option under a cmdline |
Nice, any idea when the updated BlissOs will be online? |
updated, please check |
Thx, I will check that today!! |
nope, I should have read your old post, let me try |
WORKING! THX SO MUCH! |
Please check before submitting an issue
Describe the bug
(This issue may look like #2095, but I can't verify yet)
Currently with latest KSU on BlissOS, tested on kernel 6.1 & 6.6 & 6.9 & 6.11, there are some bugs that I started to notice: While some apps like Aurora Store, Neo Store & FoxLog can be able to grant root and use normally, some are broken in functions or just straight up not working:
su
if using/system/bin/su
, and when in su no $PATH is set other than Termux or KSU own path.Allow Root
even set withrootexplorer.root
orsystem
template. I tried another app Amaze File Manager with theirRoot Explorer
&Use legacy listing for root
and it can do things like show the whole/data
but can't access anything in/data/app
.su
or/system/bin/su
All recent BlissOS builds with KernelSU 1.0.1 have this issue. And currently they are on kernel 6.1. I bumped kernel to 6.6 to test locally and it show the same behaviour. For newer kernel that are on stable branch, the same behaviour can be seen on this PR I made . Recently I do try to build for kernel 6.11 and it also have the same behaviour (here is my fork for 6.11).
To Reproduce
Expected behavior
Apps can be able to grant root and work as expected
Screenshots
An example of Termux can't use
su
can be seen here : #1785 (comment)Logs
will provide soon
Device info
Additional context
One last thing that I tested is to downgrade to 0.9.2 as some people suggested , it only work with kernel 6.1 & 6.6. If you port it to higher kernel like 6.9, it will have the exact same issue.
The text was updated successfully, but these errors were encountered: