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

Merge "[DU03-4]Add NETWORK_STACK Permission check for NetworkStatsService API"

Browse Source
This commit is contained in:
Frank Li
2022-02-10 07:11:59 +00:00
committed by Android (Google) Code Review
parent fb96afd0b9 c3c367e481
commit a531035a8c
2 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
framework-t/src/android/app/usage/NetworkStatsManager.java
View File

@@ -699,7 +699,9 @@ public class NetworkStatsManager {
* @hide * @hide
*/ */
@SystemApi @SystemApi
@RequiresPermission(NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK) @RequiresPermission(anyOf = {
NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK,
android.Manifest.permission.NETWORK_STACK})
@NonNull public android.net.NetworkStats getMobileUidStats() { @NonNull public android.net.NetworkStats getMobileUidStats() {
try { try {
return mService.getUidStatsForTransport(TRANSPORT_CELLULAR); return mService.getUidStatsForTransport(TRANSPORT_CELLULAR);
@@ -723,7 +725,9 @@ public class NetworkStatsManager {
* @hide * @hide
*/ */
@SystemApi @SystemApi
@RequiresPermission(NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK) @RequiresPermission(anyOf = {
NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK,
android.Manifest.permission.NETWORK_STACK})
@NonNull public android.net.NetworkStats getWifiUidStats() { @NonNull public android.net.NetworkStats getWifiUidStats() {
try { try {
return mService.getUidStatsForTransport(TRANSPORT_WIFI); return mService.getUidStatsForTransport(TRANSPORT_WIFI);

2
service-t/src/com/android/server/net/NetworkStatsService.java
View File

@@ -1067,7 +1067,7 @@ public class NetworkStatsService extends INetworkStatsService.Stub {
@Override @Override
public NetworkStats getUidStatsForTransport(int transport) { public NetworkStats getUidStatsForTransport(int transport) {
enforceAnyPermissionOf(NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK); PermissionUtils.enforceNetworkStackPermission(mContext);
try { try {
final String[] relevantIfaces = final String[] relevantIfaces =
transport == TRANSPORT_WIFI ? mWifiIfaces : mMobileIfaces; transport == TRANSPORT_WIFI ? mWifiIfaces : mMobileIfaces;