commit 3d652709ed40f8070c990bb275d3709d603d318e
author Yu-Han Yang <yuhany@google.com> Thu Mar 05 22:41:23 2020 -0800
committer Yu-Han Yang <yuhany@google.com> Fri Mar 06 07:34:58 2020 -0800
tree 71d2fb7a53b1209c06d854e3cebd7d5ca434bf41
parent a51c8b9f477ca7c2e7e2fff01a98decec2e0c392

Fix deadlock in cuttlefish/default implementation

Bug: 150830099

Test: atest LocationManagerFineTest#testRegisterGnssMeasurementsCallback
-c --iterations 100

Change-Id: I70aec19a481781d924ed3008765ca624a7eeb950

gnss/2.0/default/GnssMeasurement.h

diff --git a/gnss/2.0/default/GnssMeasurement.h b/gnss/2.0/default/GnssMeasurement.h
index d8ffd59..151456c 100644
--- a/gnss/2.0/default/GnssMeasurement.h
+++ b/gnss/2.0/default/GnssMeasurement.h
@@ -61,10 +61,12 @@
     void stop();
     void reportMeasurement(const GnssData&);
 
+    // Guarded by mMutex
     static sp<IGnssMeasurementCallback> sCallback;
     std::atomic<long> mMinIntervalMillis;
     std::atomic<bool> mIsActive;
     std::thread mThread;
+    // Synchronization lock for sCallback
     mutable std::mutex mMutex;
 };