commit 5d8221af6568ebd9c20f9a45d568ae7f76018ea5
author Brad Bishop <bradleyb@fuzziesquirrel.com> Tue Aug 17 13:42:58 2021 -0400
committer Deepak Kodihalli <deepak.kodihalli.83@gmail.com> Thu Aug 19 04:13:49 2021 +0000
tree 61d34229fa52137b293909ae5c0c097e8832bf7e
parent 20aa3e09fb866c1b99c1d3d702f2a2f1908558af

pldmd: fix null pointer dereference

When built with the default meson options, pldmd immediately fails to
start because of this null pointer dereference.

Change-Id: I1eed64d2491291d2607c19a673f41467b6658ee0
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

pldmd/pldmd.cpp

diff --git a/pldmd/pldmd.cpp b/pldmd/pldmd.cpp
index cc7e87b..3653855 100644
--- a/pldmd/pldmd.cpp
+++ b/pldmd/pldmd.cpp
@@ -250,16 +250,19 @@
     sdbusplus::xyz::openbmc_project::PLDM::server::Event dbusImplEvent(
         bus, "/xyz/openbmc_project/pldm");
 
-    hostPDRHandler->setHostState();
-    if (hostPDRHandler->isHostUp())
+    if (hostPDRHandler)
     {
-        hostPDRHandler->getHostPDR();
-    }
-    else
-    {
-        if (verbose)
+        hostPDRHandler->setHostState();
+        if (hostPDRHandler->isHostUp())
         {
-            std::cout << "Host is not running\n";
+            hostPDRHandler->getHostPDR();
+        }
+        else
+        {
+            if (verbose)
+            {
+                std::cout << "Host is not running\n";
+            }
         }
     }
 #endif