Re: CVS commit: src/usr.bin/su

["Perry E. Metzger" <perry%piermont.com@localhost>]

Can you edit the commit message so that that's in the permanent log?

.pm

"Jason R. Fink" <jrf%adresearch.com@localhost> writes:

> > So are you going to mention it now? :-)
> 
> Under certain conditions, an su -m will fail. Specifically
> when the user is root and the passwd db is in 
> Hesiod or NIS, and root's shell is /bin/sh.
> 
> getpwnam() calls getusershell() and looks at /etc/shells. 
> The pointer remains at the second entry of /etc/shells if
> /bin/sh is first and root is using it (I would assume the
> same would happen for any).
> 
> Fix, call setusershell() to rewind right before 
> looping with getusershell().
> 

-- 
Perry E. Metzger                perry%piermont.com@localhost