> I've updated the function to use strdup().
> 
> Current webrev at http://cr.opensolaris.org/~bartbl/6251549-20090225/.

        Why are 393-397 needed at all?  Why not return uspw->pw_name
        or NULL?

Gary..

Reply via email to