diff --git a/ChangeLog b/ChangeLog index 678e21be4c..a652f2c97a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,4 +1,8 @@ -Tue Jul 15 23:59:27 2014 Jared Jennings +Tue Jul 15 23:59:39 2014 Jared Jennings + + * ext/digest/md5/md5ossl.c: use OpenSSL EVP API instead of MD5 API + to perform MD5 hashes using OpenSSL in ext/digest. + [ruby-core:61614] [Bug #9659] * ext/digest: make built-in digest function implementations indicate success or failure of init and final functions. diff --git a/ext/digest/md5/md5ossl.c b/ext/digest/md5/md5ossl.c index d94ae2cd2f..d08d2db05e 100644 --- a/ext/digest/md5/md5ossl.c +++ b/ext/digest/md5/md5ossl.c @@ -2,8 +2,15 @@ #include "md5ossl.h" -void -MD5_Finish(MD5_CTX *pctx, unsigned char *digest) +int +rb_digest_md5osslevp_Init(EVP_MD_CTX *pctx) { - MD5_Final(digest, pctx); + return EVP_DigestInit_ex(pctx, EVP_md5(), NULL); +} + +int +rb_digest_md5osslevp_Finish(EVP_MD_CTX *pctx, unsigned char *digest) +{ + /* if EVP_DigestFinal_ex fails, we ignore that */ + return EVP_DigestFinal_ex(pctx, digest, NULL); } diff --git a/ext/digest/md5/md5ossl.h b/ext/digest/md5/md5ossl.h index 1680c4f5c9..f1028704a0 100644 --- a/ext/digest/md5/md5ossl.h +++ b/ext/digest/md5/md5ossl.h @@ -4,10 +4,22 @@ #define MD5OSSL_H_INCLUDED #include -#include +#include -#define MD5_BLOCK_LENGTH MD5_CBLOCK +#define MD5_Init rb_digest_md5osslevp_Init +#define MD5_Update EVP_DigestUpdate +#define MD5_Finish rb_digest_md5osslevp_Finish +#define MD5_CTX EVP_MD_CTX -void MD5_Finish(MD5_CTX *pctx, unsigned char *digest); +/* We should use EVP_MD_size(3) and EVP_MD_block_size(3), but the + advantage of these is that they are flexible across digest + algorithms and we are fixing the digest algorithm here; and these + numbers must be constants because the rb_digest_metadata_t + structure is declared const. Simplest way is to write literals. */ +#define MD5_BLOCK_LENGTH 64 +#define MD5_DIGEST_LENGTH 16 + +int MD5_Init(MD5_CTX *pctx); +int MD5_Finish(MD5_CTX *pctx, unsigned char *digest); #endif