Patch for fixing pb when users have a valid ticket and the server change its password

[Matthieu Patou]

Hello,

Can someone review this patch ?

On IRC simo said it was quite ok.

Matthieu.

On 25/05/2010 14:38, Matthieu Patou wrote:
> Hi all,
>
>>> This looks good.  I hope to rework this properly some day to use
>>> exclusively the keytab approach (so we don't waste CPU re-salting the
>>> machine password), but this fixes the issue without a bit change in the
>>> structure.
>>>
>>> The main concern I have is that by changing from one 'for' loop to two
>>> different for loops, that the meaning of 'break' has changed.  Some of
>>> the error conditions should cause a retry with a different enc type,
>>> while others are fatal.  As far as I can tell, you only have an early
>>> exit for success.
>> Uh right ! I guess i can refactor to add a boolean and use it to 
>> recreate the early exit in different error case (when there is a break)
> One more round with more tests and early exit if the ticket is invalid.
>
>


-- 
Matthieu Patou
Samba Team        http://samba.org