[SSSD] [PATCH] DP: Notify propperly when removing PAC responder

Pavel Březina pbrezina at redhat.com
Wed Aug 7 23:06:24 UTC 2013


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).



More information about the sssd-devel mailing list