From 0f6bac48544a7b15180fe62f970c6c9760d12b5c Mon Sep 17 00:00:00 2001 From: Peter Cai Date: Fri, 15 Dec 2023 22:48:39 -0500 Subject: [PATCH 09/10] FingerprintService: Do not use HIDL providers when AIDL is present This fixes duplicated sensor ID when config_biometric_sensors is defined on AIDL devices. Change-Id: I9942e91a95106a7a11ab7ddf145d87562d1b7c0d --- .../biometrics/sensors/fingerprint/FingerprintService.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 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 1ba12134ab29..09a24052e18f 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 @@ -913,8 +913,9 @@ public class FingerprintService extends SystemService { filteredInstances = filterAvailableHalInstances(hidlSensors, aidlSensors); final List providers = new ArrayList<>(); - providers.addAll(getHidlProviders(filteredInstances.first)); providers.addAll(getAidlProviders(filteredInstances.second)); + if (providers.isEmpty()) // Only use HIDL providers if AIDL ones are not found + providers.addAll(getHidlProviders(filteredInstances.first)); return providers; }); -- 2.44.0