tiann / KernelSU

A Kernel based root solution for Android
https://kernelsu.org
GNU General Public License v3.0
10.38k stars 1.69k forks source link

KernelSU Manager app not working #886

Closed salvogiangri closed 1 year ago

salvogiangri commented 1 year ago

Describe the bug I'm currently trying to include the KernelSU Manager app in my custom ROM via Samsung's Preloaded apps feature. However when I open the manager app, it prints KernelSU is not installed. This doesn't happens when manually installing the app

To Reproduce Flash the ROM, pass the setup wizard and open the app.

Expected behaviour The app should show the currently installed KernelSU version.

Screenshots

Smartphone (please complete the following information):

Additional context I did take a look at the code in the driver and I saw it actually expects the package to start with "/data/app/" and end with "/base.apk": https://github.com/tiann/KernelSU/blob/fb103472c631caf7feae97634dba071ae5ab3ce4/kernel/manager.c#L50-L51

Instead the app is installed at:

dm3q:/ $ pm list packages -f | grep kernelsu                                                                                                              
package:/data/app/KernelSU/KernelSU.apk=me.weishu.kernelsu

Here's some additional logs from dmesg:

dm3q:/ $ dmesg | grep kernelsu
[ 1431.394866] [7:weishu.kernelsu:16351]  KernelSU: become_manager: param pkg: /me.weishu.kernelsu
[ 1431.477858] [7:weishu.kernelsu:16351]  KernelSU: become_manager: param pkg: /me.weishu.kernelsu
[ 1437.861908] [0:            iod:  657]  !@ Read_top(KB): weishu.kernelsu(16351) 4 kworker/u16:5(320) 4 system_server(1327) 4
tiann commented 1 year ago

559

bemerguy commented 7 months ago

I have the same problem, and it was working before... where's the fix?

r8q:/ # pm list packages -f | grep kernelsu package:/data/app/~~7RCKgnngM5y6cFasOYIRnw==/me.weishu.kernelsu-jaCJ7d4mGPW3Yp4vQ0MZdg==/base.apk=me.weishu.kernelsu