diff options
author | Roshan Pius <rpius@google.com> | 2020-04-22 16:07:13 -0700 |
---|---|---|
committer | Roshan Pius <rpius@google.com> | 2020-04-22 16:11:31 -0700 |
commit | d3233e508ebb6c1919ec1dd1559e2516873f1611 (patch) | |
tree | d08b98a03614d48e67b1478b0e1ee22f00bb7d72 /service | |
parent | b774c31798b0abe4910cd8658304fe12ae0b3160 (diff) |
WifiServiceImpl: Restore the system alert bypass for legacy API surface
This was prematurely removed in R even though we later changed that
decision.
Bug: 153615530
Test: atest com.android.server.wifi
Change-Id: I8373e0f287102dd3d29a1b7c2c7d4b75a8c36433
Diffstat (limited to 'service')
-rw-r--r-- | service/java/com/android/server/wifi/WifiServiceImpl.java | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/service/java/com/android/server/wifi/WifiServiceImpl.java b/service/java/com/android/server/wifi/WifiServiceImpl.java index 6e339044a..a147c247e 100644 --- a/service/java/com/android/server/wifi/WifiServiceImpl.java +++ b/service/java/com/android/server/wifi/WifiServiceImpl.java @@ -735,10 +735,10 @@ public class WifiServiceImpl extends BaseWifiService { private boolean isTargetSdkLessThanQOrPrivileged(String packageName, int pid, int uid) { return mWifiPermissionsUtil.isTargetSdkLessThan(packageName, Build.VERSION_CODES.Q, uid) || isPrivileged(pid, uid) - // DO/PO apps should be able to add/modify saved networks. || isDeviceOrProfileOwner(uid, packageName) - // TODO: Remove this system app bypass once Q is released. - || isSystem(packageName, uid); + || isSystem(packageName, uid) + // TODO(b/140540984): Remove this bypass. + || mWifiPermissionsUtil.checkSystemAlertWindowPermission(uid, packageName); } /** @@ -749,9 +749,7 @@ public class WifiServiceImpl extends BaseWifiService { private boolean isTargetSdkLessThanROrPrivileged(String packageName, int pid, int uid) { return mWifiPermissionsUtil.isTargetSdkLessThan(packageName, Build.VERSION_CODES.R, uid) || isPrivileged(pid, uid) - // DO/PO apps should be able to add/modify saved networks. || isDeviceOrProfileOwner(uid, packageName) - // TODO: Remove this system app bypass once R is released. || isSystem(packageName, uid); } |