Fix softoff crash due to double free corruption
In its present condition, the instance database gets initialized
within the hostSoftOff function. Consequently, once the function
execution is complete, the database goes out of scope, triggering
the invocation of the instanceDb destructor. This, in turn,
prematurely destroys the instance database in the middle of the
softpoweroff flow, resulting in a double free corruption.
Tested:
After implementing the fix, softoff was tested to ensure it does
not crash.
Change-Id: I251201b06864a8a5273bfaa23468a06ad678763a
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>
diff --git a/softoff/main.cpp b/softoff/main.cpp
index e5f61d2..cfbbcd2 100644
--- a/softoff/main.cpp
+++ b/softoff/main.cpp
@@ -1,3 +1,4 @@
+#include "common/instance_id.hpp"
#include "common/utils.hpp"
#include "softoff.hpp"
@@ -13,10 +14,13 @@
// Get a handle to system D-Bus.
auto& bus = pldm::utils::DBusHandler::getBus();
+ // Obtain the instance database
+ pldm::InstanceIdDb instanceIdDb;
+
// Attach the bus to sd_event to service user requests
bus.attach_event(event.get(), SD_EVENT_PRIORITY_NORMAL);
- pldm::SoftPowerOff softPower(bus, event.get());
+ pldm::SoftPowerOff softPower(bus, event.get(), instanceIdDb);
if (softPower.isError())
{