Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix transitioning between non-accept strict policies." into oc-mr1-dev

Browse Source
This commit is contained in:
Lorenzo Colitti
2017-08-23 09:08:46 +00:00
committed by Android (Google) Code Review
parent 9b601f2066 7a2b13ea6c
commit a68cdbe7b1
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
services/core/java/com/android/server/NetworkManagementService.java
View File

@@ -1838,13 +1838,15 @@ public class NetworkManagementService extends INetworkManagementService.Stub
// netd does not keep state on strict mode policies, and cannot replace a non-accept // netd does not keep state on strict mode policies, and cannot replace a non-accept
// policy without deleting it first. Rather than add state to netd, just always send // policy without deleting it first. Rather than add state to netd, just always send
// it an accept policy when switching between two non-accept policies. // it an accept policy when switching between two non-accept policies.
// TODO: consider keeping state in netd so we can simplify this code.
if (oldPolicy != StrictMode.NETWORK_POLICY_ACCEPT && if (oldPolicy != StrictMode.NETWORK_POLICY_ACCEPT &&
policy != StrictMode.NETWORK_POLICY_ACCEPT) { policy != StrictMode.NETWORK_POLICY_ACCEPT) {
applyUidCleartextNetworkPolicy(uid, StrictMode.NETWORK_POLICY_ACCEPT);
}
applyUidCleartextNetworkPolicy(uid, policy); applyUidCleartextNetworkPolicy(uid, policy);
} }
} }
applyUidCleartextNetworkPolicy(uid, policy);
}
@Override @Override
public boolean isBandwidthControlEnabled() { public boolean isBandwidthControlEnabled() {