commit 160e4a1dee6e4aca54190c057884d0ddf869d2ef
author Sam Dubey <dubeyshubham@google.com> Fri Apr 22 13:35:08 2022 +0000
committer Sam Dubey <dubeyshubham@google.com> Fri Apr 22 13:35:08 2022 +0000
tree ce930b66c536a7e2306741f739be27b568e54cbf
parent b6686e725a5a06757d66898562f20d5ea04ea9b3

Revert "Fix use-after-free in GnssMesaurementInterface"

This reverts commit b6686e725a5a06757d66898562f20d5ea04ea9b3.

Reason for revert: Broke LocationManagerFineTest

Bug: 230085457
Change-Id: Ic699cf1583dd9b858cb5dcd2545ca448a17def6e

gnss/aidl/default/GnssMeasurementInterface.cpp

diff --git a/gnss/aidl/default/GnssMeasurementInterface.cpp b/gnss/aidl/default/GnssMeasurementInterface.cpp
index 228d5bf..2c7241b 100644
--- a/gnss/aidl/default/GnssMeasurementInterface.cpp
+++ b/gnss/aidl/default/GnssMeasurementInterface.cpp
@@ -106,14 +106,12 @@
             std::this_thread::sleep_for(std::chrono::milliseconds(mMinIntervalMillis));
         }
     });
+    mThread.detach();
 }
 
 void GnssMeasurementInterface::stop() {
     ALOGD("stop");
     mIsActive = false;
-    if (mThread.joinable()) {
-        mThread.join();
-    }
 }
 
 void GnssMeasurementInterface::reportMeasurement(const GnssData& data) {