[Pkg-shadow-devel] groupmems: invalid usernames handling && segfault fix

Lukas Kuklinek lkukline at redhat.com
Tue Jul 22 09:12:51 UTC 2008


Dne Tuesday 22 of July 2008 01:29:31 Nicolas François napsal(a):
> Hello Lukáš,
>
> Thanks for the patch. I will probably make a few changes and check the
> parameters earlier. Also, I would like to harmonize the error messages
> from groupmems with the other tools.

All right, at least you know about the problem ;)

> For 455603, instead of checking if the username is valid, shouldn't we
> check if the user exists?

Good point, I was kinda "driven" by the bugzilla entry and didn't think about 
it much... :]. But it should be fixed just by replacing check_user_name() by 
getpwnam(), right?

Thanks for reply.

Lukáš



More information about the Pkg-shadow-devel mailing list