Bug 5198 - Gecos field is passed as full name

[simo]

On Fri, 2010-02-05 at 15:56 -0500, Jesse Malone wrote:
> Hi,
> 
>  
> 
> I, and others, have noticed that the gecos field is not parsed by samba, and
> so we wind up with the entire comma seperated gecos field as a user's full
> name.
> 
>  
> 
> The bug is posted here: https://bugzilla.samba.org/show_bug.cgi?id=5198
> 
>  
> 
> It doesn't look like there has been any activity or comment on this bug from
> any knowledgeable developers. I've proposed  a potential solution, but don't
> really know samba at all and would appreciate comments from those more
> knowledgeable than me.
> 
>  
> 
> -----------------------------------------------------------------------
> 
> My proposal follows: I'll start off by saying I really don't know this
> project so opinions/corrections/suggestions are very much appreciated.
> Proposed fix (untested thus far):

Please propose a patch using unidiff format and no C++ style comments.

A git format-patch would even be better but unidiff is enough.

Once you have it attach it to the bug.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Principal Software Engineer at Red Hat, Inc. <simo at redhat.com>