commit 54b5a563950e1fbfad3c5f165dcc855758303e61
author Pavithra Barithaya <pavithra.b@ibm.com> Mon Sep 27 06:07:10 2021 -0500
committer Tom Joseph <rushtotom@gmail.com> Tue Sep 28 08:03:31 2021 +0000
tree b19a851c805a5dcd9284b3c4f2dc129af9b16d55
parent 141a86be83ab38dd1676d690477eca67586ca14a

PLDM : fix to remove a trace

The trace "Failed to decode_platform_event_message_resp:
rc=3, cc=0" was due to a minor bug in the code.
decode_platform_event_message_resp should not be subtracting
the sizeof(pldm_msg_hdr) from respMsgLen. This commit fixes that.

Signed-off-by: Pavithra Barithaya <pavithra.b@ibm.com>
Change-Id: I951d3e310f356e421b4e29e1217a05bd7b72c372

host-bmc/host_pdr_handler.cpp

diff --git a/host-bmc/host_pdr_handler.cpp b/host-bmc/host_pdr_handler.cpp
index e09989a..e5a3e6b 100644
--- a/host-bmc/host_pdr_handler.cpp
+++ b/host-bmc/host_pdr_handler.cpp
@@ -310,9 +310,8 @@
         uint8_t completionCode{};
         uint8_t status{};
         auto responsePtr = reinterpret_cast<const struct pldm_msg*>(response);
-        auto rc = decode_platform_event_message_resp(
-            responsePtr, respMsgLen - sizeof(pldm_msg_hdr), &completionCode,
-            &status);
+        auto rc = decode_platform_event_message_resp(responsePtr, respMsgLen,
+                                                     &completionCode, &status);
         if (rc || completionCode)
         {
             std::cerr << "Failed to decode_platform_event_message_resp: "