LineageOS/android_packages_modules_Connectivity
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't use framework permission strings for netd permissions.

Browse Source 
These framework permission strings were being used as arbitrary labels
that mapped to netd permissions that have completely different meaning.
This leads to confusion, so use different strings.

Bug: 18194858
Change-Id: Ib3ec377ab26ce904d3d4678f04edec6cb1260517
This commit is contained in:
Sreeram Ramachandran
2014-11-04 10:15:03 -08:00
parent 1b11eb6480
commit d9bf64ba1d
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
services/core/java/com/android/server/connectivity/PermissionMonitor.java
View File

@@ -191,8 +191,8 @@ public class PermissionMonitor {
}
try {
if (add) {
mNetd.setPermission(CHANGE_NETWORK_STATE, toIntArray(network));
mNetd.setPermission(CONNECTIVITY_INTERNAL, toIntArray(system));
mNetd.setPermission("NETWORK", toIntArray(network));
mNetd.setPermission("SYSTEM", toIntArray(system));
} else {
mNetd.clearPermission(toIntArray(network));
mNetd.clearPermission(toIntArray(system));