From 78ece1b2aab0547e94c492d86b695f08501fbd74 Mon Sep 17 00:00:00 2001 From: Jeffrey Walton Date: Tue, 3 Jul 2018 22:41:04 -0400 Subject: Fix temporary array size (GH #675) cryptest.sh and UBsan caught a "secblock.h:389:4: runtime error: load of value 206, which is not a valid value for type 'bool'". m_t[4] is accessed in UncheckedSetKey. The extra m_t[] element was inadvertently removed when ProcessAndXorBlock no longer used it. --- simeck.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'simeck.h') diff --git a/simeck.h b/simeck.h index d3feb452..40af2472 100644 --- a/simeck.h +++ b/simeck.h @@ -64,7 +64,7 @@ public: void UncheckedSetKey(const byte *userKey, unsigned int keyLength, const NameValuePairs ¶ms); FixedSizeSecBlock m_rk; - mutable FixedSizeSecBlock m_t; + mutable FixedSizeSecBlock m_t; }; /// \brief Provides implementation for encryption transformation @@ -113,7 +113,7 @@ public: void UncheckedSetKey(const byte *userKey, unsigned int keyLength, const NameValuePairs ¶ms); FixedSizeSecBlock m_rk; - mutable FixedSizeSecBlock m_t; + mutable FixedSizeSecBlock m_t; }; /// \brief Provides implementation for encryption transformation -- cgit v1.2.1