diff options
| author | Ed Tanous <ed.tanous@intel.com> | 2019-10-11 20:40:32 +0300 |
|---|---|---|
| committer | Ed Tanous <ed.tanous@intel.com> | 2019-10-11 20:40:32 +0300 |
| commit | a6f9392101c6adb9f4ad57d349c392b2396c2cb1 (patch) | |
| tree | 96bb248a460ba4ccad12690788dd6eff48521419 /redfish-core/include/utils | |
| parent | 656ec7e317a5ebf0e8dd3068f321a54cebea2273 (diff) | |
| download | bmcweb-a6f9392101c6adb9f4ad57d349c392b2396c2cb1.tar.xz | |
Fix state property on firmware properties
In the case of non-updatable firmware entities, the State variable was
not getting populated due to a poorly handled error condition.
This resolves it, and assumes non-updatable entities are just "Enabled"
Tested: Need help here. I'm not able to reproduce the original issue
on my system, so need someone else to test that this resolves it.
Signed-off-by: Ed Tanous <ed.tanous@intel.com>
Change-Id: Ibd5db741cb8b0b2451a04fe14157dc1d3cbbcc03
Diffstat (limited to 'redfish-core/include/utils')
| -rw-r--r-- | redfish-core/include/utils/fw_utils.hpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/redfish-core/include/utils/fw_utils.hpp b/redfish-core/include/utils/fw_utils.hpp index c5cf1f80db..296c900da2 100644 --- a/redfish-core/include/utils/fw_utils.hpp +++ b/redfish-core/include/utils/fw_utils.hpp @@ -240,6 +240,7 @@ void getFwStatus(std::shared_ptr<AsyncResp> asyncResp, if (error_code) { // not all fwtypes are updateable, this is ok + asyncResp->res.jsonValue["Status"]["State"] = "Enabled"; return; } boost::container::flat_map<std::string, VariantType>::const_iterator |