diff options
| author | James Feist <james.feist@linux.intel.com> | 2019-10-30 02:00:38 +0300 |
|---|---|---|
| committer | Feist, James <james.feist@intel.com> | 2019-10-30 20:53:26 +0300 |
| commit | eb3be7affc678af30e5aba6225218935d5fad4eb (patch) | |
| tree | 7608a50ef4f7debb330ca672b6d65ea661830482 | |
| parent | e81b1a8aab6858cdb42e9456f41fbd07b5c2fb7c (diff) | |
| download | provingground-eb3be7affc678af30e5aba6225218935d5fad4eb.tar.xz | |
Fix Associations
It was spelled wrong. Also, there is no need to compare
against the local object if it is globally critical, so
make the check easier.
Tested: Made a drive error and the led blinked
Change-Id: I52590f0888380a6d7460e9220d814174cb17a52b
Signed-off-by: James Feist <james.feist@linux.intel.com>
| -rw-r--r-- | callback-manager/src/callback_manager.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/callback-manager/src/callback_manager.cpp b/callback-manager/src/callback_manager.cpp index ac2fb02..2fe5e33 100644 --- a/callback-manager/src/callback_manager.cpp +++ b/callback-manager/src/callback_manager.cpp @@ -270,7 +270,7 @@ void createAssociationMatch(std::shared_ptr<sdbusplus::asio::connection>& conn) << "\n"; } - auto findAssociations = values.find("associations"); + auto findAssociations = values.find("Associations"); if (findAssociations == values.end()) { return; @@ -318,7 +318,7 @@ void createAssociationMatch(std::shared_ptr<sdbusplus::asio::connection>& conn) bool fatal = globalCritical && localCritical; bool critical = globalWarning && localCritical; - bool warning = globalWarning && localWarning; + bool warning = globalWarning && !critical; fatalAssertMap[message.get_path()]["association"] = fatal; criticalAssertMap[message.get_path()]["association"] = critical; |