commit c727fb40e59284f6fcd8d8690ad00a633b7e4990
author Andrew Jeffery <andrew@aj.id.au> Wed Jul 12 09:19:02 2023 +0930
committer ManojKiran Eda <manojkiran.eda@gmail.com> Wed Jul 19 06:24:19 2023 +0000
tree 4010cc5df6ab06985b45e64738f3b4394106b071
parent 2af90d318b4d19926e351409de1e49ac30f5654a

pldm: Migrate to pldm_bios_table_attr_value_entry_encode_integer_check()

pldm_bios_table_attr_value_entry_encode_integer() is deprecated in
libpldm as it uses assert() to sanitize its parameters. Use
pldm_bios_table_attr_value_entry_encode_integer_check() instead, which
returns a value indicating success or failure.

Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: Ic308d24a83fe3371a7ef074cb1d20092a06a1291

libpldmresponder/bios_table.cpp

diff --git a/libpldmresponder/bios_table.cpp b/libpldmresponder/bios_table.cpp
index f4fcecf..3598277 100644
--- a/libpldmresponder/bios_table.cpp
+++ b/libpldmresponder/bios_table.cpp
@@ -329,8 +329,16 @@
 
     auto tableSize = table.size();
     table.resize(tableSize + entryLength);
-    pldm_bios_table_attr_value_entry_encode_integer(
+    int rc = pldm_bios_table_attr_value_entry_encode_integer_check(
         table.data() + tableSize, entryLength, attrHandle, attrType, value);
+    if (rc != PLDM_SUCCESS)
+    {
+        lg2::error(
+            "Failed to encode BIOS attribute table integer entry: {LIBPLDM_ERROR}",
+            "LIBPLDM_ERROR", rc);
+        throw std::runtime_error(
+            "Failed to encode BIOS attribute table integery entry");
+    }
     return reinterpret_cast<pldm_bios_attr_val_table_entry*>(table.data() +
                                                              tableSize);
 }