Magisk-Modules-Alt-Repo / submission

Information regarding the submission of Magisk Modules to the Alt-Repo.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[Module] Simple BootloopSaver

ez-me opened this issue · comments

ID: simple_bootloopsaver
Name: Simple Bootloop Saver
Description: Protect your system from bootloop caused by Magisk modules by turning off all your modules and restart your system if bootloop is detected.
Link: https://github.com/ez-me/Simple_BootloopSaver
Source code: https://github.com/ez-me/Simple_BootloopSaver

The module might not handle if there is multiple zygote existing at same time, example, If there is two process with same name zygote64, it might thing that the system is bootlooping

That explains why I sometimes saw like 3 ids for the zygote process in the logs.

That led to my phone rebooting with the module after I was already in the launcher, using it normally.

However, with my latest commit, it checks the 1st and 4th IDs, which (at least for me), results in it checking the same effective ID twice, without the multiple-zygote problem.

Not just zygote itself, but any process with the name "zygote(64)". Zygote fork itself into app processes or system server, but its process name might not be changed yet. So there must be some steps such as checking if the process is executing from app_process and is init's child process, ...

You can rely on init.svc_debug_pid.zygote prop

I made a new commit that uses getprop init.svc_debug_pid.zygote rather than pidof zygote64

I rebooted normally, then caused a bootloop, then normally a few times again.
Hasn't triggered without a real cause now (multiple zygotes...), and the log looks fine.

I will add your repo soon when I have time

Your repo has been added, pls check the invitation!