diff options
| author | Jack Lloyd <[email protected]> | 2017-10-26 20:31:30 -0400 |
|---|---|---|
| committer | Jack Lloyd <[email protected]> | 2017-10-26 22:26:15 -0400 |
| commit | e6d45052efedfe49e99adb6318aaf56e0a9e8d7b (patch) | |
| tree | c6c3ccd3cff3d04285940bf1d518c809e0653947 /src/lib/block/blowfish | |
| parent | 315b002ecf00f6b6bb0f0d5200d1f39a83527e8f (diff) | |
Add checks that keyed algorithms are actually keyed before use
Previously calling update or encrypt without calling set_key first
would result in invalid outputs or else crashing.
Diffstat (limited to 'src/lib/block/blowfish')
| -rw-r--r-- | src/lib/block/blowfish/blowfish.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/lib/block/blowfish/blowfish.cpp b/src/lib/block/blowfish/blowfish.cpp index 68d73cafd..c2634bba4 100644 --- a/src/lib/block/blowfish/blowfish.cpp +++ b/src/lib/block/blowfish/blowfish.cpp @@ -197,6 +197,8 @@ const uint32_t S_INIT[1024] = { */ void Blowfish::encrypt_n(const uint8_t in[], uint8_t out[], size_t blocks) const { + verify_key_set(m_S.empty() == false); + const uint32_t* S1 = &m_S[0]; const uint32_t* S2 = &m_S[256]; const uint32_t* S3 = &m_S[512]; @@ -229,6 +231,8 @@ void Blowfish::encrypt_n(const uint8_t in[], uint8_t out[], size_t blocks) const */ void Blowfish::decrypt_n(const uint8_t in[], uint8_t out[], size_t blocks) const { + verify_key_set(m_S.empty() == false); + const uint32_t* S1 = &m_S[0]; const uint32_t* S2 = &m_S[256]; const uint32_t* S3 = &m_S[512]; |