diff options
| author | 2023-08-21 22:50:45 +0000 | |
|---|---|---|
| committer | 2023-08-21 22:50:45 +0000 | |
| commit | 12d2dfbddb649b24b74f4a91e1acb47c6e3a7b17 (patch) | |
| tree | 1e5dc48f31f97d0cbce51c271acfee78f1334bcd | |
| parent | 294b2bfadb2769949c168fea7cc27455397e7a18 (diff) | |
| parent | 99fa55cbd34e5a33528f98bec20cacd66635006f (diff) | |
Merge "Add a null pointer check for getSubscriptionsInGroup" into main am: 99fa55cbd3
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2712254
Change-Id: I2e4a4cef7018e8b21aef2c0ef32f6375589fe392
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | services/core/java/com/android/server/VcnManagementService.java | 8 |
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) { |