Subject: Re: CVS commit: src/usr.sbin/user
To: Alistair Crooks <>
From: Hubert Feyrer <>
List: source-changes
Date: 02/23/2006 19:59:49
On Thu, 23 Feb 2006, Alistair Crooks 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.

If anything, a warning would be on.

Christos, what do you think?

  - Hubert