Skip to content

Commit 18c2b6e

Browse files
Victoria LeaseAndroid (Google) Code Review
authored andcommitted
Merge "throw SecurityException in isProviderEnabled()" into jb-mr1-dev
2 parents b3f55fd + f429921 commit 18c2b6e

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

services/java/com/android/server/LocationManagerService.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1292,8 +1292,13 @@ public ProviderProperties getProviderProperties(String provider) {
12921292

12931293
@Override
12941294
public boolean isProviderEnabled(String provider) {
1295-
checkPermission();
1295+
String perms = checkPermission();
12961296
if (LocationManager.FUSED_PROVIDER.equals(provider)) return false;
1297+
if (ACCESS_COARSE_LOCATION.equals(perms) &&
1298+
!isProviderAllowedByCoarsePermission(provider)) {
1299+
throw new SecurityException("The \"" + provider +
1300+
"\" provider requires ACCESS_FINE_LOCATION permission");
1301+
}
12971302

12981303
synchronized (mLock) {
12991304
LocationProviderInterface p = mProvidersByName.get(provider);

0 commit comments

Comments
 (0)