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

Commit b5b00857 authored by Treehugger Robot's avatar Treehugger Robot Committed by Gerrit Code Review
Browse files

Merge "Check for REQUEST_NETWORK_SCORES instead of BROADCAST_NETWORK_PRIVILEGED."

parents 00514f1d c4982553

services/core/java/com/android/server/NetworkScoreService.java

+1 −1
Original line number Diff line number Diff line
@@ -604,7 +604,7 @@ public class NetworkScoreService extends INetworkScoreService.Stub { 
    @Override

    public void requestRecommendationAsync(RecommendationRequest request,

            RemoteCallback remoteCallback) {

        mContext.enforceCallingOrSelfPermission(permission.BROADCAST_NETWORK_PRIVILEGED, TAG);

        mContext.enforceCallingOrSelfPermission(permission.REQUEST_NETWORK_SCORES, TAG);



        final OneTimeCallback oneTimeCallback = new OneTimeCallback(remoteCallback);

        final Pair<RecommendationRequest, OneTimeCallback> pair =

services/tests/servicestests/src/com/android/server/NetworkScoreServiceTest.java

+2 −2
Original line number Diff line number Diff line
@@ -290,12 +290,12 @@ public class NetworkScoreServiceTest { 
    @Test

    public void testRequestRecommendationAsync_noPermission() throws Exception {

        doThrow(new SecurityException()).when(mContext)

                .enforceCallingOrSelfPermission(eq(permission.BROADCAST_NETWORK_PRIVILEGED),

                .enforceCallingOrSelfPermission(eq(permission.REQUEST_NETWORK_SCORES),

                        anyString());

        try {

            mNetworkScoreService.requestRecommendationAsync(mRecommendationRequest,

                    mRemoteCallback);

            fail("BROADCAST_NETWORK_PRIVILEGED not enforced.");

            fail("REQUEST_NETWORK_SCORES not enforced.");

        } catch (SecurityException e) {

            // expected

        }