getsentry / getsentry/sentry-java

ANRWatchDog reports ANRs from another app when used in the system app

未关闭
#4,922 2 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
Android Bug Errors
主要语言
Kotlin
星标
1.4k
派生
478
平均合并
2 天 23 小时
30 天内合并 PR
67

描述

### Integration

sentry-android

### Build System

Gradle

### AGP Version

8.6.1

### Proguard

Enabled

### Version

8.11.1

### Steps to Reproduce

Other environment details: Android 6

1. Make the main app (when sentry installed) a system app. To achieve that, add `android:sharedUserId="android.uid.system"` to the android manifest
2. Trigger an ANR in another app - can be just an example app that simply uses Thread.sleep in the main thread
3. In the main app, we use Workmanager and some periodic workers, so the application eventually wakes up, then sleeps.

To simplify testing, I did something like in the main app:
```
viewModelScope.launch {
while (true) {
Timber.tag("ANR").i("isAnr=${ANRWatchDog.isProcessNotResponding}")
delay(5000)
}
}
```
Then, I opened another app which simply sleeps forever in onCreate.

This is happening because ANRWatchDog.isProcessNotResponding may return false positives for __system__ apps. That's happening because for system app `am.getProcessesInErrorState()` may return processes that do not belong to the current app because of wider permissions. I was able to validate this by checking `ActivityManager.ProcessErrorStateInfo.processName`.

### Expected Result

Main app doesn't report ANR from other apps

### Actual Result

Main app does report ANR from other apps

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。