summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Jean Chalard <jchalard@google.com> 2023-04-14 04:14:46 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2023-04-14 04:14:46 +0000
commit6ae4584bd166baaff05e31b24871bcfe2f615dec (patch)
treeee947abb2ef9f6074568c07340f14a3a802dc42b
parentf9fd820f69ba00b62e22ac55090e7c33a2b4a0c6 (diff)
parent38904b1531552ad1125313949101d1c9cea95160 (diff)
Merge "Always startOrMigrate connection when caps change" am: 5288559979 am: 38904b1531
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2514996 Change-Id: I8ff523996249135b3df4c991d9d2bff7de8551b1 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r--services/core/java/com/android/server/connectivity/Vpn.java8
1 files changed, 3 insertions, 5 deletions
diff --git a/services/core/java/com/android/server/connectivity/Vpn.java b/services/core/java/com/android/server/connectivity/Vpn.java
index 0f17139e2678..1f4c7e6fa4e5 100644
--- a/services/core/java/com/android/server/connectivity/Vpn.java
+++ b/services/core/java/com/android/server/connectivity/Vpn.java
@@ -3744,12 +3744,10 @@ public class Vpn {
+ mUnderlyingNetworkCapabilities + " to " + nc);
final NetworkCapabilities oldNc = mUnderlyingNetworkCapabilities;
mUnderlyingNetworkCapabilities = nc;
- if (oldNc == null) {
- // A new default network is available.
+ if (oldNc == null || !nc.getSubscriptionIds().equals(oldNc.getSubscriptionIds())) {
+ // A new default network is available, or the subscription has changed.
+ // Try to migrate the session, or failing that, start a new one.
startOrMigrateIkeSession(mActiveNetwork);
- } else if (!nc.getSubscriptionIds().equals(oldNc.getSubscriptionIds())) {
- // Renew carrierConfig values.
- maybeMigrateIkeSessionAndUpdateVpnTransportInfo(mActiveNetwork);
}
}