commit 473d890ea7fa48e1f3925f085e870dae67d68527
author Gokul Sanker V G <gokul.sanker.v.g@intel.com> Fri Feb 18 01:13:53 2022 +0530
committer Gokul Sanker V G <gokul.sanker.v.g@intel.com> Fri Feb 18 04:27:15 2022 +0000
tree e416bd99c27d6f2db0cf5989389225a5205ae29a
parent 0435a483afb10a5eabe7ae93f07fbb2d2265e53f

Added correction for empty text strings

As per SMBIOS specification Text strings
associated with a given SMBIOS structure
are appended directly after the formatted
portion of the structure.
(https://www.dmtf.org/sites/default/files/standards/documents/DSP0134_3.4.0.pdf , sec 6.1.3)

If the formatted portion of the structure
contains string-reference fields and all
the string fields are set to 0
(no string references), the formatted section
of the structure is followed by two null
(00h) BYTES.

Signed-off-by: Gokul Sanker V G <gokul.sanker.v.g@intel.com>
Change-Id: Ib8d0004384bad67168cb367442ba41528a376ae2

include/smbios_mdrv2.hpp

diff --git a/include/smbios_mdrv2.hpp b/include/smbios_mdrv2.hpp
index 4f3b654..ed3cf34 100644
--- a/include/smbios_mdrv2.hpp
+++ b/include/smbios_mdrv2.hpp
@@ -244,7 +244,7 @@
 static inline std::string positionToString(uint8_t positionNum,
                                            uint8_t structLen, uint8_t* dataIn)
 {
-    if (dataIn == nullptr)
+    if (dataIn == nullptr || positionNum == 0)
     {
         return "";
     }