On Thu, Mar 14, 2019 at 10:22:05PM +0100, Pierre Emeriaud wrote:
> Hello,
>
> bgplgsh.8 shows an invocation to adduser with full path to bgplgsh,
> which is wrong:
>
> lg# echo /usr/bin/bgplgsh >> /etc/shells
>
> lg# adduser -shell /usr/bin/bgplgsh -batch rviews
> Shell ``/usr/bin/bgplgsh'' is undefined, use ``csh''
> Added user ``rviews''
>
> lg# getent passwd rviews
> rviews:*:1004:999::/home/rviews:/bin/csh <<<
>
> lg# userdel rviews
>
> lg# adduser -shell bgplgsh -batch rviews
> Added user ``rviews''
>
> lg# getent passwd rviews
> rviews:*:1004:999::/home/rviews:/usr/bin/bgplgsh
>
> on a related note, adduser(8) in interactive mode doesn't call
> shell_default_valid(), so it defaults to sh if invalid:
> lg# adduser -shell /usr/bin/bgplgsh rviews
> [...]
> Enter shell bgplgsh csh ksh nologin sh [/usr/bin/bgplgsh]: <<< accept
> default here
> [...]
> Added user ``rviews''
>
> lg# getent passwd rviews
> rviews:*:1004:999:cli looking glass user:/home/rviews: <<< no shell
> (not sure if that qualifies as a bug though - quite stupid in the
> first place to invoke adduser as such).
>
> regards
>
> --- bgplgsh.8 10 Sep 2015 15:16:44 - 1.11
> +++ bgplgsh.8 14 Mar 2019 21:20:21 -
> @@ -53,7 +53,7 @@ See
> .Xr adduser 8
> for more information about system user management.
> .Bd -literal -offset indent
> -# adduser -shell /usr/bin/bgplgsh -batch bgplg
> +# adduser -shell bgplgsh -batch bgplg
> # passwd bgplg
> .Ed
> .It
>
fixed, thanks.
jmc