summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Yan Yan <evitayan@google.com> 2023-08-21 22:25:45 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2023-08-21 22:25:45 +0000
commit99fa55cbd34e5a33528f98bec20cacd66635006f (patch)
tree45b16b64c5e2537a7843c6d2ff1606443776499c
parenta2d740da81964480f384a4925fe7ddcf3782a498 (diff)
parente1b9dcc5f089d96ea217d4532b562e31521f7658 (diff)
Merge "Add a null pointer check for getSubscriptionsInGroup" into main
-rw-r--r--services/core/java/com/android/server/VcnManagementService.java8
1 files changed, 7 insertions, 1 deletions
diff --git a/services/core/java/com/android/server/VcnManagementService.java b/services/core/java/com/android/server/VcnManagementService.java
index c6e9a7d5e2e5..7acca19f9d79 100644
--- a/services/core/java/com/android/server/VcnManagementService.java
+++ b/services/core/java/com/android/server/VcnManagementService.java
@@ -456,7 +456,13 @@ public class VcnManagementService extends IVcnManagementService.Stub {
final List<SubscriptionInfo> subscriptionInfos = new ArrayList<>();
Binder.withCleanCallingIdentity(
() -> {
- subscriptionInfos.addAll(subMgr.getSubscriptionsInGroup(subscriptionGroup));
+ List<SubscriptionInfo> subsInGroup =
+ subMgr.getSubscriptionsInGroup(subscriptionGroup);
+ if (subsInGroup == null) {
+ logWtf("Received null from getSubscriptionsInGroup");
+ subsInGroup = Collections.emptyList();
+ }
+ subscriptionInfos.addAll(subsInGroup);
});
for (SubscriptionInfo info : subscriptionInfos) {