Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2da74e08 authored by Frank Li's avatar Frank Li Committed by Gerrit Code Review
Browse files

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

parents 1d30d5d9 767c2e79

packages/ConnectivityT/framework-t/src/android/app/usage/NetworkStatsManager.java

+6 −2
Original line number Diff line number Diff line
@@ -699,7 +699,9 @@ public class NetworkStatsManager { 
     * @hide

     */

    @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() {

        try {

            return mService.getUidStatsForTransport(TRANSPORT_CELLULAR);
@@ -723,7 +725,9 @@ public class NetworkStatsManager { 
     * @hide

     */

    @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() {

        try {

            return mService.getUidStatsForTransport(TRANSPORT_WIFI);

packages/ConnectivityT/service/src/com/android/server/net/NetworkStatsService.java

+1 −1
Original line number Diff line number Diff line
@@ -1123,7 +1123,7 @@ public class NetworkStatsService extends INetworkStatsService.Stub { 


    @Override

    public NetworkStats getUidStatsForTransport(int transport) {

        enforceAnyPermissionOf(NetworkStack.PERMISSION_MAINLINE_NETWORK_STACK);

        PermissionUtils.enforceNetworkStackPermission(mContext);

        try {

            final String[] relevantIfaces =

                    transport == TRANSPORT_WIFI ? mWifiIfaces : mMobileIfaces;