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

Commit 4b05bc36 authored by Nick Desaulniers's avatar Nick Desaulniers Committed by android-build-merger
Browse files

biometrics: fingerprint: add locking to default impl am: e637aa32 

am: 3c739df2

Change-Id: I19626010eb7109ebe397ec7318b9f7aca4777113
parents 2b90ff20 3c739df2

biometrics/fingerprint/2.1/default/BiometricsFingerprint.cpp

+2 −0
Original line number Diff line number Diff line
@@ -145,6 +145,7 @@ FingerprintAcquiredInfo BiometricsFingerprint::VendorAcquiredFilter( 


Return<uint64_t> BiometricsFingerprint::setNotify(

        const sp<IBiometricsFingerprintClientCallback>& clientCallback) {

    std::lock_guard<std::mutex> lock(mClientCallbackMutex);

    mClientCallback = clientCallback;

    // This is here because HAL 2.1 doesn't have a way to propagate a

    // unique token for its driver. Subsequent versions should send a unique
@@ -259,6 +260,7 @@ fingerprint_device_t* BiometricsFingerprint::openHal() { 
void BiometricsFingerprint::notify(const fingerprint_msg_t *msg) {

    BiometricsFingerprint* thisPtr = static_cast<BiometricsFingerprint*>(

            BiometricsFingerprint::getInstance());

    std::lock_guard<std::mutex> lock(thisPtr->mClientCallbackMutex);

    if (thisPtr == nullptr || thisPtr->mClientCallback == nullptr) {

        ALOGE("Receiving callbacks before the client callback is registered.");

        return;

biometrics/fingerprint/2.1/default/BiometricsFingerprint.h

+1 −0
Original line number Diff line number Diff line
@@ -69,6 +69,7 @@ private: 
    static FingerprintAcquiredInfo VendorAcquiredFilter(int32_t error, int32_t* vendorCode);

    static BiometricsFingerprint* sInstance;



    std::mutex mClientCallbackMutex;

    sp<IBiometricsFingerprintClientCallback> mClientCallback;

    fingerprint_device_t *mDevice;

};