Subject: Re: CVS commit: src/usr.sbin/user
To: Alistair Crooks <agc@pkgsrc.org>
From: Hubert Feyrer <hubert@feyrer.de>
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.

Seconded.
If anything, a warning would be on.

Christos, what do you think?


  - Hubert