[PATCH] ccache_regain_all_now() loops forever in v3-0- and v3-2 [Was Re: [PATCH] clean the event context after fork in fork_domain_child()]

[boyang]

Jeremy Allison wrote:
> On Sat, Jan 10, 2009 at 08:01:21PM -0800, Jeremy Allison wrote:
>   
>> On Sun, Jan 11, 2009 at 11:21:48AM +0800, boyang wrote:
>>     
>>> Jeremy Allison wrote:
>>>       
>>>> On Fri, Jan 09, 2009 at 03:07:18PM +0800, boyang wrote:
>>>>   
>>>>         
>>>>> Jeremy Allison wrote:
>>>>>   
>>>>>     
>>>>>           
>>>>>> Ok, I've finished doing a lot of cleanup work on the
>>>>>> event code in winbindd. It's all checked into 3.3 and
>>>>>> master. Please review.
>>>>>>
>>>>>> If you feel it's working, feel free to back-port to 3.2
>>>>>> and/or 3.0.x and I'll commit patches for you. I'm not
>>>>>> doing that work yet as I'm not sure if we're doing any
>>>>>> more 3.2.x releases or just moving on to 3.3.0.
>>>>>>   
>>>>>>       
>>>>>>             
>>>>> Hi, Jeremy && metze:
>>>>>      I have ported the event context clean code and krb5 refresh chain
>>>>> code back to v3-0 and v3-2. There are two commit in each of the two
>>>>> attachments. One commit is for event context and another for krb5
>>>>> refresh chain. Please review it. Thanks!
>>>>>     
>>>>>           
>>>> Pushed, thanks !
>>>>
>>>> Jeremy.
>>>>   
>>>>         
>>> Hi, Jeremy:
>>>      ccache_regain_all_now() loops forever. :-). Why cur = cur->next is
>>> missed from v3-0-test and v3-2-test? It's my fault to post too many
>>> versions of the patch. :-P . Please have a look at the tiny patch. Thanks!
>>>       
>> I'm actually going to modify this function to be identical
>> to the code in 3.3 (other than the different function
>> prototypes of event_add_timed()) as we need consistency
>> in these implementations.
>>
>> It now looks like :
>>     
>
> If you could test and confirm this is working I'd
> appreciate it (it would save me some work on Monday :-).
>   
It is a pity that I have no testing or building environment around at
present, they are in company and the vpn is quite lagging. :-(  I'll
test the new code tomorrow(Monday). I'll get back to you as soon as I
finished the test.
> Thanks,
>
> Jeremy.
>
>   

-------------- next part --------------
A non-text attachment was scrubbed...
Name: boyang.vcf
Type: text/x-vcard
Size: 187 bytes
Desc: not available
Url : http://lists.samba.org/archive/samba-technical/attachments/20090111/6e9f2ca7/boyang.vcf