Magisk General Support / Discussion

Search This thread

zgfg

Senior Member
Oct 10, 2016
10,766
9,397
Redmi K20 / Xiaomi Mi 9T
Xiaomi Mi 11
The reason to restore the images is to make sure the soon to be inactive slot is stock for the next OTA update.
I know that VERY well - I said that you disk-dump the boot img manually ti restore stock

If you didn't install custom recovery, it is already stock (if not, flash/disk-dump the stock back)

SAR on A11/A12 is R E A D - O N L Y (Magisk makes makes only systemlessly changes!!!), hence complete SAR (System, Vendor, etc) is untouched by your previously installed Magisk

Magisk modules, db etc are on /data/adb and hence not part of OTA matters

I said, if you understand how these things work (particularly about the Systemless rooting mechanism), you don't need to Uninstall Magisk but instead to pinpoint what is necessary (hints above) manually.
If not, then go by the book...

I would personally kill myself
if I would have to reinstall and reconfigure everything, every time (lot of troubles with some of them to make them work or to properly configure), Viper4Android, Hide My Applist, etc

Therefore I ALWAYS just manually restore the Boot (and Recovery if needed), update the system, patch/flash the new Boot, and happily continue using the Magisk with all my previous modules and everything

But ok, if you are afraid of and/or unexperienced, go by the book
 
Last edited:

BillGoss

Senior Member
Sep 2, 2010
5,921
5,245
Sydney
OnePlus 8T
Google Pixel 8 Pro
@dl200010
FYI, on the OnePlus 8T on OOS 11 there used to be two ways of updating:
1. Let the System Update download and install the OTA.
2. Manually download the OTA and then do a manual install in System Updater.
Using the first method your could not use Magisk Install to inactive. That always led to problems.
But the with second method, Magisk Install to inactive worked like a charm.
Now on OOS 12 there are still two options:
1. System update downloads and installs
2. Manually download the update and use a OnePlus apk that lets you do a manual install.
And these two options behave the same way as the OOS 11 versions.

It seems that the update_engine used by the two methods are different and behave differently.

I've written up the various options for rooting and keeping root with OTAs for the 8T. You might want to have a look at it and see if you can get ideas on what you can do for your OnePlus 9.
 

pndwal

Senior Member
Your discussion with @pndwal is getting very interesting.
Sorry if I did not properly catch-up the problem but some thoughts you (both) may think of:

AFAIK, problem with Systemless hosts applies only to (Pixel) devices with f2fs System/SAR - if your System is ext4 filesystem type, no problem.
You can check the filesystem type by eeading (before unrooting) /vendor/etc/fstab.*
F2fs loopback fixes were around in 2017... Huawei and Motorola were slow to fix kernel bug and seems loopback module and other fixes were needed for devices produced till late 2018... I believe Pixel fixed this in their kernels earlier...

I doubt very much this bug has made it into late OnePlus kernels.
Second, Uninstall Magisk (with/without Restore images) would uninstall all modules (incl. Systemless hosts) - wouldn't it?
In that case, manually uninstalling Systemless hosts before that step would not make any difference
No; Uninstall Magisk, Complete Uninstall does this, but Restore Images doesn't touch Magisk configuration files... It does:
make sure the soon to be inactive slot is stock for the next OTA update
by 'restoring partitions modified by Magisk back to stock from backups made at install in order to pass pre-OTA block verifications' however...

/data/avb configuration remains available till next Magisk installation and won't be rebuilt either...
Third, if I understand correctly, you are afraid now that due to Delta OTA, you cannot extract the new boot.img to be able to patch.
Also, you say that although you apply Uninstall Magisk (with Restore Images) before OTA but without rebooting, that you are still running root after the Delta OTA

Well, in that case you can disk-dump the Boot from that inactive slot, after the Delta OTA finishes.
For disk-dump you need root - you say you still have it (if I understood properly).
You need to know what's your current slot, hence execute (all from Terminal) and before starting OTA:
getprop ro.boot.slot_suffix

Suppose it returns _b, that means that before OTA, your active slot is B and that your OTA (full or incremental) would apply tothe inactive slot A

After the (incremental) OTA, disk-dump the boot_a.img (ie Boot img from the just OTA updated inactive A slot):
su
dd if=/dev/block/bootdevice/by-name/boot_a of=/sdcard/Download/boot img

You can patch now that boot.img (disk-dumped to Download folder on your Internal memory) by Patch method from the Magisk app

Finally, you need to disk-dump the patched img back to the still inactive slot A:
dd if=/sdcard/Download/<PATCHED IMG> of=/dev/block/bootdevice/by-name/boot_a

And then you are ready to reboot - reboot will switch from the slot B to slot A, that was just OTA updated with Magisk already patched and flashed to it's Boot partiition

---

Finally, I don't see why Magisk should be uninstalled before OTA on these A/B devices.
Basically because 1) delta OTA won't start if pre-OTA block verifications fail, and 2) boot (and other?) partition(s) has/have been altered by Magisk, so this will bork incremental update even on A/B since inactive slot patching is generated as a function of original active slot data and updated bytes from OTA...
I know it's safer for many reasons but if you have full control of the situation (understanding how things work, modules you had installed and those potentially incompatible with the OTA), you could instead:

- Restore manually the stock boot.img by disk-dumping before OTA

- Disable only the critical modules before OTA

- After OTA updating and patching/disk-dumping the boot.img (in the still inactive slot) and after rebooting (where the updated slot now becomes active), you will boot and Magisk will be up and running with all (not-disabled) modules, previously granted root, DenyList) - it saves you a lot of time configuring all that once again.
Ie, Magisk database, modules, everything is on /data/adb and therefore it's not affected by OTA
And this is also the case with what he been doing! - Uninstall, Restore Images also leaves all this intact...
Well, suggestions above are not by the Bible (TJW), but we should be able also to think and act out-of-the-box. And generally I like and did experiment a lot with the disk-dumping (I don't have Pixel and incremental OTA but anyway)
I think you should be able to use Install to Inactive Slot to 'preserve Magisk after installation' on your A/B Xiaomi whenever an OTA arrives... There are clearly some anomalies w/ OnePlus, but generally the method is not restricted to Pixels. PW
 

J.Michael

Recognized Contributor
Jan 20, 2018
2,496
3,050
Samsung Galaxy Tab A series
The reason to restore the images is to make sure the soon to be inactive slot is stock for the next OTA update.
Maybe I'm misunderstanding your frame of reference, but if "next OTA update" means a future update, after the one you are trying to accept now, then I think the state of the currently-active-soon-to-be-inactive slot will be ignored in that not yet released future OTA update.

If by "next OTA update" you meant the update you are currently trying to accept, then excuse the ring.
 
Maybe I'm misunderstanding your frame of reference, but if "next OTA update" means a future update, after the one you are trying to accept now, then I think the state of the currently-active-soon-to-be-inactive slot will be ignored in that not yet released future OTA update.

If by "next OTA update" you meant the update you are currently trying to accept, then excuse the ring.
You taking the current update depends on if you "restore images" the last time you updated. Because the inactive slot is the one taking the update. Not the current active one.
 

J.Michael

Recognized Contributor
Jan 20, 2018
2,496
3,050
Samsung Galaxy Tab A series
You taking the current update depends on if you "restore images" the last time you updated. Because the inactive slot is the one taking the update. Not the current active one.
Yes, the inactive slot will receive the update. But what it receives is a combination of the active slot and the OTA update. That's why its so important that you return the active slot to the state expected by the people who issued the update. cf. @pndwal's notes.

If you can download a file-set containing a full image of the updated ROM, then you can burn that to either slot. The only problem would be if the official update installer refuses to run unless it is satisfied with the "current" state.

In the case of an incremental update, which consists of records like "replace bytes 17 to 42 with these 92 new bytes", you can't even make sense of the update without the right starting state.

My quibble was with the word "next": if you are currently running v3, and are about to take v4, when you referred to being ready for the "next OTA update", did "next" mean v4 or v5?
 
Yes, the inactive slot will receive the update. But what it receives is a combination of the active slot and the OTA update. That's why its so important that you return the active slot to the state expected by the people who issued the update. cf. @pndwal's notes.

If you can download a file-set containing a full image of the updated ROM, then you can burn that to either slot. The only problem would be if the official update installer refuses to run unless it is satisfied with the "current" state.

In the case of an incremental update, which consists of records like "replace bytes 17 to 42 with these 92 new bytes", you can't even make sense of the update without the right starting state.

My quibble was with the word "next": if you are currently running v3, and are about to take v4, when you referred to being ready for the "next OTA update", did "next" mean v4 or v5?
Yes, by "next" I meant v5.

If you are taking the v4, then currently active is v3 and inactive is v2. You need to restore v3 before installing v4, because when you take v5 v3 will be in inactive with a modified boot and that will fail checks and not install. Unless I am messing something.
 

pndwal

Senior Member
Maybe I'm misunderstanding your frame of reference, but if "next OTA update" means a future update, after the one you are trying to accept now, then I think the state of the currently-active-soon-to-be-inactive slot will be ignored in that not yet released future OTA update.

If by "next OTA update" you meant the update you are currently trying to accept, then excuse the ring.
Good catch!... John says 'The reason to restore the images' is as a prerequisite for available ('next') OTA...
When an OTA is available, first go to (Magisk app → Uninstall → Restore Images)...
This will restore partitions modified by Magisk back to stock from backups made at install in order to pass pre-OTA block verifications. This step is required before doing any of the following steps written below!
https://github.com/topjohnwu/Magisk/blob/master/docs/ota.md#prerequisites

You taking the current update depends on if you "restore images" the last time you updated. Because the inactive slot is the one taking the update. Not the current active one.
Yes, by "next" I meant v5.

If you are taking the v4, then currently active is v3 and inactive is v2. You need to restore v3 before installing v4, because when you take v5 v3 will be in inactive with a modified boot and that will fail checks and not install. Unless I am messing something.
But you are confusing ('messing') things a bit... v3 (inactive with a modified boot) will NOT fail the checks and cause a v5 OTA not to install! - The whole inactive 'slot' can in fact be blank and the OTA will succeed...

The active slot is NOT updated to accommodate a future OTA... Rather, data from active slot is used to generate images to be written to inactive slot in combination with data (binary patches) from the newly available OTA as has been explained... So the active slot is updated to accommodate the current/available OTA...

Of course, Restore Images simply restores current (active) slot to pre-magisk* images using image(s) from /data/magisk_backup_xxxxxxx*, but this is essential for current OTA...

Later OTA's will likewise rely on whatever slot is active at the time, NOT on the inactive slot being updated... In fact, this can be empty / blank / completely erased or corrupt and OTA will still succeed!

Details:
A/B system updates use a background daemon called update_engine to prepare the system to boot into a new, updated version. This daemon can perform the following actions:
• Read from the current slot A/B partitions and write any data to the unused slot A/B partitions as instructed by the OTA package.
• Call the boot_control interface...
• Run a post-install program from the new partition...
https://source.android.com/devices/tech/ota/ab#update-engine

An incremental update is an OTA package that contains binary patches to data already on the device. Packages with incremental updates are typically smaller as they don't need to include unchanged files. In addition, as changed files are often very similar to their previous versions, the package only needs to include an encoding of the differences between the two files.
https://source.android.com/devices/tech/ota/tools#incremental-updates

As mentioned, A/B update engine builds images for inactive slot using original block-verified images from active slot.
During incremental or delta updates, the binary data from the current slot is used to generate the data in the new slot.
https://source.android.com/devices/tech/ota/ab/ab_implement#configuration

*Nb. This further caveat from John is worth noting:
you HAVE to make sure you haven't modified an(y) read-only partitons yourself (such as /system or /vendor) in any way. Even remounting the partition to rw will tamper block verification!!
https://github.com/topjohnwu/Magisk/blob/master/docs/ota.md#ota-upgrade-guides

🤠 PW
 
Last edited:
Good catch!... John says 'The reason to restore the images' is as a prerequisite for available ('next') OTA...

https://github.com/topjohnwu/Magisk/blob/master/docs/ota.md#prerequisites



But you are confusing ('messing') things a bit... v3 (inactive with a modified boot) will NOT fail the checks and cause a v5 OTA not to install! - The whole inactive 'slot' can in fact be blank and the OTA will succeed...

The active slot is NOT updated to accommodate a future OTA... Rather, data from active slot is used to generate images to be written to inactive slot in combination with data (binary patches) from the newly available OTA as has been explained... So the active slot is updated to accommodate the current/available OTA...

Of course, Restore Images simply restores current (active) slot to pre-magisk* images using image(s) from /data/magisk_backup_xxxxxxx*, but this is essential for current OTA...

Later OTA's will likewise rely on whatever slot is active at the time, NOT on the inactive slot being updated... In fact, this can be empty / blank / completely erased or corrupt and OTA will still succeed!

Details:

https://source.android.com/devices/tech/ota/ab#update-engine


https://source.android.com/devices/tech/ota/tools#incremental-updates

As mentioned, A/B update engine builds images for inactive slot using original block-verified images from active slot.

https://source.android.com/devices/tech/ota/ab/ab_implement#configuration

*Nb. This further caveat from John is worth noting:

https://github.com/topjohnwu/Magisk/blob/master/docs/ota.md#ota-upgrade-guides

🤠 PW
So I was missing things. Thanks!
 
  • Like
Reactions: ipdev and pndwal
Hey there.
I have a Samsung S9 under Alexis Rom (based on OneUI 2.5).
When trying to update to Magisk 25.1, the direct install method leads to an error "unsupported boot image extra".
So I tried the patch images method. However, there was no “Patch vbmeta in boot image” option as described. So I let Magisk patch the original boot.img and flashed it.
It ended up to boot on a locked screen saying that the device is encrypted, which is not true. Moreover, the /data partition is readable in TWRP.

What have I done wrong?

EDIT: solved the issue with a backup of /data, then formatting it, then restoring the backup.
However, flashing again the 25.1 patched boot image resulted in bootloop: back to 24.3.
 
Last edited:

rodken

Senior Member
Jan 11, 2010
2,500
1,175
I'm stuck in a bootloop due to a module I installed (foxyboot?). I can boot normally with the stock boot.img and another boot image that gives me ADB root access.

I tried
Code:
adb wait-for-device shell magisk --remove-modules
but the device never established an ADB connection. So then I used the boot image that gives me ADB root, and I deleted /data/ADB/modules and modules_update. That didn't help. I tried data/ADB/magisk/ ./magisk32 --remove-modules, but that just caused the device to reboot, and didn't change the bootloop.

I can't install zips via TWRP currently.

Is it possible for me to just remove all the ADB files on the device eMMC and reinstall magisk manager? Where else is magisk storing files other than /data/ADB ?
For future reference:
-- Always take a working version of the latest Magisk and patch your boot.img
-- This ensures that you can flash and resort back in the event of a catastrophic event
-- Consider investing in installation of the Bootloop Saver.
-- Look into what the 'Magisk Manager for Recovery Mode' can do for you [if you have a working version of TWRP installed].

 

sharpknight

Senior Member
Mar 28, 2012
76
27
I just installed Magisk v25.1 on my LG V20 H990DS with stock oreo, and success without having kernel issues. But i can't opened the app itself (stuck on logo), but when downgrading the app to v23.0. It can be opened.
I am experiencing the same issue on my LG G7 ThinQ. I am able to install the Magisk v25.1 app but after the app install, the Magisk v25.1 app does not open. I just see the Magisk mask/face logo. I upgraded from v24.3. I'm able to downgrade back to Magisk v24.3 without problems.
 

rodken

Senior Member
Jan 11, 2010
2,500
1,175
I am experiencing the same issue on my LG G7 ThinQ. I am able to install the Magisk v25.1 app but after the app install, the Magisk v25.1 app does not open. I just see the Magisk mask/face logo. I upgraded from v24.3. I'm able to downgrade back to Magisk v24.3 without problems.
If Magisk is stuck on the splash screen:
-- Uninstall Magisk app and repatch your boot image.
-- If that doesn't work - Reflash your boot image and then patch the new boot image.
 
  • Like
Reactions: 73sydney

Top Liked Posts

  • There are no posts matching your filters.
  • 5
    I'm now having a problem hiding the TJW Magisk 27002 app ...

    I finally was able to resurrect my old Pixel 5. It's running stock A11 and was running TJW Magisk 26004 with no problem.

    The Magisk manager was showing that the upgrade to 27002 was ready for installation, and so I did the following:

    (1) Unhide Magisk
    (2) Update Magisk (27002 was listed)
    (3) After Magisk manager restarted, I did Direct Update to 27002, including reboot
    (4) Magisk 27002 indeed came up properly after reboot.
    (5) Tried to perform Hide The Magisk App.

    I entered the arbitrary name for hiding, and I clicked "OK". But the hiding never took place.

    I rebooted again, to see if perhaps that's necessary, and I repeated the hiding attempt by adding the same arbitrary name, and I again clicked "OK". But the hiding still didn't take place.

    I searched my device, and there is no app nor shortcut with that arbitrary name. I also tried other, different arbitrary names, but after clicking "OK", the hiding still didn't take place for any name I chose.

    What am I missing?
    Known issue. A fix has been merged. You can use the Debug app, revert back to 27001 or wait until 27003 is released.

    EDIT: Ninja'd 🙃
    4
    Is the Magisk 27001 APK/zip still available anywhere? I'd like to downgrade, but I can only find 27000 on Github and 27002 via update. I don't want to go to Canary in case the mounting changes break anything, or Debug as I hear the extra logging etc. causes performance issues (correct me if I'm wrong).
    Only v27000 is Stable. (and Beta). v27001 and v27002 are Canary or Debug

    Set your Update Channel to Stable and you will see (just like you can see also on GitHub) that latest Stable is v27000, there was never Stable v27001 or v27002

    Generally, Stable is always Nx100

    So, you cannot downgrade to Stable v27001 but Canary v27001 should be just like Stable v27000 (same codebase). That's also the general policy, whenever Nx100 Stable is released, then immediately comes Canary Nx100 +1, on the same code-base

    Therefore, downgrade to Stable v27000 instead
    4
    Direct Install with recovery mode checked worked for me. You were right.

    After I flashed wrong CSC firmware my baseband and EMEIs disappeared. Now I have updated bootloader and I cant to flash original CSC firmware anymore. Any ideas how to restore baseband, EMEIs or it is too late?
    Yep, I told you several times that you need Recovery mode (selected) for your Ramdisk=No device; instructions for this are also clear in official Installation Instruction page too, but apparently you have not been trying to follow official instructions, so it's no real surprise you've messed up firmware too... Please take *extra* care when modding as there are simply so many variables and pitfalls! 😬

    Are you flashing patched AP (Application Processor binary) along with all the other package binaries (X4?) for your S10+ (also in Magisk Installation page, Samsung section)??

    CP is Cellular Processor (Nb. Oft repeated references to CP as 'core processor' may be misleading) binary package (also referred to as Baseband Modem) so that should have fixed mismatched baseband etc... correct CSC (Consumer Software Customization; region, APN and carrier specific software packages/settings) and even BL (Bootloader) are most likely also important...

    Nb. BL doesn't always update bootloader version, but if later firmware has been flashed be aware that these can also cause anti-roll back version for bootloader to be incremented (in the event that an update patches serious vulns) so you original firmware may be blocked and you may need to use later packages only...

    If flashing latest complete firmware package via Odin doesn't fix your IMEI / baseband issues it may be that you've messed with partitions that contain unique device values in the past... You may need to ask in Galaxy, S10 or S10+ XDA forums for that. 🤠 PW
    4
    I'm now having a problem hiding the TJW Magisk 27002 app ...

    I finally was able to resurrect my old Pixel 5. It's running stock A11 and was running TJW Magisk 26004 with no problem.

    The Magisk manager was showing that the upgrade to 27002 was ready for installation, and so I did the following:

    (1) Unhide Magisk
    (2) Update Magisk (27002 was listed)
    (3) After Magisk manager restarted, I did Direct Update to 27002, including reboot
    (4) Magisk 27002 indeed came up properly after reboot.
    (5) Tried to perform Hide The Magisk App.

    I entered the arbitrary name for hiding, and I clicked "OK". But the hiding never took place.

    I rebooted again, to see if perhaps that's necessary, and I repeated the hiding attempt by adding the same arbitrary name, and I again clicked "OK". But the hiding still didn't take place.

    I searched my device, and there is no app nor shortcut with that arbitrary name. I also tried other, different arbitrary names, but after clicking "OK", the hiding still didn't take place for any name I chose.

    What am I missing?
    Known problem with 27002, the debug version doesn't have the problem with hiding, read back to see the discussion about it
    3
    I actually updated the app from the app itself before doing my OTA upgrade so it get the version 27 of magisk, I tried installing that one again I had in my downloaded folder. But I also downloaded again the app from github.

    I do have root, I've modified the boot file with magisk, so I'm not sure what to do now. I'll try installing an older version. But as you see of the screenshot from app manager, I don't know how the app is installed and removed almost immediately. At first I thought it was caused by an android 14 new security change.
    Run magisk -c from adb shell and see what version it reports.
    Then uninstall every copy of magisk app,
    You have more than one (including possibly a hidden one)
    Then reboot and install magisk app matching the version reported by magisk -c command.
  • 1103
    This is the place for general support and discussion regarding "Public Releases", which includes both stable and beta releases.
    All information, including troubleshoot guides and notes, are in the Announcement Thread
    156
    Hello, I haven't given much support on XDA lately. It can be resulted from
    • University started and I have limited free time. In fact, I mostly develop during midnight
    • I live in Taiwan, which has large time zone differences between my European/American contributors/testers, which usually forces me to stay up late at night to discuss/test stuffs.
    • The new version is about to come, I don't want to spend effort on supporting old releases
    The planned update is delayed again and again, to some point I think I'll shed some light about what has been happening lately, also along with some announcements.

    New Forum!
    As you might have already discovered, Magisk got its own subforum on XDA! Many thanks to all the support you gave me, and much more information/features/support is about to come!
    **For developers supporting all the devices that are not using standard Android boot format, feel free to create threads in this section (actually, PLEASE do so) for your favorite devices after v7 is out. As I currently know, Asus devices require signing the boot image before flashing, and is model dependant; Sony devices seems to use ELF kernel that is unpatchable, or some has two ramdisks (inner + outer), both requires different workarounds; LG bootloader locked devices has to manually "BUMP" the boot image after flashing Magisk..... and there may be lots of other crazy boot image formats that haven't come up to my attention yet.
    It is impossible for me to support all these non-standard boot images, and I hope the community can collaborate to make Magisk running across all the devices. Overall, community collaboration is what XDA about :D

    The Pixel Phone
    Some of you might already know this news, that the next Pixel Phone right around the corner seems like it does not have ramdisk in boot image, which pretty much wrecked Magisk in all ways. However, it pretty much doomed root itself too. Kernel modifications is inevitable IMO, so I'll try to migrate my scripts to C programs that could possibly be included into the kernel itself. Note that I'm not familiar with linux kernel, I'm not even sure if my idea and concept is correct or not. But once the device is available, I think developers will find a way to bypass all the difficulties, and I'll do my best to learn things ;)

    Current Progress
    In the past month, I've spent quite some time learning SELinux, so that I can avoid using SuperSU's sepolicy patches. Thanks to the helps and tips from @phhusson and @Chainfire, I finally have a much clearer understanding of how SELinux works. The Magisk core parts (the scripts, boot image patches, new features, more supports) are actually done some time ago. What is causing all the delays is the Magisk Manager.
    To be completely honest, although I can code in Java without much issues, Magisk Manager is actually my first Android application, I had to reach out for assistance, and fortunately awesome developers like @DVDandroid and @digitalhigh contributed a lot, which makes the current Manager awesome.
    After the repo system and module management is mostly done, I was about to do some adjustments and release, but what we really done is decided to add another feature: auto-unroot with per-app settings. I decided to wait for it to be finished, and then do my adjustments. Due to reasons that'll be mentioned later, this feature will likely not be available for the next release (should come in future updates)

    Safety Net Disaster
    Those who are using Magisk for Safety Net bypass purposes must have known that Google recently updated the detection method of my Systemless Xposed. I still have no idea what Safety Net is detecting, so currently I cannot fix it on my side (also because I'm busy working on the next update). However, suhide developed by @Chainfire is able to hide Xposed and worked fine.
    However, only my Systemless Xposed v86.2, which is based on SuperSU's su.d, is supported using that method. v86.2 and v86.5 (latest, Magisk based) have nearly identical binaries, and the only difference is the path where the binaries are stored.
    I'm still not sure what's the real issue for it not being supported, I just hope it is not done intentionally.

    Conclusion
    Due to the fact that my Safety Net bypass is not 100% perfect now, I do not want to spend any more time waiting for auto-unroot to be polished. What I'm doing now is finishing up all the things I'd like to change in Magisk Manager (it has been a while since I last contributed to Manager, my fellow developers are doing all the heavy job), which might take a little more time, after that, packed with tons of information to be announced in Magisk Section, I'll release the long awaited update.

    Hope this lengthy post gives you the idea of the whole situation, and again thanks for all your support!!
    121
    Ah, some Chainfire bashing, I hope it is not too late for me to exercise additional villainy.

    First, let me make clear I have nothing against @topjohnwu, nor against Magisk. Magisk is an interesting project and it certainly displays @topjohnwu ingenuity and persistence. I don't doubt we will see more interesting things from his hands.

    -------------------------

    What has happened here is not all that dark and complicated, from either end. I returned from holidays, and someone pointed me at Magisk. My first thought: interesting!

    Among other things, the thread lists some issues with SuperSU, which in combination with the phrase The developer also requests users to not bug Chainfire with compatibility requests for SuperSU with Magisk from the portal article, raised my left eyebrow by nigh half an inch. The popular systemless xposed mod is apparently now based on it, and apparently it now no longer works with SuperSU, and apparently I'm not supposed to fix that, nor any of the other found issues. I found that a bit weird. So yes, I have told @topjohnwu that I was a bit surprised he was posting about issues with SuperSU without notifying me about them (I can't fix or help fix issues I'm not aware of, after all).

    He's also spreading a modified version of the SuperSU package, which is not all that uncommon, nor necessarily a problem. I have not looked into what he modified, I only ran a few quick tests on one of my devices, and found some commonly used commands run as root to be broken. I have informed him of this as well.

    It appears the tool of choice for Magisk is phh's Superuser, because of some of the mentioned issues with SuperSU. That's fine by itself, but fixing issues in that superuser by incorporating SuperSU's binaries into it is a somewhat questionable practise. After all, SuperSU is a commercial closed-source package that helps pay for my dinner, and superuser is a direct competitor. I have informed him that I was surprised he did this without asking for permission. I have expressed similar surprise on him spreading a modified version of LiveBoot (which helps pay for a snack now and then).
    @topjohnwu has also stated that Magisk's scripts are largely influenced by mine (I have not checked). Scripts based on mine are used all over the place on XDA, some people have crafted amazing things based on them, I have never made an issue of this (otherwise I would have just made them binaries). But yes, I have also stated to him that I don't think it's very nice to base something on one program, and then using that to (almost exclusively) push something directly competing with that program.

    tl;dr Towards @topjohnwu, I have:
    - expressed surprise he has issues getting Magisk to work with SuperSU, and has chosen not to inform me about those
    - expressed surprise he is using SuperSU binaries in a competing superuser without permission
    - expressed surprise he is posting a modified LiveBoot without permission
    - informed him of issues with the modified SuperSU he has posted
    - let him know I thought it wasn't very nice to be applying my scripts to benefit seemingly exclusively that same competing superuser

    To be crystal clear:
    - I have not asked for an apology
    - I have not asked for Magisk to be abandoned, neither the root hiding nor systemless module parts, and certainly not systemless xposed
    - I have not made an issue of any of this anywhere, until this post
    - I have not even specifically asked for anything to be taken down (though obviously in my opinion the other superuser package mixed with SuperSU's binaries, as well as the LiveBoot package, should go)
    - I have not reported this thread to XDA moderators for copyright violations or otherwise

    While my conversation with @topjohnwu may not win any awards for being friendly (though it may win some for brevity), I think all things considered my response has been rather mild. To be perfectly honest, until the apology post, I thought this was over with already. I think the apology post was triggered because I haven't replied to his last PM for a while - I was in the zone, it happens.

    To emphasize again, I have nothing against @topjohnwu, Magisk, or systemless xposed, and it is certainly not my goal to see any of them go. If it can be made to work together with SuperSU, great.

    I get it though: you think of something, you want to see if you can make it work, you finally get it to work, you publish it, it takes off - enthusiasm gets the better of you. Maybe in the rush some mistakes are made. That doesn't mean you have to just drop it and run. None of my stuff would make it past 0.1 if I stopped at the first big mistake :)

    Aside from said being in the zone coding, I usually regret actually responding to these sort of things the day after, which has made me hesitant to reply. Surprise me.
    76
    Thread temporarily closed so everyone sees this.

    The flood of "SafetyNet isn't working for me either!" posts are not helpful, at all. Please refrain from posting further, it will be looked into. Please do not forget that not passing SafetyNet is 100% NORMAL AND INTENDED when you have an unlocked booloader or running custom firmware. These are workarounds and they will be worked around in turn.

    The Flash
    Forum Moderator

    EDIT: Thread is reopened... I will be cleaning any SafetyNet posts for a while to keep the thread clean for real issues.
    75
    Hello everyone!

    I am aware that Google has updated Safety Net that makes Magisk itself a no go for Android Pay. In fact, I witnessed the change live while I am developing the new magiskhide, which should hide all Magisk modules and Magisk installed root.

    Google is serious about Safety Net now, clearly hunting down all possibility to run Xposed with Safety Net passed. I spend quite some time examining the new security measures last midnight, and fortunately it seems that it is possible to run Magisk and root along with Safety Net if no Xposed is running. I'm glad I removed the old root toggle at the right time lol, that is no longer feasible with the latest detection.

    So stay tuned for the next update, it will come with bug fixes, along with the new magiskhide to bypass that Safety Net.

    Google, how will a few systemless mods do any harm :p:p