Re: [TLS] implementation of cookies in DTLS

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

On Mar 14, 2011, at 7:16 PM, Robin Seggelmann wrote:
> 
> On 14.03.2011, at 19:01, Michael Tüxen wrote:
> 
>> 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?
> 
> I don't copy the record sequence. The server is using one counter for multiple clients until the CH with a valid cookie. So the first HelloVerifyRequest has record seq 0, the second 1 and so on. When a cookie has been verified, the ServerHello will have the next record seq, while the next HelloVerifyRequest for an other client will have 0 again.
So does this mean that one client affects the other?

Best regards
Michael
> 
> This is because I'm using an SSL object (and its rec seq counter) for listening, which will be assigned to the first client with a valid cookie to continue the connection, so the next handshake message, that is the ServerHello, will habe the next record seq. A new SSL object (with reset counters) is then created to continue listening and responding to ClientHellos.
> 
> Best reagards,
> Robin
> 
>> 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
>> 
> 
> 
> 
> Viele Grüße
> Robin
> 
> 
> 
> 
> 
>