summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author yangjing24 <yangjing24@xiaomi.com> 2023-12-18 17:38:31 +0800
committer yangéť™ <yangjing24@xiaomi.com> 2023-12-19 01:54:29 +0000
commit65f46054e9477bfcffcbb6ba1fb6b983ea8bed76 (patch)
tree6ea0803e69bb0720e4305051ada32a7a4083e8ef
parent0ae7fbc36c726b1d6be6ace1917c14f472ba13d4 (diff)
[BugFix] Fix NPE in AnomalyReporter.initialize
Bug: 316850504 Change-Id: Ie6655bb6634f1a20e9e5c320fd137fd73dab5f16 Signed-off-by: yangjing24 <yangjing24@xiaomi.com>
-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);