Re: [PATCH 05/17] newgrp: use xgetpass() and memset_s() to group password validation

Karel Zak
2014-09-12T08:45:08Z

On Sun, Sep 07, 2014 at 01:42:53PM +0100, Sami Kerola wrote:
> Signed-off-by: Sami Kerola <kerolasa@iki.fi>
> ---
> login-utils/newgrp.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)

If we really want to support passwords for groups then it would be
better to add support for this to PAM. But it seems it's so crazy and
unnecessary that nobody has been motivated to do this change in last
20 years.

It would be better to remove support for /etc/gshadow from newgrp at
all.

Karel

--
Karel Zak <kzak@redhat.com>