From 3248e3a326e04320d0fdecba70be17d4f13bfb27 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Tue, 4 Jul 2000 18:24:11 +0000 Subject: [PATCH] Update. 2000-07-04 Ulrich Drepper * crypt/md5-crypt.c (__md5_crypt_r): If buffers for key and salt are not aligned to alignof(md5_uint32) do it before calling __md5_process_bytes. * crypt/md5.c: Make sure buffers are aligned. * crypt/md5.h: Likewise. Reported by Solar Designer . * crypt/Makefile: Add dependencies for test programs. * Rules: Define LC_ALL=C in environments of all programs we run. * intl/tst-gettext.sh (LC_ALL): Define to C and export. 2000-07-03 H.J. Lu * locale/programs/ld-ctype.c (ctype_output): The size of iov is 2 + elem + offset, not 2 + elem + offset + 2. 2000-07-04 Ulrich Drepper * posix/fnmatch_loop.c: Fix two problems uncovered by the new test suite. * posix/Makefile (tests): Add tst-fnmatch. (tst-fnmatch-ENV): Define. * posix/tst-fnmatch.c: New file. * posix/tst-fnmatch.sh: New file. --- ChangeLog | 30 ++++++++++++++++++++++++++++++ Rules | 4 ++-- crypt/Makefile | 13 +++++++++++-- crypt/md5-crypt.c | 30 +++++++++++++++++++++++++----- crypt/md5.c | 31 +++++++++++++++++++++++-------- crypt/md5.h | 4 ++-- 6 files changed, 93 insertions(+), 19 deletions(-) diff --git a/ChangeLog b/ChangeLog index 29a8e7c163..5c4d362a2b 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,32 @@ +2000-07-04 Ulrich Drepper + + * crypt/md5-crypt.c (__md5_crypt_r): If buffers for key and salt + are not aligned to alignof(md5_uint32) do it before calling + __md5_process_bytes. + * crypt/md5.c: Make sure buffers are aligned. + * crypt/md5.h: Likewise. + Reported by Solar Designer . + + * crypt/Makefile: Add dependencies for test programs. + + * Rules: Define LC_ALL=C in environments of all programs we run. + + * intl/tst-gettext.sh (LC_ALL): Define to C and export. + +2000-07-03 H.J. Lu + + * locale/programs/ld-ctype.c (ctype_output): The size of iov + is 2 + elem + offset, not 2 + elem + offset + 2. + +2000-07-04 Ulrich Drepper + + * posix/fnmatch_loop.c: Fix two problems uncovered by the new test + suite. + * posix/Makefile (tests): Add tst-fnmatch. + (tst-fnmatch-ENV): Define. + * posix/tst-fnmatch.c: New file. + * posix/tst-fnmatch.sh: New file. + 2000-07-04 NIIBE Yutaka * locale/programs/charmap.c (charmap_read): Prepend @@ -21,6 +50,7 @@ Wrap extern symbols in BP_SYM (). 2000-07-03 Andreas Jaeger + * dlfcn/Makefile (generated): New. 2000-07-03 Ulrich Drepper diff --git a/Rules b/Rules index 1ccc027694..f3d8bdd746 100644 --- a/Rules +++ b/Rules @@ -119,10 +119,10 @@ ifneq "$(strip $(tests) $(test-srcs))" "" # These are the implicit rules for making test outputs # from the test programs and whatever input files are present. $(objpfx)%.out: %.input $(objpfx)% - GCONV_PATH=$(common-objpfx)iconvdata \ + GCONV_PATH=$(common-objpfx)iconvdata LC_ALL=C \ $($*-ENV) $(built-program-cmd) $($*-ARGS) < $(word 1,$^) > $@ $(objpfx)%.out: /dev/null $(objpfx)% # Make it 2nd arg for canned sequence. - GCONV_PATH=$(common-objpfx)iconvdata \ + GCONV_PATH=$(common-objpfx)iconvdata LC_ALL=C \ $($*-ENV) $(built-program-cmd) $($*-ARGS) > $@ endif # tests diff --git a/crypt/Makefile b/crypt/Makefile index 055b76d43a..0692c2647c 100644 --- a/crypt/Makefile +++ b/crypt/Makefile @@ -45,8 +45,17 @@ $(objpfx)md5test: $(objpfx)md5.o include ../Rules -LDLIBS-cert = crypt/libcrypt -LDLIBS-md5c-test = crypt/libcrypt +LDLIBS-cert = libcrypt +LDLIBS-md5c-test = libcrypt + +ifeq (yes,$(build-shared)) +libcrypt-dep = $(objpfx)libcrypt.so +else +libcrypt-dep = $(objpfx)libcrypt.a +endif + +$(objpfx)cert.out: $(libcrypt-dep) +$(objpfx)md5c-test.out: $(libcrypt-dep) # Depend on libc.so so a DT_NEEDED is generated in the shared objects. # This ensures they will load libc.so for needed symbols if loaded by diff --git a/crypt/md5-crypt.c b/crypt/md5-crypt.c index 126103503c..3b20ed157d 100644 --- a/crypt/md5-crypt.c +++ b/crypt/md5-crypt.c @@ -1,5 +1,5 @@ /* One way encryption based on MD5 sum. - Copyright (C) 1996, 1997, 1999 Free Software Foundation, Inc. + Copyright (C) 1996, 1997, 1999, 2000 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper , 1996. @@ -18,6 +18,7 @@ write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. */ +#include #include #include #include @@ -37,9 +38,9 @@ static const char b64t[64] = /* Prototypes for local functions. */ -extern char *__md5_crypt_r __P ((const char *key, const char *salt, - char *buffer, int buflen)); -extern char *__md5_crypt __P ((const char *key, const char *salt)); +extern char *__md5_crypt_r (const char *key, const char *salt, + char *buffer, int buflen); +extern char *__md5_crypt (const char *key, const char *salt); /* This entry point is equivalent to the `crypt' function in Unix @@ -51,7 +52,8 @@ __md5_crypt_r (key, salt, buffer, buflen) char *buffer; int buflen; { - unsigned char alt_result[16]; + unsigned char alt_result[16] + __attribute__ ((__aligned__ (__alignof__ (md5_uint32)))); struct md5_ctx ctx; struct md5_ctx alt_ctx; size_t salt_len; @@ -68,6 +70,24 @@ __md5_crypt_r (key, salt, buffer, buflen) salt_len = MIN (strcspn (salt, "$"), 8); key_len = strlen (key); + if ((key - (char *) 0) % __alignof__ (md5_uint32) != 0) + { + char *tmp = (char *) alloca (key_len + __alignof__ (md5_uint32)); + key = memcpy (tmp + __alignof__ (md5_uint32) + - (tmp - (char *) 0) % __alignof__ (md5_uint32), + key, key_len); + assert ((key - (char *) 0) % __alignof__ (md5_uint32) == 0); + } + + if ((salt - (char *) 0) % __alignof__ (md5_uint32) != 0) + { + char *tmp = (char *) alloca (salt_len + __alignof__ (md5_uint32)); + salt = memcpy (tmp + __alignof__ (md5_uint32) + - (tmp - (char *) 0) % __alignof__ (md5_uint32), + salt, salt_len); + assert ((salt - (char *) 0) % __alignof__ (md5_uint32) == 0); + } + /* Prepare for the real work. */ __md5_init_ctx (&ctx); diff --git a/crypt/md5.c b/crypt/md5.c index 478429f8e6..3879ad674c 100644 --- a/crypt/md5.c +++ b/crypt/md5.c @@ -1,6 +1,6 @@ -/* md5.c - Functions to compute MD5 message digest of files or memory blocks +/* 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 Free Software Foundation, Inc. + Copyright (C) 1995, 1996, 1997, 1999, 2000 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,6 +217,8 @@ 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) @@ -224,16 +226,20 @@ 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; - if (left_over + add > 64) + if (ctx->buflen > 64) { - md5_process_block (ctx->buffer, (left_over + add) & ~63, ctx); + md5_process_block (ctx->buffer, ctx->buflen & ~63, ctx); + + ctx->buflen &= 63; /* The regions in the following copy operation cannot overlap. */ memcpy (ctx->buffer, &ctx->buffer[(left_over + add) & ~63], - (left_over + add) & 63); - ctx->buflen = (left_over + add) & 63; + ctx->buflen); } buffer = (const char *) buffer + add; @@ -251,8 +257,17 @@ md5_process_bytes (buffer, len, ctx) /* Move remaining bytes in internal buffer. */ if (len > 0) { - memcpy (ctx->buffer, buffer, len); - ctx->buflen = len; + size_t left_over = ctx->buflen; + + memcpy (&ctx->buffer[left_over], buffer, len); + left_over += len; + if (left_over >= 64) + { + md5_process_block (ctx->buffer, 64, ctx); + left_over -= 64; + memcpy (ctx->buffer, &ctx->buffer[64], left_over); + } + ctx->buflen = left_over; } } diff --git a/crypt/md5.h b/crypt/md5.h index fbc2bdbf93..b27254b216 100644 --- a/crypt/md5.h +++ b/crypt/md5.h @@ -1,6 +1,6 @@ /* Declaration of functions and data types used for MD5 sum computing library functions. - Copyright (C) 1995, 1996, 1997, 1999 Free Software Foundation, Inc. + Copyright (C) 1995, 1996, 1997, 1999, 2000 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 @@ -87,7 +87,7 @@ struct md5_ctx md5_uint32 total[2]; md5_uint32 buflen; - char buffer[128]; + char buffer[128] __attribute__ ((__aligned__ (__alignof__ (md5_uint32)))); }; /*