From 04bb47f57f261e4e7a2280924d567683b9815c23 Mon Sep 17 00:00:00 2001 From: Lioncash Date: Thu, 6 Aug 2020 02:34:08 -0400 Subject: [PATCH] partition_data_manager: Eliminate magic value We can use sizeof to make it obvious at the call site where the value is coming from. --- src/core/crypto/partition_data_manager.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/core/crypto/partition_data_manager.cpp b/src/core/crypto/partition_data_manager.cpp index b31a81560..c5c073e70 100644 --- a/src/core/crypto/partition_data_manager.cpp +++ b/src/core/crypto/partition_data_manager.cpp @@ -349,8 +349,8 @@ static bool AttemptDecrypt(const std::array& key, Package2Header& header Package2Header temp = header; AESCipher cipher(key, Mode::CTR); cipher.SetIV(header.header_ctr); - cipher.Transcode(&temp.header_ctr, sizeof(Package2Header) - 0x100, &temp.header_ctr, - Op::Decrypt); + cipher.Transcode(&temp.header_ctr, sizeof(Package2Header) - sizeof(Package2Header::signature), + &temp.header_ctr, Op::Decrypt); if (temp.magic == Common::MakeMagic('P', 'K', '2', '1')) { header = temp; return true;