summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Jack Yu <jackyu@google.com> 2023-12-19 18:10:03 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2023-12-19 18:10:03 +0000
commitb3acd62209a67b9ff839c75c08ab76fd5a8d95e1 (patch)
tree2bd1a2322cb6691ad09a02b7a2c0e931c3ee5c2c
parent6adbcaffe414c6bcc3adbbc68edb989a36c7d06a (diff)
parent65f46054e9477bfcffcbb6ba1fb6b983ea8bed76 (diff)
Merge "[BugFix] Fix NPE in AnomalyReporter.initialize" into main
-rw-r--r--telephony/java/android/telephony/AnomalyReporter.java13
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);