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

[Android 10/Q] diag_revealer not giving output on API29 / Q #34

Open
MaxwellDPS opened this issue Sep 6, 2020 · 2 comments
Open

[Android 10/Q] diag_revealer not giving output on API29 / Q #34

MaxwellDPS opened this issue Sep 6, 2020 · 2 comments

Comments

@MaxwellDPS
Copy link

Versions

  • OS: diag_revealer built on Ubuntu 18
  • Python version: 2.7 & 3.7-3.8
  • Python installation method: APT
  • Android SDK version: API 29 Android 10/Q
  • Android NDK version: 21.3.6528147
  • Device: Pixel 4 flame
  • Device OS: Stock Pixel /w Magisk
  • Build: QQ3A.200805.001

Description

The app appears to be broken on Android 10 - Q

Upon running the app no errors occur but no logs populate either.

I tried to compile just the diag_revealer and run, but I'm not getting any output from /dev/diag.
Could SELinux be causing issues?

It also looks like Android 10 has a different struct for diag_logging_mode_param_t than android 9 did.

Code and Logs

Android 10

struct diag_logging_mode_param_t {
--
  | uint32_t req_mode;
  | uint32_t peripheral_mask;
  | uint32_t pd_mask;
  | uint8_t mode_param;
  | uint8_t diag_id;
  | uint8_t pd_val;
  | uint8_t reserved;
  | int peripheral;
  | int device_mask;
  | } __packed;

Android 9

struct diag_logging_mode_param_t_pie {
    uint32_t req_mode;
	uint32_t peripheral_mask;
	uint32_t pd_mask;
	uint8_t mode_param;
	uint8_t diag_id;
	uint8_t pd_val;
	uint8_t reserved;
	int peripheral;
} __packed;
@zwyuan
Copy link
Member

zwyuan commented Sep 19, 2020

Thank you for pointing out this issue! We are looking into this and see what we can do. Diag behaves differently across major Android versions and also different vendors, which indeed caused most problems.

@MaxwellDPS
Copy link
Author

@zwyuan No problem, It also has not been working on Android 11. It seems like it may be an issue with ioctl, and possibly an SELinux.

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

2 participants