snake-4 / Zygisk-Assistant

A Zygisk module to hide root for KernelSU, Magisk and APatch, designed to work on Android 5.0 and above.
MIT License
1.02k stars 72 forks source link

Hide ext4 flag errors=remount-ro #2

Closed snake-4 closed 5 months ago

snake-4 commented 6 months ago

Compared to stock kernel, Magisk adds errors=remount-ro to all userdata mounts. Should we remount them without the flag? It may be an unreliable detection vector but it's still an abnormality.

Magisk patched kernel: /dev/block/bootdevice/by-name/userdata on /data type ext4 (rw,seclabel,nosuid,nodev,noatime,nobarrier,noauto_da_alloc,errors=remount-ro,data=ordered)

Stock kernel: /dev/block/bootdevice/by-name/userdata on /data type ext4 (rw,seclabel,nosuid,nodev,noatime,nobarrier,noauto_da_alloc,data=ordered)

snake-4 commented 6 months ago

This only seems to happen when there's a bind mount module is present. Granted I've only tested with the built-in Systemless Hosts.

An alternative is to use this module instead of Systemless Hosts (It works on Magisk): https://github.com/symbuzzer/systemless-hosts-KernelSU-module/

privacyguy123 commented 6 months ago

I use bind mounts to avoid other detector apps - you wouldn't consider this change?

snake-4 commented 6 months ago

I use bind mounts to avoid other detector apps - you wouldn't consider this change?

Bind mounts are already hidden since 2.0.3 but this specific trace is not hidden. Though I guess most apps don't check this trace because it's kind of unreliable and may also happen on a stock device.

snake-4 commented 5 months ago

Implemented in f200aa4.