From 19b835abf9472f6308a85a459404696927debb71 Mon Sep 17 00:00:00 2001 From: mattgilbride Date: Mon, 7 Nov 2022 14:24:15 +0000 Subject: [PATCH] services.core.unboosted - call PermissionEnforcer helper on methods annotated with @EnforcePermission Migrates all existing java methods annotated with @EnforcePermission such that they call the super._enforcePermission method provided by PermissionEnforcer. This required by the @EnforcePermission annotation, but is currently a no-op, and will be switched on with ag/20325484. This CL migrates incremental additions made since the original LSC ag/20189550. Bug: 234083358 Test: TH Change-Id: Ia8b9c10c7af41ac6dff699e6b78f012d50389e6c --- .../java/com/android/server/biometrics/BiometricService.java | 2 ++ .../biometrics/sensors/fingerprint/FingerprintService.java | 2 ++ .../core/java/com/android/server/input/InputManagerService.java | 2 ++ 3 files changed, 6 insertions(+) diff --git a/services/core/java/com/android/server/biometrics/BiometricService.java b/services/core/java/com/android/server/biometrics/BiometricService.java index fab7f1dfb812b..d97195383778f 100644 --- a/services/core/java/com/android/server/biometrics/BiometricService.java +++ b/services/core/java/com/android/server/biometrics/BiometricService.java @@ -780,6 +780,8 @@ public class BiometricService extends SystemService { @Override // Binder call public void resetLockout( int userId, byte[] hardwareAuthToken) { + super.resetLockout_enforcePermission(); + Slog.d(TAG, "resetLockout(userId=" + userId + ", hat=" + (hardwareAuthToken == null ? "null " : "present") + ")"); mBiometricContext.getAuthSessionCoordinator() diff --git a/services/core/java/com/android/server/biometrics/sensors/fingerprint/FingerprintService.java b/services/core/java/com/android/server/biometrics/sensors/fingerprint/FingerprintService.java index 48367b28bb406..dca9ef25a818f 100644 --- a/services/core/java/com/android/server/biometrics/sensors/fingerprint/FingerprintService.java +++ b/services/core/java/com/android/server/biometrics/sensors/fingerprint/FingerprintService.java @@ -942,6 +942,8 @@ public class FingerprintService extends SystemService { @android.annotation.EnforcePermission(android.Manifest.permission.USE_BIOMETRIC_INTERNAL) @Override public void setUdfpsOverlay(@NonNull IUdfpsOverlay controller) { + super.setUdfpsOverlay_enforcePermission(); + for (ServiceProvider provider : mRegistry.getProviders()) { provider.setUdfpsOverlay(controller); } diff --git a/services/core/java/com/android/server/input/InputManagerService.java b/services/core/java/com/android/server/input/InputManagerService.java index d2282c3ee072e..20dc9cdd2efc5 100644 --- a/services/core/java/com/android/server/input/InputManagerService.java +++ b/services/core/java/com/android/server/input/InputManagerService.java @@ -2676,6 +2676,8 @@ public class InputManagerService extends IInputManager.Stub @EnforcePermission(Manifest.permission.BLUETOOTH) @Override public String getInputDeviceBluetoothAddress(int deviceId) { + super.getInputDeviceBluetoothAddress_enforcePermission(); + return mNative.getBluetoothAddress(deviceId); }