Derive the x86 assembly implementations of MD4, MD5, and Serpent from

the normal Botan base classes. This required making data members of
MD4, MD5, and Serpent protected rather than private, which is not very
good style IMO. On the other hand it allows for removing a bit of duplicated
code, and also has the nice effect that a pointer to a Serpent_IA32 can be
used right as a Serpent object, which makes sense anyway since they implement
the same algorithm. The C++ files in the *_ia32 modules are now simply hooks
between the virtual function call runtime and the assembly code.

2 files changed, 2 insertions, 31 deletions

diff --git a/src/hash/md5_ia32/md5_ia32.cpp b/src/hash/md5_ia32/md5_ia32.cpp
index b95eb0c56..7616292e3 100644
--- a/src/hash/md5_ia32/md5_ia32.cpp
+++ b/src/hash/md5_ia32/md5_ia32.cpp
@@ -23,26 +23,4 @@ void MD5_IA32::hash(const byte input[])
    botan_md5_ia32_compress(digest, input, M);
    }
 
-/*************************************************
-* Copy out the digest                            *
-*************************************************/
-void MD5_IA32::copy_out(byte output[])
-   {
-   for(u32bit j = 0; j != OUTPUT_LENGTH; j += 4)
-      store_le(digest[j/4], output + j);
-   }
-
-/*************************************************
-* Clear memory of sensitive data                 *
-*************************************************/
-void MD5_IA32::clear() throw()
-   {
-   MDx_HashFunction::clear();
-   M.clear();
-   digest[0] = 0x67452301;
-   digest[1] = 0xEFCDAB89;
-   digest[2] = 0x98BADCFE;
-   digest[3] = 0x10325476;
-   }
-
 }
diff --git a/src/hash/md5_ia32/md5_ia32.h b/src/hash/md5_ia32/md5_ia32.h
index cf5038c1c..7ff463799 100644
--- a/src/hash/md5_ia32/md5_ia32.h
+++ b/src/hash/md5_ia32/md5_ia32.h
@@ -6,26 +6,19 @@
 #ifndef BOTAN_MD5_IA32_H__
 #define BOTAN_MD5_IA32_H__
 
-#include <botan/mdx_hash.h>
+#include <botan/md5.h>
 
 namespace Botan {
 
 /*************************************************
 * MD5                                            *
 *************************************************/
-class BOTAN_DLL MD5_IA32 : public MDx_HashFunction
+class BOTAN_DLL MD5_IA32 : public MD5
    {
    public:
-      void clear() throw();
-      std::string name() const { return "MD5"; }
       HashFunction* clone() const { return new MD5_IA32; }
-      MD5_IA32() : MDx_HashFunction(16, 64, false, true) { clear(); }
    private:
       void hash(const byte[]);
-      void copy_out(byte[]);
-
-      SecureBuffer<u32bit, 16> M;
-      SecureBuffer<u32bit, 4> digest;
    };
 
 }