diff options
author | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2021-11-06 01:07:16 +0000 |
---|---|---|
committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2021-11-06 01:07:16 +0000 |
commit | 7038c97adbc9da93498246edf818b824383f2441 (patch) | |
tree | 94db486a042c231509a1f4b7463b2f56081434e5 | |
parent | 7098082db84902c7393cb6431ec363a77ffe80e4 (diff) | |
parent | 24ead9d84a9d9ef077f4d44995e2add9a79a206d (diff) | |
download | libhardware-7038c97adbc9da93498246edf818b824383f2441.tar.gz |
Snap for 7889170 from 24ead9d84a9d9ef077f4d44995e2add9a79a206d to sc-d2-release
Change-Id: I2c94b948b7e01169cc74b391a5af76b867e643d1
-rw-r--r-- | modules/sensors/dynamic_sensor/DynamicSensorManager.h | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/modules/sensors/dynamic_sensor/DynamicSensorManager.h b/modules/sensors/dynamic_sensor/DynamicSensorManager.h index 634c5234..264582ec 100644 --- a/modules/sensors/dynamic_sensor/DynamicSensorManager.h +++ b/modules/sensors/dynamic_sensor/DynamicSensorManager.h @@ -95,15 +95,18 @@ private: // TF: int foo(sp<BaseSensorObject> obj); template <typename TF> int operateSensor(int handle, TF f) const { - std::lock_guard<std::mutex> lk(mLock); - const auto i = mMap.find(handle); - if (i == mMap.end()) { - return BAD_VALUE; - } - sp<BaseSensorObject> s = i->second.promote(); - if (s == nullptr) { - // sensor object is already gone - return BAD_VALUE; + sp<BaseSensorObject> s; + { + std::lock_guard<std::mutex> lk(mLock); + const auto i = mMap.find(handle); + if (i == mMap.end()) { + return BAD_VALUE; + } + s = i->second.promote(); + if (s == nullptr) { + // sensor object is already gone + return BAD_VALUE; + } } return f(s); } |