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 --- test/process_unittest.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'test/process_unittest.cpp') diff --git a/test/process_unittest.cpp b/test/process_unittest.cpp index 4f8c9a6..a908ee6 100644 --- a/test/process_unittest.cpp +++ b/test/process_unittest.cpp @@ -294,7 +294,7 @@ TEST(ProcessBlobCommandTest, CommandReturnsOkWithValidPayloadLength) dataLen = sizeof(request); EXPECT_CALL(crc, clear()); - EXPECT_CALL(crc, compute(_, payloadLen)); + EXPECT_CALL(crc, compute(_, payloadLen - sizeof(uint16_t))); EXPECT_CALL(crc, get()).WillOnce(Return(0x3412)); EXPECT_EQ(IPMI_CC_OK, -- cgit v1.2.3