Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Analysing /data with root enabled does not work anymore #87

Open
Massedil opened this issue Dec 7, 2023 · 8 comments
Open

Analysing /data with root enabled does not work anymore #87

Massedil opened this issue Dec 7, 2023 · 8 comments

Comments

@Massedil
Copy link

Massedil commented Dec 7, 2023

I tried the 5.0-alpha0 on F-Droid and I can't see my files when I try to show /data partition.

It was working with the 3.8.0 version.

Phone: Samsung Galaxy S5
LineageOS : 18.1

@Massedil
Copy link
Author

Massedil commented Dec 7, 2023

Screenshot_20231207-134018_Trebuchet
Screenshot_20231207-134030_Trebuchet

@IvanVolosyuk
Copy link
Owner

IvanVolosyuk commented Dec 8, 2023 via email

@Massedil
Copy link
Author

Massedil commented Dec 8, 2023

Does it work on an older version of diskusage on the same device?

Yes, it works with the 3.8.0 F-Droid version.

@Massedil
Copy link
Author

Massedil commented Dec 8, 2023

The only bug with 3.8.0 is that 2 same folders (but mounted twice with different names) are counted twice instead of one.

/data/data and /data/user/0 are the same folder.

Here is 3.8.0 on the same device :

Screenshot_20231208-120219_Trebuchet

@tuxayo
Copy link

tuxayo commented Dec 8, 2023

Same issue as this ticket and same issue with the old version:

The only bug with 3.8.0 is that 2 same folders (but mounted twice with different names) are counted twice instead of one.

phone: Fairphone 3
Android 13 (LineageOS for microG 20)
encrypted

I tried various other entries like "/data/user/0" and it doesn't data either.

@binarynoise
Copy link

The issue with limited access to /data can be worked around by calling magisk su with

  -mm, -M,
  --mount-master                Force run in the global mount namespace

And the app should check for and not follow symlinks (like /data/user/0 -> /data/data)

@IvanVolosyuk
Copy link
Owner

IvanVolosyuk commented Dec 13, 2023 via email

@binarynoise
Copy link

It seems, the problem is not only the restricted access to /data.
When running the native lib manually, it segfaults immediately, not even printing the "Need directory argument" error.
As I don't know how to debug that, I have to pass the issue to someone who does.
I could however push my change to use --mount-master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants