Subject: Re: CVS commit: src/usr.bin/su
To: Jason R. Fink <jrf@adresearch.com>
From: Perry E. Metzger <perry@piermont.com>
List: source-changes
Date: 06/19/2003 23:40:43
Can you edit the commit message so that that's in the permanent log?
.pm
"Jason R. Fink" <jrf@adresearch.com> 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