From ebf7b9b60cb57913ac20b669203cd9192b660b7e Mon Sep 17 00:00:00 2001 From: Patrick Venture Date: Tue, 13 Nov 2018 12:52:30 -0800 Subject: bugfix: skip 0'd crc bytes on reply computation Skip the 0'd CRC bytes on reply computation, mirroring the skipping during request validation. Tested: Verified via client the CRC for the reply now matches expectations. Change-Id: I18da28604a388fb4210c55f3e54410a067b9f10a Signed-off-by: Patrick Venture --- process.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'process.cpp') diff --git a/process.cpp b/process.cpp index 7e6410d..683bf77 100644 --- a/process.cpp +++ b/process.cpp @@ -136,9 +136,8 @@ ipmi_ret_t processBlobCommand(IpmiBlobHandler cmd, ManagerInterface* mgr, /* The command, whatever it was, replied, so let's set the CRC. */ crc->clear(); - replyCmdBuf[0] = 0x00; - replyCmdBuf[1] = 0x00; - crc->compute(replyCmdBuf, replyLength); + crc->compute(replyCmdBuf + sizeof(uint16_t), + replyLength - sizeof(uint16_t)); /* Copy the CRC into place. */ uint16_t crcValue = crc->get(); -- cgit v1.2.3