LineageOS/android_packages_modules_Connectivity
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge "Retain subscription IDs on restricted test networks"

Browse Source
This commit is contained in:
Yan Yan
2023-03-31 03:44:15 +00:00
committed by Gerrit Code Review
parent de078f0ba0 966fb798a1
commit 51db40f5e5
2 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
framework/src/android/net/NetworkCapabilities.java
View File

@@ -1134,14 +1134,16 @@ public final class NetworkCapabilities implements Parcelable {
mTransportTypes = mTransportTypes =
(originalTransportTypes & UNRESTRICTED_TEST_NETWORKS_ALLOWED_TRANSPORTS) (originalTransportTypes & UNRESTRICTED_TEST_NETWORKS_ALLOWED_TRANSPORTS)
| (1 << TRANSPORT_TEST); | (1 << TRANSPORT_TEST);
// SubIds are only allowed for Test Networks that only declare TRANSPORT_TEST.
setSubscriptionIds(originalSubIds);
} else { } else {
// If the test network is restricted, then it may declare any transport. // If the test network is restricted, then it may declare any transport.
mTransportTypes = (originalTransportTypes | (1 << TRANSPORT_TEST)); mTransportTypes = (originalTransportTypes | (1 << TRANSPORT_TEST));
} }
if (hasSingleTransport(TRANSPORT_TEST)) {
// SubIds are only allowed for Test Networks that only declare TRANSPORT_TEST.
setSubscriptionIds(originalSubIds);
}
mNetworkCapabilities = originalCapabilities & TEST_NETWORKS_ALLOWED_CAPABILITIES; mNetworkCapabilities = originalCapabilities & TEST_NETWORKS_ALLOWED_CAPABILITIES;
if (!hasTransport(TRANSPORT_CELLULAR)) { if (!hasTransport(TRANSPORT_CELLULAR)) {
mNetworkCapabilities |= mNetworkCapabilities |=

3
tests/common/java/android/net/NetworkCapabilitiesTest.java
View File

@@ -1387,6 +1387,9 @@ public class NetworkCapabilitiesTest {
// If the test network is restricted, then the network may declare any transport, and // If the test network is restricted, then the network may declare any transport, and
// appended with TRANSPORT_TEST. // appended with TRANSPORT_TEST.
expectedNcBuilder.addTransportType(TRANSPORT_CELLULAR); expectedNcBuilder.addTransportType(TRANSPORT_CELLULAR);
} else {
// If the test network only has TRANSPORT_TEST, then it can keep the subscription IDs.
expectedNcBuilder.setSubscriptionIds(Set.of(TEST_SUBID1));
} }
expectedNcBuilder.addTransportType(TRANSPORT_TEST); expectedNcBuilder.addTransportType(TRANSPORT_TEST);