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

Commit b7e99227 authored by Mike Lockwood's avatar Mike Lockwood
Browse files

LocationManagerService: Use Context.checkCallingOrSelfPermission for permission checks 



so other services running in the system process can use the location manager.

Signed-off-by: default avatarMike Lockwood <lockwood@android.com>
parent 61775060

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

+8 −8
Original line number Diff line number Diff line
@@ -648,14 +648,14 @@ public class LocationManagerService extends ILocationManager.Stub implements Run 


    private void checkPermissionsSafe(String provider) {

        if (LocationManager.GPS_PROVIDER.equals(provider)

            && (mContext.checkCallingPermission(ACCESS_FINE_LOCATION)

            && (mContext.checkCallingOrSelfPermission(ACCESS_FINE_LOCATION)

                != PackageManager.PERMISSION_GRANTED)) {

            throw new SecurityException("Requires ACCESS_FINE_LOCATION permission");

        }

        if (LocationManager.NETWORK_PROVIDER.equals(provider)

            && (mContext.checkCallingPermission(ACCESS_FINE_LOCATION)

            && (mContext.checkCallingOrSelfPermission(ACCESS_FINE_LOCATION)

                != PackageManager.PERMISSION_GRANTED)

            && (mContext.checkCallingPermission(ACCESS_COARSE_LOCATION)

            && (mContext.checkCallingOrSelfPermission(ACCESS_COARSE_LOCATION)

                != PackageManager.PERMISSION_GRANTED)) {

            throw new SecurityException(

                "Requires ACCESS_FINE_LOCATION or ACCESS_COARSE_LOCATION permission");
@@ -664,14 +664,14 @@ public class LocationManagerService extends ILocationManager.Stub implements Run 


    private boolean isAllowedProviderSafe(String provider) {

        if (LocationManager.GPS_PROVIDER.equals(provider)

            && (mContext.checkCallingPermission(ACCESS_FINE_LOCATION)

            && (mContext.checkCallingOrSelfPermission(ACCESS_FINE_LOCATION)

                != PackageManager.PERMISSION_GRANTED)) {

            return false;

        }

        if (LocationManager.NETWORK_PROVIDER.equals(provider)

            && (mContext.checkCallingPermission(ACCESS_FINE_LOCATION)

            && (mContext.checkCallingOrSelfPermission(ACCESS_FINE_LOCATION)

                != PackageManager.PERMISSION_GRANTED)

            && (mContext.checkCallingPermission(ACCESS_COARSE_LOCATION)

            && (mContext.checkCallingOrSelfPermission(ACCESS_COARSE_LOCATION)

                != PackageManager.PERMISSION_GRANTED)) {

            return false;

        }
@@ -1075,7 +1075,7 @@ public class LocationManagerService extends ILocationManager.Stub implements Run 
        if (mGpsStatusProvider == null) {

            return false;

        }

        if (mContext.checkCallingPermission(ACCESS_FINE_LOCATION) !=

        if (mContext.checkCallingOrSelfPermission(ACCESS_FINE_LOCATION) !=

                PackageManager.PERMISSION_GRANTED) {

            throw new SecurityException("Requires ACCESS_FINE_LOCATION permission");

        }
@@ -1103,7 +1103,7 @@ public class LocationManagerService extends ILocationManager.Stub implements Run 
        // first check for permission to the provider

        checkPermissionsSafe(provider);

        // and check for ACCESS_LOCATION_EXTRA_COMMANDS

        if ((mContext.checkCallingPermission(ACCESS_LOCATION_EXTRA_COMMANDS)

        if ((mContext.checkCallingOrSelfPermission(ACCESS_LOCATION_EXTRA_COMMANDS)

                != PackageManager.PERMISSION_GRANTED)) {

            throw new SecurityException("Requires ACCESS_LOCATION_EXTRA_COMMANDS permission");

        }