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>
1 file changed
tree: e76ebe085d269516f442bb69759b16b2da3d9063
  1. cmake/
  2. include/
  3. service_files/
  4. src/
  5. tests/
  6. .clang-format
  7. .clang-ignore
  8. .gitignore
  9. cmake-format.json
  10. CMakeLists.txt
  11. Jenkinsfile
  12. LICENSE
  13. MAINTAINERS
  14. README.md
README.md

dbus-sensors

dbus-sensors is a collection of sensor applications that provide the xyz.openbmc_project.Sensor collection of interfaces. They read sensor values from hwmon, d-bus, or direct driver access to provide readings. Some advance non-sensor features such as fan presence, pwm control, and automatic cpu detection (x86) are also supported.

key features

  • runtime re-configurable from d-bus (entity-manager or the like)

  • isolated: each sensor type is isolated into its own daemon, so a bug in one sensor is unlikely to affect another, and single sensor modifications are possible

  • async single-threaded: uses sdbusplus/asio bindings

  • multiple data inputs: hwmon, d-bus, direct driver access