commit 33a9cd0b234b63e1f32859d2f5071684e4ed651d
author Manojkiran Eda <manojkiran.eda@gmail.com> Mon Jan 22 11:06:59 2024 +0530
committer Manojkiran Eda <manojkiran.eda@gmail.com> Tue Jan 23 12:33:42 2024 +0530
tree 4fac8ac205331c046437b1bb2d4016a743e65db2
parent 7927f90c8b480cd698a4afe8fc27cdad54edadcc

host-bmc: fix fru record table size

There is a bug , where we are removing the header size twice
from the fru record table. Because of this bug, we are chopping
off few last bytes from the fru record table. This commit would fix
the issue.

The first place where the header size is being removed from the
response message is at :
https://github.com/openbmc/pldm/blob/master/pldmd/pldmd.cpp#L142

Change-Id: Id65fd5373deeb12069897e6376bed91ce9a357b9
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>

host-bmc/host_pdr_handler.cpp

diff --git a/host-bmc/host_pdr_handler.cpp b/host-bmc/host_pdr_handler.cpp
index 2d1c436..86eea92 100644
--- a/host-bmc/host_pdr_handler.cpp
+++ b/host-bmc/host_pdr_handler.cpp
@@ -1027,9 +1027,9 @@
                                                    sizeof(pldm_msg_hdr));
         auto responsePtr = reinterpret_cast<const struct pldm_msg*>(response);
         auto rc = decode_get_fru_record_table_resp(
-            responsePtr, respMsgLen - sizeof(pldm_msg_hdr), &cc,
-            &next_data_transfer_handle, &transfer_flag,
-            fru_record_table_data.data(), &fru_record_table_length);
+            responsePtr, respMsgLen, &cc, &next_data_transfer_handle,
+            &transfer_flag, fru_record_table_data.data(),
+            &fru_record_table_length);
 
         if (rc != PLDM_SUCCESS || cc != PLDM_SUCCESS)
         {