From edb833a1a59f4619e08aa8abc2500e9299155db5 Mon Sep 17 00:00:00 2001 From: Chiachang Wang Date: Mon, 26 Oct 2020 19:59:31 +0800 Subject: [PATCH] Replace hidden NMS permission control API Replace the hidden setNetworkPermission NMS APIs with accessing INetd directly for the incoming ConnectivityService mainline. Bug: 170598012 Test: atest FrameworksNetTests CtsNetTestCasesLatestSdk Change-Id: I37ed1003355677b98cbb741f774ba0fa4d193572 --- .../core/java/com/android/server/ConnectivityService.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/services/core/java/com/android/server/ConnectivityService.java b/services/core/java/com/android/server/ConnectivityService.java index 3291e96f4d..79e6218706 100644 --- a/services/core/java/com/android/server/ConnectivityService.java +++ b/services/core/java/com/android/server/ConnectivityService.java @@ -6264,9 +6264,9 @@ public class ConnectivityService extends IConnectivityManager.Stub final int newPermission = getNetworkPermission(newNc); if (oldPermission != newPermission && nai.created && !nai.isVPN()) { try { - mNMS.setNetworkPermission(nai.network.netId, newPermission); - } catch (RemoteException e) { - loge("Exception in setNetworkPermission: " + e); + mNetd.networkSetPermissionForNetwork(nai.network.netId, newPermission); + } catch (RemoteException | ServiceSpecificException e) { + loge("Exception in networkSetPermissionForNetwork: " + e); } } }