Merge "Check for REQUEST_NETWORK_SCORES instead of BROADCAST_NETWORK_PRIVILEGED."
am: b5b00857eb
Change-Id: I54a69f11284efb07ee6be58650d7a246540010dc
This commit is contained in:
@@ -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 =
|
||||
|
||||
@@ -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
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user