Back-off on Hash asserts (GH #652)

The asserts were a little aggressive and caused very noisy Debug runs. The library itself was one of the biggest offenders.

1 files changed, 75 insertions, 71 deletions

diff --git a/md5.cpp b/md5.cpp
index 3bdc7379..99a0e34b 100644
--- a/md5.cpp
+++ b/md5.cpp
@@ -2,12 +2,13 @@
 // any modifications are placed in the public domain

 

 #include "pch.h"

+

 #define CRYPTOPP_ENABLE_NAMESPACE_WEAK 1

 #include "md5.h"

 #include "misc.h"

 

 NAMESPACE_BEGIN(CryptoPP)

-namespace Weak1 {

+NAMESPACE_BEGIN(Weak1)

 

 #if defined(CRYPTOPP_DEBUG) && !defined(CRYPTOPP_DOXYGEN_PROCESSING)

 void MD5_TestInstantiations()

@@ -24,8 +25,11 @@ void MD5::InitState(HashWordType *state)
 	state[3] = 0x10325476L;

 }

 

-void MD5::Transform (word32 *digest, const word32 *in)

+void MD5::Transform (word32 *digest, const word32 *data)

 {

+    CRYPTOPP_ASSERT(digest);

+    CRYPTOPP_ASSERT(data);

+

 // #define F1(x, y, z) (x & y | ~x & z)

 #define F1(x, y, z) (z ^ (x & (y ^ z)))

 #define F2(x, y, z) F1(z, x, y)

@@ -42,73 +46,73 @@ void MD5::Transform (word32 *digest, const word32 *in)
 	c=digest[2];

 	d=digest[3];

 

-    MD5STEP(F1, a, b, c, d, in[0] + 0xd76aa478, 7);

-    MD5STEP(F1, d, a, b, c, in[1] + 0xe8c7b756, 12);

-    MD5STEP(F1, c, d, a, b, in[2] + 0x242070db, 17);

-    MD5STEP(F1, b, c, d, a, in[3] + 0xc1bdceee, 22);

-    MD5STEP(F1, a, b, c, d, in[4] + 0xf57c0faf, 7);

-    MD5STEP(F1, d, a, b, c, in[5] + 0x4787c62a, 12);

-    MD5STEP(F1, c, d, a, b, in[6] + 0xa8304613, 17);

-    MD5STEP(F1, b, c, d, a, in[7] + 0xfd469501, 22);

-    MD5STEP(F1, a, b, c, d, in[8] + 0x698098d8, 7);

-    MD5STEP(F1, d, a, b, c, in[9] + 0x8b44f7af, 12);

-    MD5STEP(F1, c, d, a, b, in[10] + 0xffff5bb1, 17);

-    MD5STEP(F1, b, c, d, a, in[11] + 0x895cd7be, 22);

-    MD5STEP(F1, a, b, c, d, in[12] + 0x6b901122, 7);

-    MD5STEP(F1, d, a, b, c, in[13] + 0xfd987193, 12);

-    MD5STEP(F1, c, d, a, b, in[14] + 0xa679438e, 17);

-    MD5STEP(F1, b, c, d, a, in[15] + 0x49b40821, 22);

-

-    MD5STEP(F2, a, b, c, d, in[1] + 0xf61e2562, 5);

-    MD5STEP(F2, d, a, b, c, in[6] + 0xc040b340, 9);

-    MD5STEP(F2, c, d, a, b, in[11] + 0x265e5a51, 14);

-    MD5STEP(F2, b, c, d, a, in[0] + 0xe9b6c7aa, 20);

-    MD5STEP(F2, a, b, c, d, in[5] + 0xd62f105d, 5);

-    MD5STEP(F2, d, a, b, c, in[10] + 0x02441453, 9);

-    MD5STEP(F2, c, d, a, b, in[15] + 0xd8a1e681, 14);

-    MD5STEP(F2, b, c, d, a, in[4] + 0xe7d3fbc8, 20);

-    MD5STEP(F2, a, b, c, d, in[9] + 0x21e1cde6, 5);

-    MD5STEP(F2, d, a, b, c, in[14] + 0xc33707d6, 9);

-    MD5STEP(F2, c, d, a, b, in[3] + 0xf4d50d87, 14);

-    MD5STEP(F2, b, c, d, a, in[8] + 0x455a14ed, 20);

-    MD5STEP(F2, a, b, c, d, in[13] + 0xa9e3e905, 5);

-    MD5STEP(F2, d, a, b, c, in[2] + 0xfcefa3f8, 9);

-    MD5STEP(F2, c, d, a, b, in[7] + 0x676f02d9, 14);

-    MD5STEP(F2, b, c, d, a, in[12] + 0x8d2a4c8a, 20);

-

-    MD5STEP(F3, a, b, c, d, in[5] + 0xfffa3942, 4);

-    MD5STEP(F3, d, a, b, c, in[8] + 0x8771f681, 11);

-    MD5STEP(F3, c, d, a, b, in[11] + 0x6d9d6122, 16);

-    MD5STEP(F3, b, c, d, a, in[14] + 0xfde5380c, 23);

-    MD5STEP(F3, a, b, c, d, in[1] + 0xa4beea44, 4);

-    MD5STEP(F3, d, a, b, c, in[4] + 0x4bdecfa9, 11);

-    MD5STEP(F3, c, d, a, b, in[7] + 0xf6bb4b60, 16);

-    MD5STEP(F3, b, c, d, a, in[10] + 0xbebfbc70, 23);

-    MD5STEP(F3, a, b, c, d, in[13] + 0x289b7ec6, 4);

-    MD5STEP(F3, d, a, b, c, in[0] + 0xeaa127fa, 11);

-    MD5STEP(F3, c, d, a, b, in[3] + 0xd4ef3085, 16);

-    MD5STEP(F3, b, c, d, a, in[6] + 0x04881d05, 23);

-    MD5STEP(F3, a, b, c, d, in[9] + 0xd9d4d039, 4);

-    MD5STEP(F3, d, a, b, c, in[12] + 0xe6db99e5, 11);

-    MD5STEP(F3, c, d, a, b, in[15] + 0x1fa27cf8, 16);

-    MD5STEP(F3, b, c, d, a, in[2] + 0xc4ac5665, 23);

-

-    MD5STEP(F4, a, b, c, d, in[0] + 0xf4292244, 6);

-    MD5STEP(F4, d, a, b, c, in[7] + 0x432aff97, 10);

-    MD5STEP(F4, c, d, a, b, in[14] + 0xab9423a7, 15);

-    MD5STEP(F4, b, c, d, a, in[5] + 0xfc93a039, 21);

-    MD5STEP(F4, a, b, c, d, in[12] + 0x655b59c3, 6);

-    MD5STEP(F4, d, a, b, c, in[3] + 0x8f0ccc92, 10);

-    MD5STEP(F4, c, d, a, b, in[10] + 0xffeff47d, 15);

-    MD5STEP(F4, b, c, d, a, in[1] + 0x85845dd1, 21);

-    MD5STEP(F4, a, b, c, d, in[8] + 0x6fa87e4f, 6);

-    MD5STEP(F4, d, a, b, c, in[15] + 0xfe2ce6e0, 10);

-    MD5STEP(F4, c, d, a, b, in[6] + 0xa3014314, 15);

-    MD5STEP(F4, b, c, d, a, in[13] + 0x4e0811a1, 21);

-    MD5STEP(F4, a, b, c, d, in[4] + 0xf7537e82, 6);

-    MD5STEP(F4, d, a, b, c, in[11] + 0xbd3af235, 10);

-    MD5STEP(F4, c, d, a, b, in[2] + 0x2ad7d2bb, 15);

-    MD5STEP(F4, b, c, d, a, in[9] + 0xeb86d391, 21);

+    MD5STEP(F1, a, b, c, d, data[0] + 0xd76aa478, 7);

+    MD5STEP(F1, d, a, b, c, data[1] + 0xe8c7b756, 12);

+    MD5STEP(F1, c, d, a, b, data[2] + 0x242070db, 17);

+    MD5STEP(F1, b, c, d, a, data[3] + 0xc1bdceee, 22);

+    MD5STEP(F1, a, b, c, d, data[4] + 0xf57c0faf, 7);

+    MD5STEP(F1, d, a, b, c, data[5] + 0x4787c62a, 12);

+    MD5STEP(F1, c, d, a, b, data[6] + 0xa8304613, 17);

+    MD5STEP(F1, b, c, d, a, data[7] + 0xfd469501, 22);

+    MD5STEP(F1, a, b, c, d, data[8] + 0x698098d8, 7);

+    MD5STEP(F1, d, a, b, c, data[9] + 0x8b44f7af, 12);

+    MD5STEP(F1, c, d, a, b, data[10] + 0xffff5bb1, 17);

+    MD5STEP(F1, b, c, d, a, data[11] + 0x895cd7be, 22);

+    MD5STEP(F1, a, b, c, d, data[12] + 0x6b901122, 7);

+    MD5STEP(F1, d, a, b, c, data[13] + 0xfd987193, 12);

+    MD5STEP(F1, c, d, a, b, data[14] + 0xa679438e, 17);

+    MD5STEP(F1, b, c, d, a, data[15] + 0x49b40821, 22);

+

+    MD5STEP(F2, a, b, c, d, data[1] + 0xf61e2562, 5);

+    MD5STEP(F2, d, a, b, c, data[6] + 0xc040b340, 9);

+    MD5STEP(F2, c, d, a, b, data[11] + 0x265e5a51, 14);

+    MD5STEP(F2, b, c, d, a, data[0] + 0xe9b6c7aa, 20);

+    MD5STEP(F2, a, b, c, d, data[5] + 0xd62f105d, 5);

+    MD5STEP(F2, d, a, b, c, data[10] + 0x02441453, 9);

+    MD5STEP(F2, c, d, a, b, data[15] + 0xd8a1e681, 14);

+    MD5STEP(F2, b, c, d, a, data[4] + 0xe7d3fbc8, 20);

+    MD5STEP(F2, a, b, c, d, data[9] + 0x21e1cde6, 5);

+    MD5STEP(F2, d, a, b, c, data[14] + 0xc33707d6, 9);

+    MD5STEP(F2, c, d, a, b, data[3] + 0xf4d50d87, 14);

+    MD5STEP(F2, b, c, d, a, data[8] + 0x455a14ed, 20);

+    MD5STEP(F2, a, b, c, d, data[13] + 0xa9e3e905, 5);

+    MD5STEP(F2, d, a, b, c, data[2] + 0xfcefa3f8, 9);

+    MD5STEP(F2, c, d, a, b, data[7] + 0x676f02d9, 14);

+    MD5STEP(F2, b, c, d, a, data[12] + 0x8d2a4c8a, 20);

+

+    MD5STEP(F3, a, b, c, d, data[5] + 0xfffa3942, 4);

+    MD5STEP(F3, d, a, b, c, data[8] + 0x8771f681, 11);

+    MD5STEP(F3, c, d, a, b, data[11] + 0x6d9d6122, 16);

+    MD5STEP(F3, b, c, d, a, data[14] + 0xfde5380c, 23);

+    MD5STEP(F3, a, b, c, d, data[1] + 0xa4beea44, 4);

+    MD5STEP(F3, d, a, b, c, data[4] + 0x4bdecfa9, 11);

+    MD5STEP(F3, c, d, a, b, data[7] + 0xf6bb4b60, 16);

+    MD5STEP(F3, b, c, d, a, data[10] + 0xbebfbc70, 23);

+    MD5STEP(F3, a, b, c, d, data[13] + 0x289b7ec6, 4);

+    MD5STEP(F3, d, a, b, c, data[0] + 0xeaa127fa, 11);

+    MD5STEP(F3, c, d, a, b, data[3] + 0xd4ef3085, 16);

+    MD5STEP(F3, b, c, d, a, data[6] + 0x04881d05, 23);

+    MD5STEP(F3, a, b, c, d, data[9] + 0xd9d4d039, 4);

+    MD5STEP(F3, d, a, b, c, data[12] + 0xe6db99e5, 11);

+    MD5STEP(F3, c, d, a, b, data[15] + 0x1fa27cf8, 16);

+    MD5STEP(F3, b, c, d, a, data[2] + 0xc4ac5665, 23);

+

+    MD5STEP(F4, a, b, c, d, data[0] + 0xf4292244, 6);

+    MD5STEP(F4, d, a, b, c, data[7] + 0x432aff97, 10);

+    MD5STEP(F4, c, d, a, b, data[14] + 0xab9423a7, 15);

+    MD5STEP(F4, b, c, d, a, data[5] + 0xfc93a039, 21);

+    MD5STEP(F4, a, b, c, d, data[12] + 0x655b59c3, 6);

+    MD5STEP(F4, d, a, b, c, data[3] + 0x8f0ccc92, 10);

+    MD5STEP(F4, c, d, a, b, data[10] + 0xffeff47d, 15);

+    MD5STEP(F4, b, c, d, a, data[1] + 0x85845dd1, 21);

+    MD5STEP(F4, a, b, c, d, data[8] + 0x6fa87e4f, 6);

+    MD5STEP(F4, d, a, b, c, data[15] + 0xfe2ce6e0, 10);

+    MD5STEP(F4, c, d, a, b, data[6] + 0xa3014314, 15);

+    MD5STEP(F4, b, c, d, a, data[13] + 0x4e0811a1, 21);

+    MD5STEP(F4, a, b, c, d, data[4] + 0xf7537e82, 6);

+    MD5STEP(F4, d, a, b, c, data[11] + 0xbd3af235, 10);

+    MD5STEP(F4, c, d, a, b, data[2] + 0x2ad7d2bb, 15);

+    MD5STEP(F4, b, c, d, a, data[9] + 0xeb86d391, 21);

 

 	digest[0]+=a;

 	digest[1]+=b;

@@ -116,5 +120,5 @@ void MD5::Transform (word32 *digest, const word32 *in)
 	digest[3]+=d;

 }

 

-}

-NAMESPACE_END

+NAMESPACE_END  // Weak

+NAMESPACE_END  // CryptoPP