commit f2fc1b3e464b5bc0690b5683b3b4f6dc2bede5d8
author Vikash Chandola <vikash.chandola@intel.com> Thu Aug 29 15:59:27 2024 +0530
committer Zhikui Ren <zhikui.ren@intel.com> Tue Sep 17 20:23:57 2024 +0000
tree 64a2719849dec8614c6be2114d6bbcfa69ec8fcd
parent 10480779a99326039e7ed22a12e16a7345e01443

psusensor: Check for present field before using

Using Present field in cpuPresenceHandler without checking for it's
availability may lead to undefined behavior. Check for "Present" field
presence in received signal message before using it.

Tested: psusensor rejected signals if "Present" field is absent.

Change-Id: I083f2c7cfb8b5208eebf85518f8859eee262cf58
Signed-off-by: Vikash Chandola <vikash.chandola@intel.com>

src/PSUSensorMain.cpp

diff --git a/src/PSUSensorMain.cpp b/src/PSUSensorMain.cpp
index f300430..fb4e0d5 100644
--- a/src/PSUSensorMain.cpp
+++ b/src/PSUSensorMain.cpp
@@ -1209,6 +1209,10 @@
             boost::container::flat_map<std::string, std::variant<bool>> values;
             message.read(objectName, values);
             auto findPresence = values.find("Present");
+            if (findPresence == values.end())
+            {
+                return;
+            }
             try
             {
                 cpuPresence[index] = std::get<bool>(findPresence->second);