commit 5591cf0860895607bda0b8ae713e6b05ac0623b1
author James Feist <james.feist@linux.intel.com> Wed Jul 15 16:44:54 2020 -0700
committer James Feist <james.feist@linux.intel.com> Wed Jul 15 16:44:54 2020 -0700
tree ed92c2bfb4c0dd96b31c1fcf31e0335d0462f7a6
parent d320a2e17c960b62728e9921bf46a43d2641ae85

Fix adcsensor crash

We were taking a reference to a temporary variable.
Change it so we don't.

Tested: Passed cycling script that exposed issue

Change-Id: I9226e3d11b5c892cdfe02f138680ed98844596ef
Signed-off-by: James Feist <james.feist@linux.intel.com>

src/HwmonTempMain.cpp

diff --git a/src/HwmonTempMain.cpp b/src/HwmonTempMain.cpp
index e84bf0e..ef38bae 100644
--- a/src/HwmonTempMain.cpp
+++ b/src/HwmonTempMain.cpp
@@ -58,13 +58,13 @@
     boost::container::flat_map<std::string, std::shared_ptr<HwmonTempSensor>>&
         sensors,
     std::shared_ptr<sdbusplus::asio::connection>& dbusConnection,
-    const std::unique_ptr<boost::container::flat_set<std::string>>&
+    const std::shared_ptr<boost::container::flat_set<std::string>>&
         sensorsChanged)
 {
     auto getter = std::make_shared<GetSensorConfiguration>(
         dbusConnection,
         std::move([&io, &objectServer, &sensors, &dbusConnection,
-                   &sensorsChanged](
+                   sensorsChanged](
                       const ManagedObjectType& sensorConfigurations) {
             bool firstScan = sensorsChanged == nullptr;
 
@@ -260,8 +260,8 @@
     boost::container::flat_map<std::string, std::shared_ptr<HwmonTempSensor>>
         sensors;
     std::vector<std::unique_ptr<sdbusplus::bus::match::match>> matches;
-    std::unique_ptr<boost::container::flat_set<std::string>> sensorsChanged =
-        std::make_unique<boost::container::flat_set<std::string>>();
+    auto sensorsChanged =
+        std::make_shared<boost::container::flat_set<std::string>>();
 
     io.post([&]() {
         createSensors(io, objectServer, sensors, systemBus, nullptr);