[1219] in Athena Bugs
passwd.real on demeter
daemon@ATHENA.MIT.EDU (probe@ATHENA.MIT.EDU)
Sun Oct 23 04:31:59 1988
From: <probe@ATHENA.MIT.EDU>
Date: Sun, 23 Oct 88 04:31:44 EDT
To: jik@ATHENA.MIT.EDU
Cc: bugs@ATHENA.MIT.EDU, paul@ATHENA.MIT.EDU
In-Reply-To: <jik@ATHENA.MIT.EDU>'s message of Sat, 22 Oct 88 22:34:06 EDT <8810230234.AA09477@W20-575-50.MIT.EDU>
Reply-To: Richard Basch <probe@ATHENA.MIT.EDU>
First, let me state that the problem was fixed in one of the early 6.0
releases, so the rest of the message is just a FYI. (The fact that it
had been fixed has been mentioned in the release notes).
> From: <jik@ATHENA.MIT.EDU>
> Date: Sat, 22 Oct 88 22:34:06 EDT
> System type, version: VAX750 version 5.5T
> System name: demeter
> What's wrong:
> demeter% whoami
> ceg
> demeter% passwd.real -s
> Changing login shell for ceg.
> Old shell: /bin/unsupported_tcsh
> New shell: /bin/csh
> /bin/csh is unacceptable as a new shell.
> demeter% cat /etc/shells
> # List of acceptable shells for chsh/passwd -s
> # Ftpd will not allow users to connect who do not have one of these shells
> #
> # $Source: /source/config/common/RCS/shells,v $
> # $Header: shells,v 1.1 87/08/09 13:10:55 treese Locked $
> /bin/sh
> /bin/csh
> /bin/unsupported_tcsh
> What should have happened:
> Passwd.real -s should allow me to change to /bin/csh. It works on
> charon.
Knowing it is fixed on charon, does not help substantially. Is charon
running Athena's version? Where are charon's sources? I think you see
what I mean... (Never mind that I am in SIPB; not all the people on bugs
are).
The problem is that it did not rewind the shell's file before
re-reading it, so you could only change to a shell that occurred later
in the file. The reason it had to check the file twice was so that it
could test to see if the current shell was one of those "unrestricted"
shells listed in /etc/shells. If this was the case, then you were
permitted to change your shell and you could select from the
"unrestricted" shells. This simply meant adding a setusershell() call
to /paris/source/4.3/bin/passwd.c: getloginshell().
The fix for demeter: upgrade its software, which I hope will be done to
all servers so that the NFS null bug problem will go away.
-Richard
