Bruce Evans wrote: > On Wed, 6 Jun 2001, Maxim Sobolev wrote: > > > It is very strange, but control keys [^C,^Z etc] no longer work (nop) > > in the /bin/sh and bash2 after today's build/installworld. I see this > > misbehaviour on two machines. > > PAM now blocks keyboard signals when reading the password, and usually > forgets to unblock them. I use the workaround of backing out the broken > code (rev.1.4 of /usr/src/contrib/libpam/libpam_misc/misc_conv.c). That explains... Attached patch solved the problem for me. > > Even more strange that /bin/tcsh doesn't > > have this problem. > > This may be a bug in tcsh. Maybe... -Maxim
Index: misc_conv.c =================================================================== RCS file: /home/ncvs/src/contrib/libpam/libpam_misc/misc_conv.c,v retrieving revision 1.4 diff -d -u -r1.4 misc_conv.c --- misc_conv.c 2001/06/04 20:59:49 1.4 +++ misc_conv.c 2001/06/07 08:10:55 @@ -132,6 +132,7 @@ static char *read_string(int echo, const char *prompt) { struct termios term_before, term_tmp; + char *input; char line[INPUTSIZE]; struct sigaction old_sig; int delay, nc, have_term=0; @@ -191,8 +192,6 @@ if (expired) { delay = get_delay(); } else if (nc > 0) { /* we got some user input */ - char *input; - if (nc > 0 && line[nc-1] == '\n') { /* <NUL> terminate */ line[--nc] = '\0'; } else { @@ -201,25 +200,27 @@ input = x_strdup(line); _pam_overwrite(line); - return input; /* return malloc()ed string */ + goto cleanexit; /* return malloc()ed string */ } else if (nc == 0) { /* Ctrl-D */ char *input; D(("user did not want to type anything")); input = x_strdup(""); - return input; /* return malloc()ed string */ + goto cleanexit; /* return malloc()ed string */ } } } /* getting here implies that the timer expired */ + memset(line, 0, INPUTSIZE); /* clean up */ + input = NULL; + +cleanexit: if (have_term) { (void)_sigprocmask(SIG_SETMASK, &oset, NULL); (void) tcsetattr(STDIN_FILENO, TCSADRAIN, &term_before); } - - memset(line, 0, INPUTSIZE); /* clean up */ - return NULL; + return input; } /* end of read_string functions */