diff options
author | Edwin Mons <edwin.mons@isode.com> | 2018-10-29 14:31:18 (GMT) |
---|---|---|
committer | Edwin Mons <edwin.mons@isode.com> | 2018-11-14 11:02:05 (GMT) |
commit | c0615a472f8d23ce449fd59bbb1cdf7071082a43 (patch) | |
tree | b2815062ce279824446febd403c0029f04c29f33 /Swiften/Crypto/OpenSSLCryptoProvider.cpp | |
parent | 30639ed8dbb419890eab5a0b46d3a78896c7f22d (diff) | |
download | swift-c0615a472f8d23ce449fd59bbb1cdf7071082a43.zip swift-c0615a472f8d23ce449fd59bbb1cdf7071082a43.tar.bz2 |
Catch bad_numeric_casts in crypto code
Exceptions thrown by boost::numeric_cast are now caught and an assert
explicitly triggered.
Test-Information:
Unit tests pass on macOS 10.13
Change-Id: I9a1cbe5ae2765e4275bf35473a871ef8468fd729
Diffstat (limited to 'Swiften/Crypto/OpenSSLCryptoProvider.cpp')
-rw-r--r-- | Swiften/Crypto/OpenSSLCryptoProvider.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/Swiften/Crypto/OpenSSLCryptoProvider.cpp b/Swiften/Crypto/OpenSSLCryptoProvider.cpp index e8c1c73..73f46a6 100644 --- a/Swiften/Crypto/OpenSSLCryptoProvider.cpp +++ b/Swiften/Crypto/OpenSSLCryptoProvider.cpp @@ -107,7 +107,12 @@ namespace { ByteArray getHMACSHA1Internal(const T& key, const ByteArray& data) { unsigned int len = SHA_DIGEST_LENGTH; std::vector<unsigned char> result(len); - HMAC(EVP_sha1(), vecptr(key), boost::numeric_cast<int>(key.size()), vecptr(data), data.size(), vecptr(result), &len); + try { + HMAC(EVP_sha1(), vecptr(key), boost::numeric_cast<int>(key.size()), vecptr(data), boost::numeric_cast<int>(data.size()), vecptr(result), &len); + } + catch (const boost::numeric::bad_numeric_cast&) { + assert(false); + } return result; } } |