commit 54ffb271b995db4f0b00f0661baab23763fae5bd
author AppaRao Puli <apparao.puli@linux.intel.com> Tue Jun 02 19:09:38 2020 +0530
committer AppaRao Puli <apparao.puli@linux.intel.com> Wed Jun 03 21:29:54 2020 +0530
tree e76ebe085d269516f442bb69759b16b2da3d9063
parent 41061e2c3198c0f597d4f6bb702b690a273ab45d

Fix for service crash - hwmontempsensor

D-Bus object path accepts only "[a-zA-Z0-9_/]+".
If any invalid characters present on path leads to
throw exception at sdbusplus and leads to service
crashes.

  terminate called after throwing an instance of
  'sdbusplus::exception::SdBusError'
  what():  Invalid path or interface:
  org.freedesktop.DBus.Error.InvalidArgs: Invalid argument

Example sensor name: "PCH M.2 Temp"

Replace any invalid characters with "_".

Tested:
 No service crashes observed after fix.

Change-Id: Ie487c732db253d01e7c20fa1e79b86a3cca2b263
Signed-off-by: AppaRao Puli <apparao.puli@linux.intel.com>

include/sensor.hpp

diff --git a/include/sensor.hpp b/include/sensor.hpp
index 4e98089..ff272b5 100644
--- a/include/sensor.hpp
+++ b/include/sensor.hpp
@@ -18,7 +18,7 @@
            std::vector<thresholds::Threshold>&& thresholdData,
            const std::string& configurationPath, const std::string& objectType,
            const double max, const double min) :
-        name(name),
+        name(std::regex_replace(name, std::regex("[^a-zA-Z0-9_/]+"), "_")),
         configurationPath(configurationPath), objectType(objectType),
         maxValue(max), minValue(min), thresholds(std::move(thresholdData)),
         hysteresisTrigger((max - min) * 0.01),