From 0838e0b9265a9ec55d1a11fa3f32616ec7dbf74c Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Fri, 14 Dec 2001 06:33:57 +0000 Subject: [PATCH] Update. 2001-12-13 Ulrich Drepper * crypt/md5.c (md5_process_bytes): Correct handling of alignment. Patch by Eric Sharkey . * crypt/md5test.c (main): Add test for multiple calls to __md5_process_bytes to itererate over input string. --- ChangeLog | 8 ++++++++ crypt/md5.c | 34 ++++++++++++++++++++++++---------- crypt/md5test.c | 9 +++++++++ 3 files changed, 41 insertions(+), 10 deletions(-) diff --git a/ChangeLog b/ChangeLog index 3428c7bca0..9541804292 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2001-12-13 Ulrich Drepper + + * crypt/md5.c (md5_process_bytes): Correct handling of alignment. + Patch by Eric Sharkey . + + * crypt/md5test.c (main): Add test for multiple calls to + __md5_process_bytes to itererate over input string. + 2001-12-12 Ulrich Drepper * nis/nis_findserv.c (__nis_findfastest_with_timeout): Mostly the code diff --git a/crypt/md5.c b/crypt/md5.c index d6b801307e..04bce5ab37 100644 --- a/crypt/md5.c +++ b/crypt/md5.c @@ -1,6 +1,6 @@ /* Functions to compute MD5 message digest of files or memory blocks. according to the definition of MD5 in RFC 1321 from April 1992. - Copyright (C) 1995, 1996, 1997, 1999, 2000 Free Software Foundation, Inc. + Copyright (C) 1995,1996,1997,1999,2000,2001 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -217,8 +217,6 @@ md5_process_bytes (buffer, len, ctx) size_t len; struct md5_ctx *ctx; { - //const void aligned_buffer = buffer; - /* When we already have some bits in our internal buffer concatenate both inputs first. */ if (ctx->buflen != 0) @@ -226,9 +224,6 @@ md5_process_bytes (buffer, len, ctx) size_t left_over = ctx->buflen; size_t add = 128 - left_over > len ? len : 128 - left_over; - /* Only put full words in the buffer. */ - add -= add % __alignof__ (md5_uint32); - memcpy (&ctx->buffer[left_over], buffer, add); ctx->buflen += add; @@ -247,11 +242,30 @@ md5_process_bytes (buffer, len, ctx) } /* Process available complete blocks. */ - if (len > 64) + if (len >= 64) { - md5_process_block (buffer, len & ~63, ctx); - buffer = (const char *) buffer + (len & ~63); - len &= 63; +#if !_STRING_ARCH_unaligned +/* To check alignment gcc has an appropriate operator. Other + compilers don't. */ +# if __GNUC__ >= 2 +# define UNALIGNED_P(p) (((md5_uintptr) p) % __alignof__ (md5_uint32) != 0) +# else +# define UNALIGNED_P(p) (((md5_uintptr) p) % sizeof (md5_uint32) != 0) +# endif + if (UNALIGNED_P (buffer)) + while (len > 64) + { + md5_process_block (memcpy (ctx->buffer, buffer, 64), 64, ctx); + buffer = (const char *) buffer + 64; + len -= 64; + } + else +#endif + { + md5_process_block (buffer, len & ~63, ctx); + buffer = (const char *) buffer + (len & ~63); + len &= 63; + } } /* Move remaining bytes in internal buffer. */ diff --git a/crypt/md5test.c b/crypt/md5test.c index cfd9605099..5397e5ac61 100644 --- a/crypt/md5test.c +++ b/crypt/md5test.c @@ -35,10 +35,19 @@ main (int argc, char *argv[]) for (cnt = 0; cnt < (int) (sizeof (tests) / sizeof (tests[0])); ++cnt) { + int i; + int j; + __md5_init_ctx (&ctx); __md5_process_bytes (tests[cnt].input, strlen (tests[cnt].input), &ctx); __md5_finish_ctx (&ctx, sum); result |= memcmp (tests[cnt].result, sum, 16); + + __md5_init_ctx (&ctx); + for (i = 0; tests[cnt].input[i] != '\0'; ++i) + __md5_process_bytes (&tests[cnt].input[i], 1, &ctx); + __md5_finish_ctx (&ctx, sum); + result |= memcmp (tests[cnt].result, sum, 16); } return result;