[PATCH] Start to fix bug 8630

[Uri Simchoni]

On 01/01/2017 10:57 PM, Andreas Schneider wrote:
> On Sunday, 1 January 2017 17:17:47 CET Volker Lendecke wrote:
>> On Fri, Dec 30, 2016 at 05:01:30PM +0100, Volker Lendecke wrote:
>>> On Fri, Dec 30, 2016 at 02:50:20PM +0100, Volker Lendecke wrote:
>>>> The attached patchset is a rewrite of how winbind puts together user
>>>> information. Its goal is to remove domain_list dependency and make us
>>>> work better in complex trust scenarios. It is waiting for the last few
>>>> tests in autobuild, but all the winbind ones have passwd, autobuild
>>>> has already found some quirks in early versions.
>>>
>>> Gna. Failed in samba.blackbox.wbinfo after 3h22m12s. So more fixups
>>> needed, but I would still appreciate some review already.
>>
>> This one just survived an autobuild for me. The diff against the
>> previous version is also attached.
> 
> I can review this tomorrow.
> 
> 
> 	Andreas
> 
RB+ me.

Can we add a WHATSNEW that documents the slight changes in behavior? The
ones I spotted were:
- if the getpw{nam,sid} user has no netsamlogon cache entry, the gecos
field is now empty (before there was an attempt by the backend to fill
it up)
- getgrouplist() nss call (or id utility) returns only the primary group
id if the user doesn't have a netsamlogon cache entry (i.e. if he hasn't
logged on, as in the case of "force user" parameter for example).

Andreas, do you want to have a look too or shall I push it?
Thanks,
Uri.