[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