commit 6a4d1cf836f2bf291890d56ad5d071f205ea3ef0
author Varsha Kaverappa <vkaverap@in.ibm.com> Wed Nov 24 21:15:42 2021 -0600
committer ManojKiran Eda <manojkiran.eda@gmail.com> Thu Dec 16 05:17:12 2021 +0000
tree 01ed5de68699e7e04695a3be659f67610525d1eb
parent d02724816608f1118a1e3c16ddb7b7a771262717

PLDM BIOS attr enum return error if byte value is 2

Signed-off-by: Varsha Kaverappa <vkaverap@in.ibm.com>
Change-Id: I67da5157da04649474ff4fa770f7d2652a09e607

libpldmresponder/bios_config.cpp

diff --git a/libpldmresponder/bios_config.cpp b/libpldmresponder/bios_config.cpp
index e955f26..c808002 100644
--- a/libpldmresponder/bios_config.cpp
+++ b/libpldmresponder/bios_config.cpp
@@ -642,7 +642,10 @@
                 table::attribute_value::decodeEnumEntry(attrValueEntry);
             auto [pvHdls, defIndex] =
                 table::attribute::decodeEnumEntry(attrEntry);
-            assert(value.size() == 1);
+            if (!(value.size() == 1))
+            {
+                return PLDM_ERROR_INVALID_LENGTH;
+            }
             if (value[0] >= pvHdls.size())
             {
                 std::cerr << "Enum: Illgeal index, Index = " << (int)value[0]