commit f1e32c170bed67501a172dd6d0e111dc08b6957f
author tal-yac <talycb8@gmail.com> Mon Aug 12 13:12:30 2024 +0300
committer ManojKiran Eda <manojkiran.eda@gmail.com> Wed Aug 14 07:43:26 2024 +0000
tree 4f5818e82f28d19aa22f777dc8e56ce8a79a7bd2
parent 3166ef23d9ffdc47d09afcea68d7a934c61a9f6c

platform-mc: Fix sensor aux name pdr parsing

This fixes an issue with counting the null terminator twice:
both on the name length and on the pdr pointer increment.
This caused to_bytes to throw an exception on sensors with composite
sensor count larger than 1.

Change-Id: I85c2c12854698402b213fe2dd66a42ac8c299902
Signed-off-by: tal-yac <talycb8@gmail.com>
Signed-off-by: Thu Nguyen <thu@os.amperecomputing.com>

platform-mc/terminus.cpp

diff --git a/platform-mc/terminus.cpp b/platform-mc/terminus.cpp
index 5b2ab3c..004158a 100644
--- a/platform-mc/terminus.cpp
+++ b/platform-mc/terminus.cpp
@@ -283,8 +283,7 @@
             }
             memcpy(alignedBuffer, ptr, u16NameStringLen * sizeof(uint16_t));
             std::u16string u16NameString(alignedBuffer, u16NameStringLen);
-            ptr += (u16NameString.size() + sizeof(nullTerminator)) *
-                   sizeof(uint16_t);
+            ptr += u16NameString.size() * sizeof(uint16_t);
             std::transform(u16NameString.cbegin(), u16NameString.cend(),
                            u16NameString.begin(),
                            [](uint16_t utf16) { return be16toh(utf16); });