diff options
author | Abhijith Shastry <ashastry@google.com> | 2016-03-17 16:05:14 -0700 |
---|---|---|
committer | Abhijith Shastry <ashastry@google.com> | 2016-03-17 17:09:38 -0700 |
commit | 0b78245b351a8bbd1ed34c14cb9a47c0f9dc5abf (patch) | |
tree | 7b0c049c0c6e2cc47c8d6c786ce6d47e3f8af6e3 | |
parent | aee4f90bfe7a774ff726b8c8c561b483d759f8aa (diff) | |
download | BlockedNumberProvider-0b78245b351a8bbd1ed34c14cb9a47c0f9dc5abf.tar.gz |
Permit any app to access canCurrentUserBlockNumbers API.
BUG: 27719849
Change-Id: I041d951f39327ddf4a5295e432da7a85cdff0cd5
-rw-r--r-- | src/com/android/providers/blockednumber/BlockedNumberProvider.java | 3 | ||||
-rw-r--r-- | tests/src/com/android/providers/blockednumber/BlockedNumberProviderTest.java | 7 |
2 files changed, 2 insertions, 8 deletions
diff --git a/src/com/android/providers/blockednumber/BlockedNumberProvider.java b/src/com/android/providers/blockednumber/BlockedNumberProvider.java index 837a35a..fa07a47 100644 --- a/src/com/android/providers/blockednumber/BlockedNumberProvider.java +++ b/src/com/android/providers/blockednumber/BlockedNumberProvider.java @@ -291,8 +291,7 @@ public class BlockedNumberProvider extends ContentProvider { res.putBoolean(BlockedNumberContract.RES_NUMBER_IS_BLOCKED, isBlocked(arg)); break; case BlockedNumberContract.METHOD_CAN_CURRENT_USER_BLOCK_NUMBERS: - enforceReadPermission(); - + // No permission checks: any app should be able to access this API. res.putBoolean( BlockedNumberContract.RES_CAN_BLOCK_NUMBERS, canCurrentUserBlockUsers()); break; diff --git a/tests/src/com/android/providers/blockednumber/BlockedNumberProviderTest.java b/tests/src/com/android/providers/blockednumber/BlockedNumberProviderTest.java index de6a096..ca7dfb8 100644 --- a/tests/src/com/android/providers/blockednumber/BlockedNumberProviderTest.java +++ b/tests/src/com/android/providers/blockednumber/BlockedNumberProviderTest.java @@ -524,12 +524,7 @@ public class BlockedNumberProviderTest extends AndroidTestCase { doReturn(PackageManager.PERMISSION_DENIED) .when(mMockContext).checkCallingPermission(anyString()); - try { - BlockedNumberContract.canCurrentUserBlockNumbers(mMockContext); - fail("SecurityException expected"); - } catch (SecurityException expected) { - } - + assertFalse(BlockedNumberContract.canCurrentUserBlockNumbers(mMockContext)); try { insert(cv(BlockedNumbers.COLUMN_ORIGINAL_NUMBER, "123")); |