diff options
| author | 2018-07-26 17:33:14 +0800 | |
|---|---|---|
| committer | 2018-07-31 09:26:15 +0000 | |
| commit | 473eb872f66bd51a2debc18616720f2432f1d121 (patch) | |
| tree | 0b01e2ef5a22fd26786d1b5ae1af72595867f846 | |
| parent | 716c04b29a0bd117aab33d8b81621a3fff0af7a7 (diff) | |
Strict-related commands porting
remove strict enable command
netd will handle it
Test: runtest frameworks-net passes
Test: run cts StrictModeTest
Change-Id: I34c4f270ec1d4d9ae1f60c0f9abec2a7d966ce9b
| -rw-r--r-- | services/core/java/com/android/server/NetworkManagementService.java | 22 |
1 files changed, 9 insertions, 13 deletions
diff --git a/services/core/java/com/android/server/NetworkManagementService.java b/services/core/java/com/android/server/NetworkManagementService.java index 6d6fd84dafaa..e4f26f858a89 100644 --- a/services/core/java/com/android/server/NetworkManagementService.java +++ b/services/core/java/com/android/server/NetworkManagementService.java @@ -641,12 +641,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub SystemProperties.set(PROP_QTAGUID_ENABLED, mBandwidthControlEnabled ? "1" : "0"); - try { - mConnector.execute("strict", "enable"); - mStrictEnabled = true; - } catch (NativeDaemonConnectorException e) { - Log.wtf(TAG, "Failed strict enable", e); - } + mStrictEnabled = true; setDataSaverModeEnabled(mDataSaverMode); @@ -1805,26 +1800,26 @@ public class NetworkManagementService extends INetworkManagementService.Stub } private void applyUidCleartextNetworkPolicy(int uid, int policy) { - final String policyString; + final int policyValue; switch (policy) { case StrictMode.NETWORK_POLICY_ACCEPT: - policyString = "accept"; + policyValue = INetd.PENALTY_POLICY_ACCEPT; break; case StrictMode.NETWORK_POLICY_LOG: - policyString = "log"; + policyValue = INetd.PENALTY_POLICY_LOG; break; case StrictMode.NETWORK_POLICY_REJECT: - policyString = "reject"; + policyValue = INetd.PENALTY_POLICY_REJECT; break; default: throw new IllegalArgumentException("Unknown policy " + policy); } try { - mConnector.execute("strict", "set_uid_cleartext_policy", uid, policyString); + mNetdService.strictUidCleartextPenalty(uid, policyValue); mUidCleartextPolicy.put(uid, policy); - } catch (NativeDaemonConnectorException e) { - throw e.rethrowAsParcelableException(); + } catch (RemoteException | ServiceSpecificException e) { + throw new IllegalStateException(e); } } @@ -1842,6 +1837,7 @@ public class NetworkManagementService extends INetworkManagementService.Stub return; } + // TODO: remove this code after removing prepareNativeDaemon() if (!mStrictEnabled) { // Module isn't enabled yet; stash the requested policy away to // apply later once the daemon is connected. |