commit 9e458e107fa65e11381393c3b6c2f89367f431c9
author Kasun Athukorala <kasunath@google.com> Sat Sep 13 00:37:59 2025 +0000
committer Andrew Jeffery <andrew@codeconstruct.com.au> Tue Sep 30 10:34:52 2025 +0930
tree 6c51c106a0e39026c9e97b665d9b4e1016a587ae
parent d50035f598d40309c20c0c75024fda73a6d6cb62

dsp: base: Omit checksum for Multipart receive ACKNOWLEDGE_COMPLETION

This fix is based on DSP0240 v1.2.0 section 9.6.6, Table 17
DataIntegrityChecksum property description.

DataIntegrityChecksum property should be there unless the
response TransferFlag property is ACKNOWLEDGE_COMPLETION

This patch fixes the issue where DataIntegrityChecksum is only
included if the TransferFlag is PLDM_END or PLDM_START_AND_END

Change-Id: I6d715d21d9d9d55bbda6139108be9cfb288ad209
Fixes: 21a639d6e67d ("dsp: base: Add encode resp for MultipartReceive command")
Signed-off-by: Kasun Athukorala <kasunath@google.com>

tests/dsp/base.cpp

diff --git a/tests/dsp/base.cpp b/tests/dsp/base.cpp
index b2e9294..88539ba 100644
--- a/tests/dsp/base.cpp
+++ b/tests/dsp/base.cpp
@@ -1093,37 +1093,23 @@
 {
     uint8_t instance_id = 0;
     uint8_t completionCode = PLDM_SUCCESS;
-    uint8_t transferFlag = PLDM_BASE_MULTIPART_RECEIVE_TRANSFER_FLAG_START;
-    uint32_t nextDataTransferHandle = 0x16;
-    static constexpr const uint32_t dataLength = 9;
-    std::vector<uint8_t> data = {1, 2, 3, 4, 5, 6, 7, 8, 9};
+    uint8_t transferFlag =
+        PLDM_BASE_MULTIPART_RECEIVE_TRANSFER_FLAG_ACK_COMPLETION;
+    uint32_t nextDataTransferHandle = 0;
+    static constexpr const uint32_t dataLength = 0;
     static constexpr const size_t responseMsgLength =
         PLDM_BASE_MULTIPART_RECEIVE_RESP_MIN_BYTES + dataLength;
     size_t payload_length = responseMsgLength;
 
-    struct variable_field payload = {data.data(), dataLength};
+    struct variable_field payload = {nullptr, dataLength};
     struct pldm_base_multipart_receive_resp resp_data = {
         completionCode, transferFlag, nextDataTransferHandle, payload};
     std::array<uint8_t, responseMsgLength> responseMsg = {
-        completionCode,
-        transferFlag,
-        0x16, // nextDataTransferHandle
-        0x00,
-        0x00,
-        0x00,
-        0x09, // dataLength
-        0x00,
-        0x00,
-        0x00,
-        0x1, // data
-        0x2,
-        0x3,
-        0x4,
-        0x5,
-        0x6,
-        0x7,
-        0x8,
-        0x9};
+        completionCode, transferFlag,
+        0x00, // nextDataTransferHandle
+        0x00,           0x00,         0x00,
+        0x00, // dataLength
+        0x00,           0x00,         0x00};
 
     PLDM_MSG_DEFINE_P(responsePtr, responseMsgLength);
     int rc;