getsentry / getsentry/sentry-java

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

オープン
#4,922 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
Android Bug Errors
主要言語
Kotlin
スター
1.4k
フォーク
478
平均マージ
2日 23時間
マージ済み PR(30日)
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 を短くまとめたダイジェスト。