Re: CVS commit: src/usr.sbin/user

[Hubert Feyrer <hubert%feyrer.de@localhost>]

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