commit e715d14b258a764f0e41559fe4a5e9f7d2efe314
author Ed Tanous <ed@tanous.net> Thu Mar 07 15:47:37 2024 -0800
committer Ed Tanous <ed@tanous.net> Thu Mar 07 16:44:33 2024 -0800
tree 57c49353681da3f862cfb8a765db3f17d4b751dd
parent ce73d5c8517e0b85d85de345f10c82c24cbb067b

Fix behavior when service is missing

Fix regression as part of 33e1f122b740c5de679dc0350b5f41e8d975499f
AllowedHostTransitions: look for on dbus

I have no idea why checking against .value() matters here, but on
systems that doesn't have this daemon present, checking against ec ==
instead of ec.value() == causes this to return 500.

Loaded on qemu, without an implementation of AllowedValues.
```
curl -vvvv  -k --user "root:0penBmc" -H "Content-Type: application/json" https://192.168.7.2/redfish/v1/Systems/system/ResetActionInfo
```

Now succeeds.

Change-Id: I05758c3f9fd9d0f1c90830ba005c74bc6f874bd6
Signed-off-by: Ed Tanous <ed@tanous.net>

redfish-core/lib/systems.hpp

diff --git a/redfish-core/lib/systems.hpp b/redfish-core/lib/systems.hpp
index 37599e7..a8751b0 100644
--- a/redfish-core/lib/systems.hpp
+++ b/redfish-core/lib/systems.hpp
@@ -3643,8 +3643,9 @@
 
     if (ec)
     {
-        if (ec == boost::system::linux_error::bad_request_descriptor ||
-            ec == boost::asio::error::basic_errors::host_unreachable)
+        if ((ec.value() ==
+             boost::system::linux_error::bad_request_descriptor) ||
+            (ec.value() == boost::asio::error::basic_errors::host_unreachable))
         {
             // Property not implemented so just return defaults
             BMCWEB_LOG_DEBUG("Property not available {}", ec);