diff options
| author | Ben Laurie <ben@openssl.org> | 2001-12-20 12:18:08 +0000 |
|---|---|---|
| committer | Ben Laurie <ben@openssl.org> | 2001-12-20 12:18:08 +0000 |
| commit | 7c517a04b16de79b822573d5401d65e3a146b642 (patch) | |
| tree | dd5091a6db7a821c6fd00784864333690abde850 /crypto/ui | |
| parent | d78e52987e18b9ce7950d9215b9d6096941e35f6 (diff) | |
Security fix.
Diffstat (limited to 'crypto/ui')
| -rw-r--r-- | crypto/ui/ui_openssl.c | 39 |
1 files changed, 17 insertions, 22 deletions
diff --git a/crypto/ui/ui_openssl.c b/crypto/ui/ui_openssl.c index a958a5b878..95e0b6e921 100644 --- a/crypto/ui/ui_openssl.c +++ b/crypto/ui/ui_openssl.c @@ -148,7 +148,6 @@ #include <signal.h> #include <stdio.h> #include <string.h> -#include <setjmp.h> #include <errno.h> #ifdef OPENSSL_SYS_VMS /* prototypes for sys$whatever */ @@ -256,7 +255,6 @@ static struct sigaction savsig[NX509_SIG]; #else static void (*savsig[NX509_SIG])(int ); #endif -static jmp_buf save; #ifdef OPENSSL_SYS_VMS static struct IOSB iosb; @@ -374,6 +372,8 @@ static void read_till_nl(FILE *in) } while (strchr(buf,'\n') == NULL); } +static sig_atomic_t intr_signal; + static int read_string_inner(UI *ui, UI_STRING *uis, int echo, int strip_nl) { static int ps; @@ -383,29 +383,31 @@ static int read_string_inner(UI *ui, UI_STRING *uis, int echo, int strip_nl) char *p; #ifndef OPENSSL_SYS_WIN16 - if ((ok = setjmp(save))) - { - if (ok == 1) ok=0; - goto error; - } + intr_signal=0; ok=0; ps=0; pushsig(); ps=1; - if (!echo) noecho_console(ui); + if (!echo && !noecho_console(ui)) + goto error; ps=2; result[0]='\0'; #ifdef OPENSSL_SYS_MSDOS if (!echo) + { noecho_fgets(result,maxsize,tty_in); + p=result; /* FIXME: noecho_fgets doesn't return errors */ + } else - fgets(result,maxsize,tty_in); + p=fgets(result,maxsize,tty_in); #else - fgets(result,maxsize,tty_in); + p=fgets(result,maxsize,tty_in); #endif + if(!p) + goto error; if (feof(tty_in)) goto error; if (ferror(tty_in)) goto error; if ((p=(char *)strchr(result,'\n')) != NULL) @@ -419,9 +421,11 @@ static int read_string_inner(UI *ui, UI_STRING *uis, int echo, int strip_nl) ok=1; error: + if (intr_signal == SIGINT) + ok=-1; if (!echo) fprintf(tty_out,"\n"); - if (ps >= 2 && !echo) - echo_console(ui); + if (ps >= 2 && !echo && !echo_console(ui)) + ok=0; if (ps >= 1) popsig(); @@ -602,18 +606,9 @@ static void popsig(void) static void recsig(int i) { - switch(i) - { - case SIGINT: - longjmp(save,-1); - break; - default: - break; - } - longjmp(save,1); + intr_signal=i; } - /* Internal functions specific for Windows */ #if defined(OPENSSL_SYS_MSDOS) && !defined(OPENSSL_SYS_WIN16) static int noecho_fgets(char *buf, int size, FILE *tty) |