From 1c3bff246cd5c22565ba6fbec1658852c9f99224 Mon Sep 17 00:00:00 2001 From: Steve Langasek Date: Tue, 10 Jul 2001 20:24:16 +0000 Subject: Relevant BUGIDs: 440107 Purpose of commit: bugfix/cleanup Commit summary: --------------- Removed superfluous use of static variables in md5 and bigcrypt routines, bringing us a step closer to thread-safeness. Eliminated some variable indirection along the way. --- modules/pam_unix/bigcrypt.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'modules/pam_unix/bigcrypt.c') diff --git a/modules/pam_unix/bigcrypt.c b/modules/pam_unix/bigcrypt.c index b1568d6b..6b73f3d2 100644 --- a/modules/pam_unix/bigcrypt.c +++ b/modules/pam_unix/bigcrypt.c @@ -25,6 +25,7 @@ */ #include +#include #include char *crypt(const char *key, const char *salt); @@ -45,7 +46,7 @@ char *bigcrypt(const char *key, const char *salt); char *bigcrypt(const char *key, const char *salt) { - static char dec_c2_cryptbuf[CBUF_SIZE]; /* static storage area */ + char *dec_c2_cryptbuf; unsigned long int keylen, n_seg, j; char *cipher_ptr, *plaintext_ptr, *tmp_ptr, *salt_ptr; @@ -54,6 +55,10 @@ char *bigcrypt(const char *key, const char *salt) D(("called with key='%s', salt='%s'.", key, salt)); /* reset arrays */ + dec_c2_cryptbuf = malloc(CBUF_SIZE); + if (!dec_c2_cryptbuf) { + return NULL; + } memset(keybuf, 0, KEYBUF_SIZE + 1); memset(dec_c2_cryptbuf, 0, CBUF_SIZE); -- cgit v1.2.3