Network-related commands porting

Test: runtest frameworks-net passes
Test: manual testing of network related function works
Change-Id: Id197d20c80b04378731b113ad74676672bd8b67e
This commit is contained in:
Luke Huang
2018-08-24 20:33:16 +08:00
parent 90a0bdb98a
commit b913c8165e

View File

@@ -4789,15 +4789,14 @@ public class ConnectivityService extends IConnectivityManager.Stub
} }
} }
private String getNetworkPermission(NetworkCapabilities nc) { private int getNetworkPermission(NetworkCapabilities nc) {
// TODO: make these permission strings AIDL constants instead.
if (!nc.hasCapability(NET_CAPABILITY_NOT_RESTRICTED)) { if (!nc.hasCapability(NET_CAPABILITY_NOT_RESTRICTED)) {
return NetworkManagementService.PERMISSION_SYSTEM; return INetd.PERMISSION_SYSTEM;
} }
if (!nc.hasCapability(NET_CAPABILITY_FOREGROUND)) { if (!nc.hasCapability(NET_CAPABILITY_FOREGROUND)) {
return NetworkManagementService.PERMISSION_NETWORK; return INetd.PERMISSION_NETWORK;
} }
return null; return INetd.PERMISSION_NONE;
} }
/** /**
@@ -4870,9 +4869,9 @@ public class ConnectivityService extends IConnectivityManager.Stub
if (Objects.equals(nai.networkCapabilities, newNc)) return; if (Objects.equals(nai.networkCapabilities, newNc)) return;
final String oldPermission = getNetworkPermission(nai.networkCapabilities); final int oldPermission = getNetworkPermission(nai.networkCapabilities);
final String newPermission = getNetworkPermission(newNc); final int newPermission = getNetworkPermission(newNc);
if (!Objects.equals(oldPermission, newPermission) && nai.created && !nai.isVPN()) { if (oldPermission != newPermission && nai.created && !nai.isVPN()) {
try { try {
mNMS.setNetworkPermission(nai.network.netId, newPermission); mNMS.setNetworkPermission(nai.network.netId, newPermission);
} catch (RemoteException e) { } catch (RemoteException e) {