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

Update VpnTest#testDefault to use otherUid that has internet permission

Browse source 
Bug: 342557183
Test: VpnTest
Change-Id: If4a256df9ef96285c65dd285c7d37696dbd82298
This commit is contained in:
Motomu Utsumi 2024-06-07 14:50:25 +09:00
commit dda2d35b0c
1 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
tests/cts/hostside/app/src/com/android/cts/net/hostside/VpnTest.java
View file

@ -971,8 +971,12 @@ public class VpnTest {
final TestableNetworkCallback otherUidCallback = new TestableNetworkCallback();
final TestableNetworkCallback myUidCallback = new TestableNetworkCallback();
if (SdkLevel.isAtLeastS()) {
final int otherUid =
UserHandle.of(5 /* userId */).getUid(Process.FIRST_APPLICATION_UID);
// Using the same appId with the test to make sure otherUid has the internet permission.
// This works because the UID permission map only stores the app ID and not the whole
// UID. If the otherUid does not have the internet permission, network access from
// otherUid could be considered blocked on V+.
final int appId = UserHandle.getAppId(Process.myUid());
final int otherUid = UserHandle.of(5 /* userId */).getUid(appId);
final Handler h = new Handler(Looper.getMainLooper());
runWithShellPermissionIdentity(() -> {
registerSystemDefaultNetworkCallback(systemDefaultCallback, h);