commit d6c397b96f642f26d4b153a064b853bb92016b9e
author Hariharan Rangasamy <hariharanr@ami.com> Thu Oct 30 11:49:22 2025 +0530
committer Mohammed Javith Akthar M <mohammedjavitham@ami.com> Mon Nov 03 06:01:37 2025 +0000
tree 9f630cac8d14a15d93fee9367b3c736155399f1f
parent 19c962b8d2f324408250229053eb4d7abacc4865

Fix cppcheck errors

dbus-sdr/sensorcommands.cpp:927:17: error: Uninitialized variable:
operation [uninitvar]
operation |= static_cast<uint8_t>(

dbus-sdr/sensorcommands.cpp:1623:13: error: Uninitialized variable:
sensorEventStatus [uninitvar]
sensorEventStatus |= static_cast<uint8_t>(

Change-Id: I43d68e7660eb35d375e59b3f17909e8349cc622a
Signed-off-by: Hariharan Rangasamy <hariharanr@ami.com>

dbus-sdr/sensorcommands.cpp

diff --git a/dbus-sdr/sensorcommands.cpp b/dbus-sdr/sensorcommands.cpp
index bd96ea2..83b2dc4 100644
--- a/dbus-sdr/sensorcommands.cpp
+++ b/dbus-sdr/sensorcommands.cpp
@@ -916,7 +916,7 @@
             return ipmi::responseIllegalCommand();
         }
 
-        uint8_t operation;
+        uint8_t operation = 0;
         try
         {
             ipmi::sensor::GetSensorResponse getResponse =
@@ -1617,7 +1617,7 @@
         std::bitset<16> assertions;
         // deassertions are not used.
         std::bitset<16> deassertions = 0;
-        uint8_t sensorEventStatus;
+        uint8_t sensorEventStatus = 0;
         if (response.readingOrStateUnavailable)
         {
             sensorEventStatus |= static_cast<uint8_t>(