On 08/08/2013 12:37 AM, Jakub Hrozek wrote:
On Thu, Aug 08, 2013 at 12:32:37AM +0200, Pavel Březina wrote:
> On 08/05/2013 04:36 PM, Ondrej Kos wrote:
>> Hi,
>>
>> During testing, I noticed, that all responders are logged propperly when
>> being shut down:
>>
>> [sssd[be[DOM]]] [be_client_destructor] (0x0400): Removed PAM client
>> [sssd[be[DOM]]] [be_client_destructor] (0x0400): Removed NSS client
>>
>> But not PAC:
>>
>> [sssd[be[DOM]]] [be_client_destructor] (0x0020): Unknown client removed ...
>>
>> Attached patch adds pac_cli pointer, to fix this behavior.
>>
>> Ondra
>
> Ack.
Do the _cli pointers actually serve any purpose except for the
backchannel towards nss responder? Shouldn't we rather clean them up?
That's a good question.
Only nss_cli is used, but in initgroups callback where it can be
accessed by be_req->becli (which is used by other callbacks).