Re: [Isis-wg] Security review of draft-ietf-isis-extended-sequence-no-tlv-04

Uma Chunduri <uma.chunduri@ericsson.com> Thu, 02 April 2015 21:08 UTC

Return-Path: <uma.chunduri@ericsson.com>
X-Original-To: expand-draft-ietf-isis-extended-sequence-no-tlv.all@virtual.ietf.org
Delivered-To: isis-wg@ietfa.amsl.com
Received: by ietfa.amsl.com (Postfix, from userid 65534) id 8E4391A1F20; Thu, 2 Apr 2015 14:08:15 -0700 (PDT)
X-Original-To: xfilter-draft-ietf-isis-extended-sequence-no-tlv.all@ietfa.amsl.com
Delivered-To: xfilter-draft-ietf-isis-extended-sequence-no-tlv.all@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6A1041A1EF2 for <xfilter-draft-ietf-isis-extended-sequence-no-tlv.all@ietfa.amsl.com>; Thu, 2 Apr 2015 14:08:15 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.234
X-Spam-Level:
X-Spam-Status: No, score=-1.234 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, SPF_SOFTFAIL=0.665] autolearn=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id CaKEPnQsuzRh for <xfilter-draft-ietf-isis-extended-sequence-no-tlv.all@ietfa.amsl.com>; Thu, 2 Apr 2015 14:08:12 -0700 (PDT)
Received: from zinfandel.tools.ietf.org (zinfandel.tools.ietf.org [IPv6:2001:1890:123a::1:2a]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 489291A1EF1 for <draft-ietf-isis-extended-sequence-no-tlv.all@ietf.org>; Thu, 2 Apr 2015 14:08:06 -0700 (PDT)
Received: from usevmg21.ericsson.net ([198.24.6.65]:44080) by zinfandel.tools.ietf.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.82_1-5b7a7c0-XX) (envelope-from <uma.chunduri@ericsson.com>) id 1YdmLM-0002XT-P7 for draft-ietf-isis-extended-sequence-no-tlv.all@tools.ietf.org; Thu, 02 Apr 2015 14:08:06 -0700
X-AuditID: c6180641-f790b6d000004359-a6-551d4d4f0fb2
Received: from EUSAAHC007.ericsson.se (Unknown_Domain [147.117.188.93]) by usevmg21.ericsson.net (Symantec Mail Security) with SMTP id 0C.78.17241.F4D4D155; Thu, 2 Apr 2015 16:08:16 +0200 (CEST)
Received: from EUSAAMB105.ericsson.se ([147.117.188.122]) by EUSAAHC007.ericsson.se ([147.117.188.93]) with mapi id 14.03.0210.002; Thu, 2 Apr 2015 17:07:57 -0400
From: Uma Chunduri <uma.chunduri@ericsson.com>
To: "Adam W. Montville" <adam.w.montville@gmail.com>
Thread-Topic: Security review of draft-ietf-isis-extended-sequence-no-tlv-04
Thread-Index: AQHQbH/Zhsqwc8PAg0OvMaxLgCljg504XO/ggAF8m4CAAFtuAA==
Date: Thu, 02 Apr 2015 21:07:56 +0000
Message-ID: <1B502206DFA0C544B7A60469152008633F61F2BE@eusaamb105.ericsson.se>
References: <718D753F-E377-4197-9F01-C34C12EC5CA0@gmail.com> <1B502206DFA0C544B7A60469152008633F61E19F@eusaamb105.ericsson.se> <587CC58A-C9D6-4FCE-8BF0-80BC30CE5D2B@gmail.com>
In-Reply-To: <587CC58A-C9D6-4FCE-8BF0-80BC30CE5D2B@gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [147.117.188.10]
Content-Type: multipart/alternative; boundary="_000_1B502206DFA0C544B7A60469152008633F61F2BEeusaamb105erics_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFupgkeLIzCtJLcpLzFFi42KZXLonVjfAVzbUYN1mLostD9tYLPbdfMdu MePPRGaLDwsfsjiweOycdZfdY8mSn0weXy5/ZgtgjuKySUnNySxLLdK3S+DKOLH6GWPBlwam ijdd7A2MU34xdjFyckgImEgc/32bHcIWk7hwbz0biC0kcJRR4uJnsS5GLiB7GaNER28vWIJN QE/i49SfYA0iQM1fvj1jAiliFjjEKLHuSQdYQljAW+JTdxsbRJGPxOZJj4HiHEC2k0TvLxeQ MIuAisS2pgdMIDavgK/E3N/HmCGWbWaUWD1hGzNIglPAVuJ2/zdWEJsR6Lrvp9aANTALiEvc ejKfCeJqAYkle84zQ9iiEi8f/2OFsJUkJi09xwpRny/xYl87G8QyQYmTM5+wTGAUnYVk1Cwk ZbOQlM0COptZQFNi/S59iBJFiSndD9khbA2J1jlz2ZHFFzCyr2LkKC1OLctNNzLcxAiMumMS bI47GBd8sjzEKMDBqMTDm5AlEyrEmlhWXJl7iFGag0VJnLfsysEQIYH0xJLU7NTUgtSi+KLS nNTiQ4xMHJxSDYwHXVx3qW7YYP95/1qN29+PVR4MXajhz7SdQ3ldU1PBo0oWnd/zsj4FON7x WbubZ+PHy8l7aw6nnHuW18zTZv2KSVtPdWrlOSEPtffvw6fa3jE7VnbV7sDHc7ZPVl3U2mqe mrbEccterpLtV3YenzXNuiaHo0T75OXMw6G1ppeOMbuJBf37dVNGiaU4I9FQi7moOBEAFO1H 7psCAAA=
X-SA-Exim-Connect-IP: 198.24.6.65
X-SA-Exim-Rcpt-To: draft-ietf-isis-extended-sequence-no-tlv.all@tools.ietf.org
X-SA-Exim-Mail-From: uma.chunduri@ericsson.com
X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000)
X-SA-Exim-Scanned: Yes (on zinfandel.tools.ietf.org)
Resent-To: draft-ietf-isis-extended-sequence-no-tlv.all@ietf.org
Resent-Message-Id: <20150402210806.489291A1EF1@ietfa.amsl.com>
Resent-Date: Thu, 02 Apr 2015 14:08:06 -0700
Resent-From: uma.chunduri@ericsson.com
Archived-At: <http://mailarchive.ietf.org/arch/msg/draft-ietf-isis-extended-sequence-no-tlv.all@tools/KHne7lphJ37DohifNnB8xBQ6fC4>
Archived-At: <http://mailarchive.ietf.org/arch/msg/isis-wg/feJkJ2TfmcO3Y-kz-Zl4vhUkdQs>
X-Mailman-Approved-At: Fri, 03 Apr 2015 08:05:47 -0700
Cc: "draft-ietf-isis-extended-sequence-no-tlv.all@tools.ietf.org" <draft-ietf-isis-extended-sequence-no-tlv.all@tools.ietf.org>, The IESG <iesg@ietf.org>, "secdir@ietf.org" <secdir@ietf.org>
Subject: Re: [Isis-wg] Security review of draft-ietf-isis-extended-sequence-no-tlv-04
X-BeenThere: isis-wg@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: IETF IS-IS working group <isis-wg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/isis-wg>, <mailto:isis-wg-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/isis-wg/>
List-Post: <mailto:isis-wg@ietf.org>
List-Help: <mailto:isis-wg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/isis-wg>, <mailto:isis-wg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 02 Apr 2015 21:08:15 -0000

Hi Adam,

Thank you. My quick response in-line [Uma1]:

--
Uma C.

From: Adam W. Montville [mailto:adam.w.montville@gmail.com]
Sent: Thursday, April 02, 2015 4:26 AM
To: Uma Chunduri
Cc: The IESG; secdir@ietf.org; draft-ietf-isis-extended-sequence-no-tlv.all@tools.ietf.org
Subject: Re: Security review of draft-ietf-isis-extended-sequence-no-tlv-04

Thanks for your response.  I have a few more comments inline.


On Apr 1, 2015, at 5:07 PM, Uma Chunduri <uma.chunduri@ericsson.com<mailto:uma.chunduri@ericsson.com>> wrote:

Hi Adam,

Thanks for your review and suggestions.
Please see my replies in-line [Uma]:

-----Original Message-----
From: Adam W. Montville [mailto:adam.w.montville@gmail.com]
Sent: Wednesday, April 01, 2015 6:29 AM
To: The IESG; secdir@ietf.org<mailto:secdir@ietf.org>; draft-ietf-isis-extended-sequence-no-tlv.all@tools.ietf.org<mailto:draft-ietf-isis-extended-sequence-no-tlv.all@tools.ietf.org>
Subject: Security review of draft-ietf-isis-extended-sequence-no-tlv-04

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.

This draft is ready with issues.

There are two circumstances within IS-IS that are subject to replay attacks.  One can happen when adjacencies are brought up and an IS sends an IIH (hello).  The other circumstance pertains to replaying CSNP, which may cause replay of PSNP and thus cause LSP flooding.

This draft proposes a sequence to mitigate replay attacks in both circumstances.  The structure of the sequence is defined as a new TLV called "ESN TLV".  It constists of the three fields, Type, Length, and Value, where Value is to be interpreted as two separate fields.  These two fields are known as Extended Session Sequence Number (ESSN; a 64-bit value, which is non-zero and unsigned), and Packet Sequence Number (PSN; a 32-bit value).  A given ESN is associated with a particular Protocol Data Unit (PDU).  A PDU, therefore, "has" an ESSN, which is associated with a monotonically increasing PSN.  If the PSN wraps or if the device needs to be restarted, then the next ESSN must be larger than the previous ESSN.

This is where I see a potential issue, but maybe I'm missing a limit of practicality.  What happens when the ESSN is at 2^64-1 and needs to increment?  Granted that's a large number, so it might be the case that under no practical circumstance would the ESSN ever become that large; but, if that's the case, then it might be best to state so in the draft.

[Uma]: As you stated though this is possible theoretically, it is almost impractical if the right method of encoding is used. For every PDU only 32-bit PSN  increases and only when it wraps ESSN value is incremented. There are two approaches specified in Appendix (only as mere guidelines) of this draft to encode ESSN (64-bit) value.  If timestamps are used (Appendix A.1) to encode this value I don’t see the possibility for this to happen before the lifetime of the router perhaps.


It would be useful to include such information in the draft, as it may not be readily clear to readers that the authors feel the 64-bit value is “intractable” in a sense.

[Uma1]: I shall add a reference to Appendix A.2 perhaps and clarification text surrounding the same.

However, in Appendix A.2, non-volatile storage (Page 7)  it’s clearly stated that –
    “If the non-volatile
   storage is ever repaired or upgraded such that the contents are lost,
   keys MUST be changed to prevent replay attacks.”



In the Appendix, the draft also goes into how timestamps may be leveraged (which are potentially more effective at mitigating replay attacks than are sequences), but does not clearly indicate how neighbors would negotiate the use of timestamps or how exactly the timestamps would be used or which format then must be used (a suggestion to look at RFC5905 is made).

[Uma]: Appendix only proposes to use timestamps as a potential approach to encode the ESSN field of the TLV to get the ever increasing property in all cases. Please  note the approach for replay attack mitigation here is Extended Sequence numbers as specified in the TLV. There could be and may be more mechanisms to solve the same problem at hand but those are beyond the scope of this document. There is *no negotiation* required with neighbor on how to encode this value. This is purely a local matter as long as the “ever increasing” property is maintained, when 32-bit PSN is ever wrapped/cold-restart cases as explained in section 3.1.

It may be the case that I have a less-than-complete understanding of the problem domain.  My understanding is that one of the mitigation circumstances pertains to IS-IS Hello, where neighbors are being brought up and sending IIH to others.  Does this not make it a neighborhood problem?
Even if all neighbors understand the “ever increasing” property, how would they understand which encoding is in place without some additional information?
[Uma1]: There is no need to understand the encoding by neighbor as long as it’s property is maintained (as receiver would verify the unsigned integer value).

It would probably be beneficial to provide some indication in the TLV as to how ESSN is encoded.  Alternatively, if there is a method to deduce the ever increasing property without knowing the ESSN encoding, that should be explained (or at least referenced, if explained elsewhere).

 [Uma1]:  I felt it’s clear - It’s an unsigned integer value per current text at the end of Section 3 (last paragraph). I shall work with you to finalize more clarifying text around this.