commit 3ad197160a5845de3760aa82aa586228f61d19f5
author Lei YU <yulei.sh@bytedance.com> Mon Jan 27 05:23:03 2025 +0000
committer Vernon Mauery <vernon.mauery@gmail.com> Tue Mar 18 15:54:22 2025 +0000
tree 76ec124e2b5800592701f4bf77aa771b57828626
parent 9706bc8d269accb99ec9ada035838f28db441f97

Fix clang++ warnings

To enable clang-tidy, fix the build warnings by clang++.

Signed-off-by: Lei YU <yulei.sh@bytedance.com>
Change-Id: I4a1e8dd897f631493e9600267b3b9b999cda902b

storagehandler.cpp

diff --git a/storagehandler.cpp b/storagehandler.cpp
index d96f2d6..d85673f 100644
--- a/storagehandler.cpp
+++ b/storagehandler.cpp
@@ -376,13 +376,14 @@
         auto readLength =
             std::min(diff, static_cast<int>(requestData->readLength));
 
-        std::memcpy(response, &record.nextRecordID,
-                    sizeof(record.nextRecordID));
-        std::memcpy(static_cast<uint8_t*>(response) +
-                        sizeof(record.nextRecordID),
-                    &record.event.eventRecord.recordID + requestData->offset,
+        uint16_t nextRecordID = record.nextRecordID;
+        std::memcpy(response, &nextRecordID, sizeof(nextRecordID));
+
+        const ipmi::sel::SELEventRecordFormat* evt = &record.event;
+        std::memcpy(static_cast<uint8_t*>(response) + sizeof(nextRecordID),
+                    reinterpret_cast<const uint8_t*>(evt) + requestData->offset,
                     readLength);
-        *data_len = sizeof(record.nextRecordID) + readLength;
+        *data_len = sizeof(nextRecordID) + readLength;
     }
 
     return IPMI_CC_OK;