Re: [TLS] implementation of cookies in DTLS

[Michael Tüxen <[e-mail]>]

On Mar 14, 2011, at 5:20 PM, Robin Seggelmann wrote:
> 
> 
> On 14.03.2011, at 16:52, Robin Seggelmann <[e-mail]> wrote:
> 
>> 
>> On Mar 14, 2011, at 4:30 PM, Nikos Mavrogiannopoulos wrote:
>> 
>>> On 03/14/2011 04:12 PM, Robin Seggelmann wrote:
>>> 
>>>>> Hello, I've been reading the section "Denial of Service
>>>>> Countermeasures" of DTLS and as I understand it the proposed
>>>>> subsystem (client-hello and client-hello-verify-request) is
>>>>> expected to operate before allocating state for the session to
>>>>> discard requests from clients with forged addresses.
>>>>> 
>>>>> Some comments: 1. The document says: If a server receives a
>>>>> ClientHello with an invalid cookie, it SHOULD treat it the same as
>>>>> a ClientHello with no cookie.
>>>>> 
>>>>> What does it mean with regards to the handshake sequence. Does the 
>>>>> second HelloVerifyRequest has seq=0 or seq=1?
>>>> 
>>>> The second HelloVerifyRequest should have seq=0 again, since the
>>>> server must not change its state. That's how I implemented it in
>>>> OpenSSL.  The server does not change any sequence numbers until a
>>>> cookie has been verified and the regular handshake continues.
>>> 
>>> What about the record_seq. Is it also 0? If yes  a client
>>> would see this record packet as a replay and discard it. If no
>>> it seems you have to keep state...
>> 
>> The record_seq is still increased, because it has to be unique. I wouldn't consider this as a problem because it's just a counter you're increasing. There is no resource allocation necessary and there is no benefit for an attacker to send many ClientHellos to increase the sequence number. It's only used for HMAC verification and replay check and will be reset to 0 after the ChangCipherSpec anyway.
>> 
>> However, I see your point. Maybe the document should state more explicitly that handshake sequence numbers must not be increased, because they always have to be the same and always have to start with 0, while the record_seq has to be unique and therefore has to change.
> 
> I forgot to mention that in OpenSSL I'm using the same record_seq counter for every incoming request until a CH with a valid cookie arrives. This is due to a workaround to implement an UDP based and thus one-to-many style protocol in OpenSSL's one-to-one architecture, but so I'm not required to allocate anything.
Hi Robin,

can you be more specific? Did you implement it such that Nikos
message exchange is valid? So do you copy the rec. seq. number
from the incoming packet? Do you use the same counter for multiple
client until you receive a CH with a valid cookie?

Best regards
Michael
> 
>> Regards,
>> Robin
>> 
>>>>> e.g. if I receive ClientHello (record_seq=0, seq=0)  ------>
>>>>> 
>>>>> <-- HelloVerifyRequest (record_seq=0, seq=0)
>>>>> 
>>>>> ClientHello (record_seq=1, seq=1)  ------> (wrong cookie)
>>>>> 
>>>>> <-- HelloVerifyRequest (record_seq=1, seq=1) [seq copied from
>>>>> clienthello]
>>>>> 
>>>>> ClientHello (record_seq=2, seq=2)  ------> (wrong cookie)
>>>>> 
>>>>> <-- HelloVerifyRequest (record_seq=2, seq=2) [seq copied from
>>>>> clienthello]
>>>>> 
>>>>> ClientHello (record_seq=3, seq=3)  ------>
>>>>> 
>>>>> [server allocates session and continues handshake]
>>>> That may be helpful, but in my opinion "must not change its state"
>>>> includes not changing any sequence numbers.
>>> 
>>> What is of most concern is the record_seq. A client would just discard
>>> packets with the same sequence number.
>>> 
>>> regards,
>>> Nikos
>> 
>> 
> _______________________________________________
> TLS mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/tls