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

Commit 6d27f88e authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Clear calling identity when notifying alert/limit reached" am: a4df5870 

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1896540

Change-Id: If2b8b6e8b8856d6db2aa167484c71163e06828c2
parents 5dc7e50d a4df5870

services/core/java/com/android/server/net/NetworkStatsService.java

+8 −3
Original line number Diff line number Diff line
@@ -150,6 +150,7 @@ import com.android.internal.util.ArrayUtils; 
import com.android.internal.util.DumpUtils;

import com.android.internal.util.FileRotator;

import com.android.internal.util.IndentingPrintWriter;

import com.android.net.module.util.BinderUtils;

import com.android.server.EventLogTags;

import com.android.server.LocalServices;
@@ -2097,14 +2098,18 @@ public class NetworkStatsService extends INetworkStatsService.Stub { 


        @Override

        public void notifyAlertReached() throws RemoteException {

            mAlertObserver.limitReached(LIMIT_GLOBAL_ALERT, null /* unused */);

            // This binder object can only have been obtained by a process that holds

            // NETWORK_STATS_PROVIDER. Thus, no additional permission check is required.

            BinderUtils.withCleanCallingIdentity(() ->

                    mAlertObserver.limitReached(LIMIT_GLOBAL_ALERT, null /* unused */));

        }



        @Override

        public void notifyWarningOrLimitReached() {

            Log.d(TAG, mTag + ": notifyWarningOrLimitReached");

            BinderUtils.withCleanCallingIdentity(() ->

                    LocalServices.getService(NetworkPolicyManagerInternal.class)

                    .onStatsProviderWarningOrLimitReached(mTag);

                            .onStatsProviderWarningOrLimitReached(mTag));

        }



        @Override