Re: [secdir] secdir review of draft-ietf-sipcore-invfix-01

[Robert Sparks <[e-mail]>]

Joseph -

As one of the document editors, I'd like to thank you for your review.

As the document notes, the additional state that is being maintained
is necessary for correct operation, and that state is taken on only when
the server has decided to accept the INVITE request with a 200 OK.  
The techniques you point to would be useful if applied before getting to 
this point, and the concept is called out in the security considerations
section of RFC3261 in section 26.3.2.4. I will suggest adding a pointer
to that section in the security considerations in this document.

Thanks!

RjS

On Jun 21, 2010, at 5:53 PM, Joseph Salowey (jsalowey) wrote:

> I have reviewed this document as part of the security directorate's 
> ongoing effort to review all IETF documents being processed by the 
> IESG.  These comments were written primarily for the benefit of the 
> security area directors.  Document editors and WG chairs should treat 
> these comments just like any other last call comments.
> 
> The document fixes a real problem and seems to address some existing
> security issues, namely the forwarding of stray responses.  This is
> good.  It does introduce more of a possibility of denial of service
> since state must be maintained to track valid responses and this is
> noted in the security considerations as well.  I'm not all that familiar
> with SIP, but there are techniques used in other protocols to avoid
> "flood" type of attacks by delaying committing state until the client
> has successfully processed a server message.   TCP cookies, IKEv2 and
> DTLS have examples of this. This would be something to consider if
> implementations were vulnerable to flood type attacks in deployments.  
> 
> 
> Joe