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

Commit a453b88b authored by Takayuki Hoshi's avatar Takayuki Hoshi Committed by Android Git Automerger
Browse files

am c1e8f471: am ee57224c: Merge "Fix a bug where LocationManager\'s... 

am c1e8f471: am ee57224c: Merge "Fix a bug where LocationManager\'s addGpsStatusListener does not throw a SecurityException if LocationManagerService.mGpsStatusProvider is null even when ACCESS_FINE_LOCATION permission is not present." into lmp-mr1-dev automerge: e6ad

* commit 'c1e8f471':
  Fix a bug where LocationManager's addGpsStatusListener does not throw a SecurityException if LocationManagerService.mGpsStatusProvider is null even when ACCESS_FINE_LOCATION permission is not present.
parents cdf4a624 c1e8f471

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

+4 −3
Original line number Diff line number Diff line
@@ -1796,9 +1796,6 @@ public class LocationManagerService extends ILocationManager.Stub { 


    @Override

    public boolean addGpsStatusListener(IGpsStatusListener listener, String packageName) {

        if (mGpsStatusProvider == null) {

            return false;

        }

        int allowedResolutionLevel = getCallerAllowedResolutionLevel();

        checkResolutionLevelIsSufficientForProviderUse(allowedResolutionLevel,

                LocationManager.GPS_PROVIDER);
@@ -1813,6 +1810,10 @@ public class LocationManagerService extends ILocationManager.Stub { 
            Binder.restoreCallingIdentity(ident);

        }



        if (mGpsStatusProvider == null) {

            return false;

        }



        try {

            mGpsStatusProvider.addGpsStatusListener(listener);

        } catch (RemoteException e) {