pldmd: fix crash when re-request D-Bus name
`bus.request_name("xyz.openbmc_project.PLDM")` throws D-Bus exception
when the name `xyz.openbmc_project.PLDM` is already requested. This
causes the `pldmd` service will be crashed when build pldm source with
`oem-ibm` disabled & `system-specific-bios-json` disabled. Add
`try...catch...` to prevent the crash of `pldmd`.
Tested:
Was able to successfully start pldm service and see that the bus name
`xyz.openbmc_project.PLDM` was claimed with these options setups:
+ `oem-ibm` disabled & `system-specific-bios-json` enabled.
+ `oem-ibm` disabled & `system-specific-bios-json` disabled.
Signed-off-by: Thu Nguyen <thu@os.amperecomputing.com>
Change-Id: I2ac70b686d1468b9c7484d5bf2543db2b4f33ded
diff --git a/libpldmresponder/bios_config.cpp b/libpldmresponder/bios_config.cpp
index 2b1a901..51fad71 100644
--- a/libpldmresponder/bios_config.cpp
+++ b/libpldmresponder/bios_config.cpp
@@ -92,7 +92,10 @@
{
error("System specific bios attribute directory {DIR} does not exit",
"DIR", dir);
- requestPLDMServiceName();
+ if (registerService)
+ {
+ requestPLDMServiceName();
+ }
return;
}
constructAttributes();