OemLockService: remove canUserAllowOemUnlock()

No longer used by Settings as the logic was wrong.

Test: gts-tradefed run gts -m GtsOemLockServiceTestCases
Bug: 65124732
Change-Id: I44e5f697aabd2b5eefecf64060502b5c9ef5f911
(cherry picked from commit d37fe2d3e1)
This commit is contained in:
Andrew Scull
2017-09-07 14:03:21 +01:00
parent 5ead92fd93
commit 85be690734
3 changed files with 6 additions and 33 deletions

View File

@@ -28,7 +28,6 @@ interface IOemLockService {
void setOemUnlockAllowedByUser(boolean allowed);
boolean isOemUnlockAllowedByUser();
boolean canUserAllowOemUnlock();
boolean isOemUnlockAllowed();
boolean isDeviceOemUnlocked();
}

View File

@@ -117,24 +117,6 @@ public class OemLockManager {
}
}
/**
* Returns whether all parties other than the user allow OEM unlock meaning the user can
* directly control whether or not the device can be OEM unlocked.
*
* If this is true, {@link #isOemUnlockAllowedByUser} is the same as {@link #isOemUnlockAllowed}
*
* @return Whether the user can directly control whether the device can be OEM unlocked.
*
* @hide
*/
public boolean canUserAllowOemUnlock() {
try {
return mService.canUserAllowOemUnlock();
} catch (RemoteException e) {
throw e.rethrowFromSystemServer();
}
}
/**
* @return Whether the bootloader is able to OEM unlock the device.
*

View File

@@ -149,8 +149,12 @@ public class OemLockService extends SystemService {
final long token = Binder.clearCallingIdentity();
try {
if (!canUserAllowOemUnlock()) {
throw new SecurityException("User cannot allow OEM unlock");
if (!isOemUnlockAllowedByAdmin()) {
throw new SecurityException("Admin does not allow OEM unlock");
}
if (!mOemLock.isOemUnlockAllowedByCarrier()) {
throw new SecurityException("Carrier does not allow OEM unlock");
}
mOemLock.setOemUnlockAllowedByDevice(allowedByUser);
@@ -171,18 +175,6 @@ public class OemLockService extends SystemService {
}
}
@Override
public boolean canUserAllowOemUnlock() {
enforceOemUnlockReadPermission();
final long token = Binder.clearCallingIdentity();
try {
return isOemUnlockAllowedByAdmin() && mOemLock.isOemUnlockAllowedByCarrier();
} finally {
Binder.restoreCallingIdentity(token);
}
}
@Override
public boolean isOemUnlockAllowed() {
enforceOemUnlockReadPermission();