[PATCH] clean the event context after fork in fork_domain_child()

Stefan (metze) Metzmacher metze at samba.org
Tue Jan 6 16:47:04 GMT 2009


Jeremy Allison schrieb:
> On Tue, Jan 06, 2009 at 09:28:32AM +0100, Stefan (metze) Metzmacher wrote:
>> Jeremy Allison schrieb:
>>> On Tue, Jan 06, 2009 at 01:19:41PM +0800, boyang wrote:
>>>
>>>> v2-event-context-master.diff frees check_online_event. But I think you
>>>> don't need it now. Because all events freeing work will be done in
>>>> winbindd_reinit_after_fork() :-). As you said.
>>> Yep, as soon as I've written it :-).
>> Hi Jeremy,
>>
>> could make winbindd_reinit_after_fork() call reinit_after_fork(),
>> it would be nice to keep the common code in one spot.
> 
> Yes, that's the plan. All the xxxx_reinit_after_fork()
> calls should do their own specific cleanups, then call
> reinit_after_fork().

The best would be if we would create completely new
event_context and messaging_context instead of reinit the current
ones. that would remove the samba3 specifix event_context_reinit(),
but we need to register all messaging callbacks again (but that
would make the code much more understandable for me).

I would have done this, but I wasn't sure what this would break.

metze


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 252 bytes
Desc: OpenPGP digital signature
Url : http://lists.samba.org/archive/samba-technical/attachments/20090106/e4d0c952/signature.bin


More information about the samba-technical mailing list