diff options
| author | 2023-12-19 18:10:03 +0000 | |
|---|---|---|
| committer | 2023-12-19 18:10:03 +0000 | |
| commit | b3acd62209a67b9ff839c75c08ab76fd5a8d95e1 (patch) | |
| tree | 2bd1a2322cb6691ad09a02b7a2c0e931c3ee5c2c | |
| parent | 6adbcaffe414c6bcc3adbbc68edb989a36c7d06a (diff) | |
| parent | 65f46054e9477bfcffcbb6ba1fb6b983ea8bed76 (diff) | |
Merge "[BugFix] Fix NPE in AnomalyReporter.initialize" into main
| -rw-r--r-- | telephony/java/android/telephony/AnomalyReporter.java | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/telephony/java/android/telephony/AnomalyReporter.java b/telephony/java/android/telephony/AnomalyReporter.java index db38f8873a02..575ec27622a5 100644 --- a/telephony/java/android/telephony/AnomalyReporter.java +++ b/telephony/java/android/telephony/AnomalyReporter.java @@ -187,14 +187,15 @@ public final class AnomalyReporter { } for (ResolveInfo r : packages) { - if (r.activityInfo == null - || pm.checkPermission( + if (r.activityInfo == null) { + Rlog.w(TAG, "Found package without activity"); + continue; + } else if (pm.checkPermission( android.Manifest.permission.READ_PRIVILEGED_PHONE_STATE, r.activityInfo.packageName) - != PackageManager.PERMISSION_GRANTED) { - Rlog.w(TAG, - "Found package without proper permissions or no activity" - + r.activityInfo.packageName); + != PackageManager.PERMISSION_GRANTED) { + Rlog.w(TAG, "Found package without proper permissions" + + r.activityInfo.packageName); continue; } Rlog.d(TAG, "Found a valid package " + r.activityInfo.packageName); |