tree 6f4216b1b76d321522c6a0b4093f7e7f49ad06d1
parent 964154309b4ffbf26bd1536b3bef4045aabbc660
author Jae Hyun Yoo <jae.hyun.yoo@intel.com> 1521851176 -0700
committer Jae Hyun Yoo <jae.hyun.yoo@intel.com> 1522105531 -0700

Fix a FruDevice service crash issue

In some systems, the FruDevice service stopped while it is adding
Dbus properties with below logs:

Mar 19 17:17:38 wolfpass systemd[1]: Starting Fru Device...
Mar 19 17:17:38 wolfpass fru-device[1061]: failed to read bus 5 address 112
Mar 19 17:18:04 wolfpass fru-device[1061]: failed to read bus 5 address 112
Mar 19 17:18:18 wolfpass fru-device[1061]: process 1061: arguments to dbus_message_iter_append_basic() were incorrect, asserti
on "_dbus_check_is_valid_utf8 (*string_p)" failed in file ../../dbus-1.10.10/dbus/dbus-message.c line 2749.
Mar 19 17:18:18 wolfpass fru-device[1061]: This is normally a bug in some application using the D-Bus library.
Mar 19 17:18:18 wolfpass fru-device[1061]:   D-Bus not built with -rdynamic so unable to print a backtrace
Mar 19 17:18:28 wolfpass systemd[1]: [[0;1;39mxyz.openbmc_project.FruDevice.service: Main process exited, code=killed, status=
6/ABRT[[0m
Mar 19 17:18:28 wolfpass systemd[1]: [[0;1;31mFailed to start Fru Device.[[0m
Mar 19 17:18:28 wolfpass systemd[1]: [[0;1;39mxyz.openbmc_project.FruDevice.service: Unit entered failed state.[[0m
Mar 19 17:18:28 wolfpass systemd[1]: [[0;1;39mxyz.openbmc_project.FruDevice.service: Failed with result 'signal'.[[0m

This is caused by an invalid string property from some components
so this patch adds a regex_replace() call to avoid _dbus_assert
crashes.

Change-Id: I551f12f028e08685ce5dbb3a3f770d199ff8fc3b
Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@intel.com>