From 560b5642eeeee9135e8995c04f99613387a7bfa8 Mon Sep 17 00:00:00 2001 From: Edwin Mons Date: Mon, 19 Nov 2018 11:59:53 +0100 Subject: Fix incorrect cast in OpenSSL provider A numeric cast was added to the call to HMAC that doesn't belong there, this commit reverts that. Test-Information: Unit tests pass on macOS 10.14 and Debian 9 Change-Id: I8deb568f7bff4d401fc6dd6de21edf7f061ebd59 diff --git a/Swiften/Crypto/OpenSSLCryptoProvider.cpp b/Swiften/Crypto/OpenSSLCryptoProvider.cpp index 73f46a6..5245bd8 100644 --- a/Swiften/Crypto/OpenSSLCryptoProvider.cpp +++ b/Swiften/Crypto/OpenSSLCryptoProvider.cpp @@ -108,7 +108,7 @@ namespace { unsigned int len = SHA_DIGEST_LENGTH; std::vector result(len); try { - HMAC(EVP_sha1(), vecptr(key), boost::numeric_cast(key.size()), vecptr(data), boost::numeric_cast(data.size()), vecptr(result), &len); + HMAC(EVP_sha1(), vecptr(key), boost::numeric_cast(key.size()), vecptr(data), data.size(), vecptr(result), &len); } catch (const boost::numeric::bad_numeric_cast&) { assert(false); -- cgit v0.10.2-6-g49f6