From 45544b10d855762854cc8c7c66cc07ed3d479f4c Mon Sep 17 00:00:00 2001 From: Chiachang Wang Date: Thu, 31 Jan 2019 09:06:13 +0800 Subject: [PATCH] WifiManagerTest: allow NetworkStack to grant network permissions NetworkStack is moved from system to its own UID but NetworkStack requires NETWORK_SETTINGS permission and play as a system level app. It should not be treated as a normal app. Bug: 123540252 Test: atest WifiManagerTest Change-Id: I794a603bb1b354819ae56ffb555bf26103528e1c --- tests/cts/net/src/android/net/wifi/cts/WifiManagerTest.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/cts/net/src/android/net/wifi/cts/WifiManagerTest.java b/tests/cts/net/src/android/net/wifi/cts/WifiManagerTest.java index 0337119dd1..4297a73cae 100644 --- a/tests/cts/net/src/android/net/wifi/cts/WifiManagerTest.java +++ b/tests/cts/net/src/android/net/wifi/cts/WifiManagerTest.java @@ -872,8 +872,8 @@ public class WifiManagerTest extends AndroidTestCase { * Verify that the {@link android.Manifest.permission#NETWORK_SETTINGS} permission is * never held by any package. *

- * Only Settings, SysUi and shell apps should ever attempt to acquire this - * permission, since it would give those apps extremely broad access to connectivity + * Only Settings, SysUi, NetworkStack and shell apps should ever attempt to acquire + * this permission, since it would give those apps extremely broad access to connectivity * functionality. The permission is intended to be granted to only those apps with direct user * access and no others. */ @@ -886,6 +886,7 @@ public class WifiManagerTest extends AndroidTestCase { allowedUIDs.add(Process.SYSTEM_UID); allowedUIDs.add(Process.SHELL_UID); allowedUIDs.add(Process.PHONE_UID); + allowedUIDs.add(Process.NETWORK_STACK_UID); // only quick settings is allowed to bind to the BIND_QUICK_SETTINGS_TILE permission, using // this fact to determined allowed package name for sysui