Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add on-device patcher module for Magisk/KernelSU #34

Merged
merged 5 commits into from
Jan 25, 2025

Conversation

devnoname120
Copy link
Collaborator

@devnoname120 devnoname120 commented Jan 15, 2025

Tested and working on Android 15 QPR2 (library: /apex/com.android.btservices/lib64/libbluetooth_jni.so)

Notes:

  • I'm using a static build of busybox 1.36.1 for a standalone tar command and to create an xz shim that is required for radare2 to run without Termux.
  • I created a custom build of radare2 5.9.9 for Android with /data/local/tmp/aln_unzip/org.radare.radare2installer as a prefix.
    • It's necessary because the prefix is defined at compile time and radare2 wouldn't find the necessary files with the default build that has a Termux prefix instead.
    • This means that if you change the path /data/local/tmp/aln_unzip/org.radare.radare2installer without recompiling radare2 it won't work. You would have to recompile radare2.
  • The radare2 build is in a .tar.gz because ZIP fucks up file permissions, symbolic links, etc. whereas .tar.gz is natively designed to preserve them.
  • ARM64-only for now. ARM32 will follow in another PR once the ARM64 is well-tested.

@devnoname120 devnoname120 force-pushed the magisk-module-autopatcher branch from b1ef207 to ab67d7d Compare January 25, 2025 13:58
@devnoname120 devnoname120 changed the title [WIP] On-device patcher with Magisk module Add on-device patcher module for Magisk/KernelSU Jan 25, 2025
@devnoname120
Copy link
Collaborator Author

@kavishdevar Ready for review!

@kavishdevar kavishdevar merged commit 8d67f54 into kavishdevar:main Jan 25, 2025
@devnoname120 devnoname120 deleted the magisk-module-autopatcher branch January 26, 2025 12:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants