diff options
| author | 2024-06-05 12:41:37 -0700 | |
|---|---|---|
| committer | 2024-06-07 12:13:45 -0700 | |
| commit | 6ed23fc1c4e71bde803aaee698cfa4f736ab14aa (patch) | |
| tree | 478dea9981c969facaccab4728ffa1c7907feb2b | |
| parent | f8e96bdd308418a3cd622e45b49f4503c15a6467 (diff) | |
Fix NPE for null plans
Test: build, atest NetworkPolicyManagerServiceTest
Bug: 327985127
Change-Id: I5df9986d9159efb586f7e19def75c1beceb8f31f
| -rw-r--r-- | services/core/java/com/android/server/net/NetworkPolicyManagerService.java | 4 | ||||
| -rw-r--r-- | services/core/java/com/android/server/net/OWNERS | 1 |
2 files changed, 3 insertions, 2 deletions
diff --git a/services/core/java/com/android/server/net/NetworkPolicyManagerService.java b/services/core/java/com/android/server/net/NetworkPolicyManagerService.java index c60ac3a74ebd..f03c639c103c 100644 --- a/services/core/java/com/android/server/net/NetworkPolicyManagerService.java +++ b/services/core/java/com/android/server/net/NetworkPolicyManagerService.java @@ -3965,8 +3965,8 @@ public class NetworkPolicyManagerService extends INetworkPolicyManager.Stub { // allow override without having plans defined. synchronized (mNetworkPoliciesSecondLock) { final SubscriptionPlan plan = getPrimarySubscriptionPlanLocked(subId); - if (overrideMask != SUBSCRIPTION_OVERRIDE_UNMETERED && plan == null - || plan.getDataLimitBehavior() == SubscriptionPlan.LIMIT_BEHAVIOR_UNKNOWN) { + if (overrideMask != SUBSCRIPTION_OVERRIDE_UNMETERED && (plan == null + || plan.getDataLimitBehavior() == SubscriptionPlan.LIMIT_BEHAVIOR_UNKNOWN)) { throw new IllegalStateException( "Must provide valid SubscriptionPlan to enable overriding"); } diff --git a/services/core/java/com/android/server/net/OWNERS b/services/core/java/com/android/server/net/OWNERS index 669cdaaf3ab5..bbc7c013f170 100644 --- a/services/core/java/com/android/server/net/OWNERS +++ b/services/core/java/com/android/server/net/OWNERS @@ -1,5 +1,6 @@ set noparent file:platform/packages/modules/Connectivity:main:/OWNERS_core_networking +per-file NetworkPolicyManagerService.java=jackyu@google.com, sarahchin@google.com jsharkey@android.com sudheersai@google.com |