platform-mc: stop multipart PDR processing on TRANSFER_END PLDM_PLATFORM_TRANSFER_END indicates that the complete PDR record has been received. After moving the assembled record into terminus->pdrs, continuing the multipart transfer loop is unnecessary and can leave a potential post-move access path for receivedPdr. Break out of the loop immediately after handling TRANSFER_END to make the completion condition explicit, simplify control flow, and resolve the cppcheck accessMoved warning. No functional change is intended for valid transfers. Change-Id: I2f5cb57757dcc69d3d622780f17126777d4d5928 Signed-off-by: Meghana Vangapandu <meghanav@ami.com>
diff --git a/platform-mc/platform_manager.cpp b/platform-mc/platform_manager.cpp index bb90053..c5ef1c3 100644 --- a/platform-mc/platform_manager.cpp +++ b/platform-mc/platform_manager.cpp
@@ -387,6 +387,7 @@ { terminus->pdrs.emplace_back(std::move(receivedPdr)); recordHndl = nextRecordHndl; + break; } } while (nextDataTransferHndl != 0 && receivedRecordSize < largestRecordSize);