diff options
| author | Steve Langasek <vorlon@debian.org> | 2001-07-10 20:24:16 +0000 |
|---|---|---|
| committer | Steve Langasek <vorlon@debian.org> | 2001-07-10 20:24:16 +0000 |
| commit | 1c3bff246cd5c22565ba6fbec1658852c9f99224 (patch) | |
| tree | 64d882a1c56887e470b6d55fe7d6dd5fdf9228e2 /modules/pam_unix/bigcrypt.c | |
| parent | e5d527e8dfba82f1c47f6b1d3751cf2f17cf2cab (diff) | |
| download | pam-1c3bff246cd5c22565ba6fbec1658852c9f99224.tar.gz pam-1c3bff246cd5c22565ba6fbec1658852c9f99224.tar.bz2 pam-1c3bff246cd5c22565ba6fbec1658852c9f99224.zip | |
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.
Diffstat (limited to 'modules/pam_unix/bigcrypt.c')
| -rw-r--r-- | modules/pam_unix/bigcrypt.c | 7 |
1 files changed, 6 insertions, 1 deletions
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 <string.h> +#include <stdlib.h> #include <security/_pam_macros.h> 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); |