joy/android_packages_modules_Connectivity
Watch
1
Fork
You've already forked android_packages_modules_Connectivity
 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge "Fix transport override" into main

Browse source
This commit is contained in:
Patrick Rohr 2025-03-26 19:14:29 -07:00 • committed by Gerrit Code Review
commit c20c1224bc
2 changed files with 21 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
service-t/src/com/android/server/ethernet/EthernetTracker.java
View file

@ -933,7 +933,11 @@ public class EthernetTracker {
if (isAtLeastB && capabilitiesString.equals("*")) {
// On Android B+, a "*" string defaults to the same set of default
// capabilities assigned to unconfigured interfaces.
return new NetworkCapabilities(DEFAULT_CAPABILITIES);
// Note that the transport type is populated later with the result of
// parseTransportType().
return new NetworkCapabilities.Builder(DEFAULT_CAPABILITIES)
.removeTransportType(NetworkCapabilities.TRANSPORT_ETHERNET)
.build();
}
for (String strNetworkCapability : capabilitiesString.split(",")) {

17
tests/unit/java/com/android/server/ethernet/EthernetTrackerTest.java
View file

@ -18,6 +18,7 @@ package com.android.server.ethernet;
import static android.net.TestNetworkManager.TEST_TAP_PREFIX;
import static com.android.server.ethernet.EthernetTracker.DEFAULT_CAPABILITIES;
import static com.google.common.truth.Truth.assertThat;
import static org.junit.Assert.assertFalse;
@ -185,7 +186,7 @@ public class EthernetTrackerTest {
// On Android B+, "*" defaults to using DEFAULT_CAPABILITIES.
p = new EthernetConfigParser("eth0;*;;;;;;", true /*isAtLeastB*/);
assertThat(p.mCaps).isEqualTo(EthernetTracker.DEFAULT_CAPABILITIES);
assertThat(p.mCaps).isEqualTo(DEFAULT_CAPABILITIES);
// But not so before B.
p = new EthernetConfigParser("eth0;*", false /*isAtLeastB*/);
@ -205,10 +206,24 @@ public class EthernetTrackerTest {
// TRANSPORT_VPN (4) is not allowed.
p = new EthernetConfigParser("eth0;;;4", false /*isAtLeastB*/);
assertThat(p.mCaps.hasSingleTransport(NetworkCapabilities.TRANSPORT_ETHERNET)).isTrue();
p = new EthernetConfigParser("eth0;*;;4", true /*isAtLeastB*/);
assertThat(p.mCaps.hasSingleTransport(NetworkCapabilities.TRANSPORT_ETHERNET)).isTrue();
// invalid capability and transport type
p = new EthernetConfigParser("eth0;-1,a,1000,,;;-1", false /*isAtLeastB*/);
assertThat(p.mCaps).isEqualTo(baseNc);
p = new EthernetConfigParser("eth0;*;;0", false /*isAtLeastB*/);
assertThat(p.mCaps.hasSingleTransport(NetworkCapabilities.TRANSPORT_CELLULAR)).isTrue();
p = new EthernetConfigParser("eth0;*;;0", true /*isAtLeastB*/);
assertThat(p.mCaps.hasSingleTransport(NetworkCapabilities.TRANSPORT_CELLULAR)).isTrue();
NetworkCapabilities nc = new NetworkCapabilities.Builder(DEFAULT_CAPABILITIES)
.removeTransportType(NetworkCapabilities.TRANSPORT_ETHERNET)
.addTransportType(NetworkCapabilities.TRANSPORT_CELLULAR)
.build();
p = new EthernetConfigParser("eth0;*;;0", true /*isAtLeastB*/);
assertThat(p.mCaps).isEqualTo(nc);
}
@Test