[SCM] Samba Shared Repository - branch master updated - tevent-0-9-8-590-g00e378f

[Stefan (metze) Metzmacher]

Volker Lendecke schrieb:
> On Wed, Sep 30, 2009 at 12:51:59PM +0200, Stefan (metze) Metzmacher wrote:
>> BTW: in master I need this
>> http://gitweb.samba.org/?p=metze/samba/wip.git;a=commitdiff;h=19dec031c7c8d5f4b646ff5f450c16b9f98f9dac
>> diff --git a/lib/tevent/tevent_signal.c b/lib/tevent/tevent_signal.c
>> index d3325b6..46bcfca 100644 (file)
>> --- a/lib/tevent/tevent_signal.c
>> +++ b/lib/tevent/tevent_signal.c
>> @@ -94,6 +94,8 @@ static void tevent_common_signal_handler(int signum)
>>         for (sl = sig_state->sig_handlers[signum]; sl; sl = sl->next) {
>>                 if (sl->se->event_ctx != ev) {
>>                         ev = sl->se->event_ctx;
>> +                       if (!ev) continue;
>> +                       if (!ev->pipe_fds) continue;
>>                         /* doesn't matter if this pipe overflows */
>>                         res = write(ev->pipe_fds[1], &c, 1);
>>                 }
>>
>> Otherwise winbindd childs crash after the fork.
> 
> Please talk to rusty about this one. I thought I had fixed
> it, but then he said my fix is wrong. I did not have enough
> time to look at that yet.

No problem, it seems to work in v3-4-test, it's only master that has
this problem...

I try to reproduce the both problems with master later this week
and debug it.

metze