Subject: Re: CVS commit: src/usr.sbin/user
To: Hubert Feyrer <>
From: Alistair Crooks <>
List: source-changes
Date: 02/23/2006 18:54:15
On Thu, Feb 23, 2006 at 01:04:53PM +0100, Hubert Feyrer wrote:
> On Fri, 6 Jan 2006, Christos Zoulas wrote:
> >Modified Files:
> >	src/usr.sbin/user: user.c
> >
> >Log Message:
> >Add checks for valid shell. From Liam Foy.
> How do I add a user with /bin/nologin as shell?
> (which shouldn't be in /etc/shells as it would allow the user to login via 
> FTP).
> I'm not sure this change is good...

It's a bit late, but you are right - that was why we didn't mandate
that the login shell be a valid one, as found in /etc/shells.

The login shell should be allowed to be anything.