While working on something else, I stumbled over this. Obviously no one\ 1
authorBernd Petrovitsch <bernd@firmix.at>
Fri, 2 Nov 2007 15:51:46 +0000 (15:51 +0000)
committerBernd Petrovitsch <bernd@firmix.at>
Fri, 2 Nov 2007 15:51:46 +0000 (15:51 +0000)
ever used the "secure" plugin on 64bit hardware:
- fixed lib/secure/src/md5.h: This was broken as it had
  "typedef unsigned long int UINT4;". "unsigned long int" is 8 bytes on
  x86_64.
  We are using now the standardized types from <inttypes.h>.
- fixed warnings and improved lib/secure/src/md5.c:
  * we are using memcpy() and memset() instead of the open-coded loops as
    suggested
  * const'ified
  * added function prototypes for static functions
  * moved static functions to the top so that their declaration is before
    their use to allow gcc to inline if only used once.
  It remains ugly - God knows why there are that so many useless
  type-casts.

CHANGELOG
lib/secure/src/md5.c
lib/secure/src/md5.h

index cb40910..bd16059 100644 (file)
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,5 +1,5 @@
 This file states changes as of version 0.2.4:
-$Id: CHANGELOG,v 1.105 2007/11/02 10:57:30 bernd67 Exp $
+$Id: CHANGELOG,v 1.106 2007/11/02 15:51:46 bernd67 Exp $
 
 0.5.5 ---------------------------------------------------------------------
 
@@ -20,6 +20,18 @@ PATCHES and CLEANUPS by Bernd Petrovitsch <bernd@firmix.at>
     dependency generation and beautify (read: "$(strip)") $(CPPFLAGS).
   * OpenBSD needs an explicit "#include <sys/types.h>" before
     "#include <netinet/in.h>".
+- fixed lib/secure/src/md5.h: This was broken as it had
+  "typedef unsigned long int UINT4;". "unsigned long int" is 8 bytes on x86_64.
+  We are using now the standardized types from <inttypes.h>.
+- fixed warnings and improved lib/secure/src/md5.c:
+  * we are using memcpy() and memset() instead of the open-coded loops as
+    suggested 
+  * const'ified
+  * added function prototypes for static functions
+  * moved static functions to the top so that their declaration is before
+    their use to allow gcc to inline if only used once.
+  It remains ugly - God knows why there are that so many useless
+  type-casts.
 
 0.5.4 ---------------------------------------------------------------------
 
index 9023e0e..a58c3ac 100644 (file)
@@ -25,6 +25,8 @@
 
 #include "md5.h"
 
+#include <string.h>
+
 /* Constants for MD5Transform routine.
  */
 #define S11 7
 #define S43 15
 #define S44 21
 
-static void MD5Transform(UINT4 [4], const unsigned char [64]);
-static void Encode(unsigned char *, UINT4 *, unsigned int);
-static void Decode(UINT4 *, const unsigned char *, const unsigned int);
-static void MD5_memcpy(POINTER, POINTER, unsigned int);
-static void MD5_memset(POINTER, int, unsigned int);
-
-static unsigned char PADDING[64] = {
+static const unsigned char PADDING[64] = {
   0x80, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0
@@ -91,86 +87,57 @@ static unsigned char PADDING[64] = {
     (a) += (b); \
   }
 
-/* MD5 initialization. Begins an MD5 operation, writing a new context.
+#if 0
+/* Note: Replace "for loop" with standard memcpy if possible.
  */
-void MD5Init (MD5_CTX *context)
+static void MD5_memcpy (POINTER output, POINTER input, unsigned int len)
 {
-  context->count[0] = context->count[1] = 0;
-
-  /* Load magic initialization constants.
-   */
-  context->state[0] = 0x67452301;
-  context->state[1] = 0xefcdab89;
-  context->state[2] = 0x98badcfe;
-  context->state[3] = 0x10325476;
+  unsigned int i;
+  
+  for (i = 0; i < len; i++)
+    output[i] = input[i];
 }
 
-/* MD5 block update operation. Continues an MD5 message-digest
-     operation, processing another message block, and updating the
-     context.
+/* Note: Replace "for loop" with standard memset if possible.
  */
-void MD5Update (MD5_CTX *context, const unsigned char *input, const unsigned int inputLen)
+static void MD5_memset (POINTER output, int value, unsigned int len)
 {
-  unsigned int i, index, partLen;
-
-  /* Compute number of bytes mod 64 */
-  index = (unsigned int)((context->count[0] >> 3) & 0x3F);
-
-  /* Update number of bits */
-  if ((context->count[0] += ((UINT4)inputLen << 3))
-      < ((UINT4)inputLen << 3))
-    context->count[1]++;
-  context->count[1] += ((UINT4)inputLen >> 29);
-  
-  partLen = 64 - index;
-  
-  /* Transform as many times as possible.
-   */
-  if (inputLen >= partLen) {
-    MD5_memcpy
-      ((POINTER)&context->buffer[index], (POINTER)input, partLen);
-    MD5Transform (context->state, context->buffer);
-  
-    for (i = partLen; i + 63 < inputLen; i += 64)
-      MD5Transform (context->state, &input[i]);
-    
-    index = 0;
-  }
-  else
-    i = 0;
+  unsigned int i;
   
-  /* Buffer remaining input */
-  MD5_memcpy 
-    ((POINTER)&context->buffer[index], (POINTER)&input[i],
-     inputLen-i);
+  for (i = 0; i < len; i++)
+    ((char *)output)[i] = (char)value;
 }
+#else
+#define MD5_memcpy(dst, src, len) memcpy((dst), (src), (len)) 
+#define MD5_memset(dst, val, len) memset((dst), (val), (len)) 
+#endif
 
-/* MD5 finalization. Ends an MD5 message-digest operation, writing the
-     the message digest and zeroizing the context.
+
+/* Encodes input (UINT4) into output (unsigned char). Assumes len is
+     a multiple of 4.
  */
-void MD5Final (unsigned char digest[16], MD5_CTX *context)
+static void Encode (unsigned char *output, UINT4 *input, unsigned int len)
 {
-  unsigned char bits[8];
-  unsigned int index, padLen;
+  unsigned int i, j;
 
-  /* Save number of bits */
-  Encode (bits, context->count, 8);
+  for (i = 0, j = 0; j < len; i++, j += 4) {
+    output[j] = (unsigned char)(input[i] & 0xff);
+    output[j+1] = (unsigned char)((input[i] >> 8) & 0xff);
+    output[j+2] = (unsigned char)((input[i] >> 16) & 0xff);
+    output[j+3] = (unsigned char)((input[i] >> 24) & 0xff);
+  }
+}
 
-  /* Pad out to 56 mod 64.
-   */
-  index = (unsigned int)((context->count[0] >> 3) & 0x3f);
-  padLen = (index < 56) ? (56 - index) : (120 - index);
-  MD5Update (context, PADDING, padLen);
-  
-  /* Append length (before padding) */
-  MD5Update (context, bits, 8);
+/* Decodes input (unsigned char) into output (UINT4). Assumes len is
+     a multiple of 4.
+ */
+static void Decode (UINT4 *output, const unsigned char *input, unsigned int len)
+{
+  unsigned int i, j;
 
-  /* Store state in digest */
-  Encode (digest, context->state, 16);
-  
-  /* Zeroize sensitive information.
-   */
-  MD5_memset ((POINTER)context, 0, sizeof (*context));
+  for (i = 0, j = 0; j < len; i++, j += 4)
+    output[i] = ((UINT4)input[j]) | (((UINT4)input[j+1]) << 8) |
+      (((UINT4)input[j+2]) << 16) | (((UINT4)input[j+3]) << 24);
 }
 
 /* MD5 basic transformation. Transforms state based on block.
@@ -263,49 +230,85 @@ static void MD5Transform (UINT4 state[4], const unsigned char block[64])
   MD5_memset ((POINTER)x, 0, sizeof (x));
 }
 
-/* Encodes input (UINT4) into output (unsigned char). Assumes len is
-     a multiple of 4.
+
+/* MD5 initialization. Begins an MD5 operation, writing a new context.
  */
-static void Encode (unsigned char *output, UINT4 *input, unsigned int len)
+void MD5Init (MD5_CTX *context)
 {
-  unsigned int i, j;
+  context->count[0] = context->count[1] = 0;
 
-  for (i = 0, j = 0; j < len; i++, j += 4) {
-    output[j] = (unsigned char)(input[i] & 0xff);
-    output[j+1] = (unsigned char)((input[i] >> 8) & 0xff);
-    output[j+2] = (unsigned char)((input[i] >> 16) & 0xff);
-    output[j+3] = (unsigned char)((input[i] >> 24) & 0xff);
-  }
+  /* Load magic initialization constants.
+   */
+  context->state[0] = 0x67452301;
+  context->state[1] = 0xefcdab89;
+  context->state[2] = 0x98badcfe;
+  context->state[3] = 0x10325476;
 }
 
-/* Decodes input (unsigned char) into output (UINT4). Assumes len is
-     a multiple of 4.
+/* MD5 block update operation. Continues an MD5 message-digest
+     operation, processing another message block, and updating the
+     context.
  */
-static void Decode (UINT4 *output, const unsigned char *input, const unsigned int len)
+void MD5Update (MD5_CTX *context, const unsigned char *input, const unsigned int inputLen)
 {
-  unsigned int i, j;
+  unsigned int i, index, partLen;
 
-  for (i = 0, j = 0; j < len; i++, j += 4)
-    output[i] = ((UINT4)input[j]) | (((UINT4)input[j+1]) << 8) |
-      (((UINT4)input[j+2]) << 16) | (((UINT4)input[j+3]) << 24);
-}
+  /* Compute number of bytes mod 64 */
+  index = (unsigned int)((context->count[0] >> 3) & 0x3F);
 
-/* Note: Replace "for loop" with standard memcpy if possible.
- */
-static void MD5_memcpy (POINTER output, POINTER input, unsigned int len)
-{
-  unsigned int i;
+  /* Update number of bits */
+  if ((context->count[0] += ((UINT4)inputLen << 3))
+      < ((UINT4)inputLen << 3))
+    context->count[1]++;
+  context->count[1] += ((UINT4)inputLen >> 29);
   
-  for (i = 0; i < len; i++)
-    output[i] = input[i];
+  partLen = 64 - index;
+  
+  /* Transform as many times as possible.
+   */
+  if (inputLen >= partLen) {
+    MD5_memcpy
+      ((POINTER)&context->buffer[index], (POINTER)input, partLen);
+    MD5Transform (context->state, context->buffer);
+  
+    for (i = partLen; i + 63 < inputLen; i += 64)
+      MD5Transform (context->state, &input[i]);
+    
+    index = 0;
+  }
+  else
+    i = 0;
+  
+  /* Buffer remaining input */
+  MD5_memcpy 
+    ((POINTER)&context->buffer[index], (POINTER)&input[i],
+     inputLen-i);
 }
 
-/* Note: Replace "for loop" with standard memset if possible.
+/* MD5 finalization. Ends an MD5 message-digest operation, writing the
+     the message digest and zeroizing the context.
  */
-static void MD5_memset (POINTER output, int value, unsigned int len)
+void MD5Final (unsigned char digest[16], MD5_CTX *context)
 {
-  unsigned int i;
+  unsigned char bits[8];
+  unsigned int index, padLen;
+
+  /* Save number of bits */
+  Encode (bits, context->count, 8);
+
+  /* Pad out to 56 mod 64.
+   */
+  index = (unsigned int)((context->count[0] >> 3) & 0x3f);
+  padLen = (index < 56) ? (56 - index) : (120 - index);
+  MD5Update (context, PADDING, padLen);
   
-  for (i = 0; i < len; i++)
-    ((char *)output)[i] = (char)value;
+  /* Append length (before padding) */
+  MD5Update (context, bits, 8);
+
+  /* Store state in digest */
+  Encode (digest, context->state, 16);
+  
+  /* Zeroize sensitive information.
+   */
+  MD5_memset ((POINTER)context, 0, sizeof (*context));
 }
index 4404678..6880eec 100644 (file)
 #ifndef _MD5_H_
 #define _MD5_H_
 
+#include <inttypes.h>
+
 /* POINTER defines a generic pointer type */
-typedef unsigned char *POINTER;
+typedef uint8_t *POINTER;
 /* UINT2 defines a two byte word */
-typedef unsigned short int UINT2;
+typedef uint16_t UINT2;
 /* UINT4 defines a four byte word */
-typedef unsigned long int UINT4;
+typedef uint32_t UINT4;
 
 /* MD5 context. */
 typedef struct {