Return-Path: <steve.hanna@infineon.com>
X-Original-To: secdir@ietfa.amsl.com
Delivered-To: secdir@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1])
 by ietfa.amsl.com (Postfix) with ESMTP id 13E0312D924;
 Thu,  1 Sep 2016 04:57:11 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -7.468
X-Spam-Level: 
X-Spam-Status: No, score=-7.468 tagged_above=-999 required=5
 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5,
 RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01,
 RP_MATCHES_RCVD=-0.548, SPF_PASS=-0.001]
 autolearn=ham autolearn_force=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 mrofdj92XbrU; Thu,  1 Sep 2016 04:57:09 -0700 (PDT)
Received: from smtp2.infineon.com (smtp2.infineon.com [217.10.52.18])
 (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits))
 (No client certificate requested)
 by ietfa.amsl.com (Postfix) with ESMTPS id 1F69812D911;
 Thu,  1 Sep 2016 04:57:07 -0700 (PDT)
X-SBRS: None
Received: from unknown (HELO mucxv003.muc.infineon.com) ([172.23.11.20])
 by smtp2.infineon.com with ESMTP/TLS/AES256-SHA; 01 Sep 2016 13:57:05 +0200
Received: from MUCSE607.infineon.com (unknown [172.23.7.108])
 by mucxv003.muc.infineon.com (Postfix) with ESMTPS;
 Thu,  1 Sep 2016 13:57:05 +0200 (CEST)
Received: from MUCSE613.infineon.com (172.23.7.84) by MUCSE607.infineon.com
 (172.23.7.108) with Microsoft SMTP Server (TLS) id 15.0.1178.4; Thu, 1 Sep
 2016 13:57:05 +0200
Received: from MUCSE609.infineon.com (172.23.7.110) by MUCSE613.infineon.com
 (172.23.7.84) with Microsoft SMTP Server (TLS) id 15.0.1178.4; Thu, 1 Sep
 2016 13:57:04 +0200
Received: from MUCSE609.infineon.com ([172.23.103.71]) by
 MUCSE609.infineon.com ([172.23.103.71]) with mapi id 15.00.1178.000; Thu, 1
 Sep 2016 13:57:04 +0200
From: <Steve.Hanna@infineon.com>
To: <iesg@ietf.org>, <secdir@ietf.org>,
 <draft-ietf-pals-mc-pon-04.all@ietf.org>
Thread-Topic: secdir review of draft-ietf-pals-mc-pon-04
Thread-Index: AdIERGNJ4u3omnmWQ2+8lMLF4/56GQ==
Date: Thu, 1 Sep 2016 11:57:04 +0000
Message-ID: <9ca53ebc0523442d9be8ab8a1bc3b45e@MUCSE609.infineon.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [172.23.8.247]
Content-Type: multipart/alternative;
 boundary="_000_9ca53ebc0523442d9be8ab8a1bc3b45eMUCSE609infineoncom_"
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/secdir/i64tFG53qAuT7928ObyOBtEaK-g>
Subject: [secdir] secdir review of draft-ietf-pals-mc-pon-04
X-BeenThere: secdir@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: Security Area Directorate <secdir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/secdir>,
 <mailto:secdir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/secdir/>
List-Post: <mailto:secdir@ietf.org>
List-Help: <mailto:secdir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/secdir>,
 <mailto:secdir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 01 Sep 2016 11:57:11 -0000

--_000_9ca53ebc0523442d9be8ab8a1bc3b45eMUCSE609infineoncom_
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

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 co=
mments were written primarily for the benefit of the security area director=
s.  Document editors and WG chairs should treat these comments just like an=
y other last call comments.



Summary: Ready with issues



The security considerations section of this document seems reasonable and c=
orrect. However, I'm concerned that bringing ICCP into the PON has some sec=
urity risks that have not been properly called out and mitigated. As the Se=
curity Considerations section says:



   In many passive optical networks the optical paths between

   OLT and ONTs traverse publicly accessible facilities including

   public attachments (e.g. telephone poles)



Note that I'm pretty sure that ONT should be ONU here and in several other =
places in the Security Considerations.



My concern is that section 4.2 says that "When a fault is detected on its w=
orking PW (e.g., by VCCV BFD), a working OLT SHOULD turn off its associated=
 PON interface and then send an ICCP message with PON State TLV with local =
PON Port State being set to notify the protection OLT of the PON fault." Si=
nce the working PW has failed, the working OLT will presumably send this IC=
CP message to the protection PW over the PON. That means that any other aut=
horized or unauthorized party on the PON could also send such a message. I'=
m not very familiar with ICCP but the Security Considerations section of RF=
C 7275 seems to say that ICCP messages are frequently authenticated only by=
 looking at the source IP address, which is an exceedingly weak method of a=
uthentication susceptible to easy forgery. Using MD5 (as permitted by RFC 7=
275) isn't much better. The impact of permitting attackers to easily forge =
ICCP messages on the PON is not clear to me but it seems that the attacker =
could at least prevent proper failover and maybe force failover or even for=
ce failure. Of course, an attacker on the PON could also just jam the PON b=
y setting their laser always on but ICCP attacks would be much trickier to =
detect. I would suggest that TCP-AO be required at least for ICCP messages =
sent or received on the PON.


Two minor typos:


1)      In section 3, "protect OLTs" should be "protection OLTs".


2)      In the Security Considerations, "ONT" should be "ONU". At least, I =
assume so. If not, ONT should be defined.

Thanks for your consideration,

Steve


--_000_9ca53ebc0523442d9be8ab8a1bc3b45eMUCSE609infineoncom_
Content-Type: text/html; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

<html xmlns:v=3D"urn:schemas-microsoft-com:vml" xmlns:o=3D"urn:schemas-micr=
osoft-com:office:office" xmlns:w=3D"urn:schemas-microsoft-com:office:word" =
xmlns:dt=3D"uuid:C2F41010-65B3-11d1-A29F-00AA00C14882" xmlns:m=3D"http://sc=
hemas.microsoft.com/office/2004/12/omml" xmlns=3D"http://www.w3.org/TR/REC-=
html40">
<head>
<meta http-equiv=3D"Content-Type" content=3D"text/html; charset=3Dus-ascii"=
>
<meta name=3D"Generator" content=3D"Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
	{font-family:Calibri;
	panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
	{margin:0in;
	margin-bottom:.0001pt;
	font-size:11.0pt;
	font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
	{mso-style-priority:99;
	color:blue;
	text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
	{mso-style-priority:99;
	color:purple;
	text-decoration:underline;}
p.MsoPlainText, li.MsoPlainText, div.MsoPlainText
	{mso-style-priority:99;
	mso-style-link:"Plain Text Char";
	margin:0in;
	margin-bottom:.0001pt;
	font-size:11.0pt;
	font-family:"Calibri","sans-serif";}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
	{mso-style-priority:34;
	margin-top:0in;
	margin-right:0in;
	margin-bottom:0in;
	margin-left:.5in;
	margin-bottom:.0001pt;
	font-size:11.0pt;
	font-family:"Calibri","sans-serif";}
span.EmailStyle17
	{mso-style-type:personal-compose;
	font-family:"Calibri","sans-serif";
	color:windowtext;}
span.PlainTextChar
	{mso-style-name:"Plain Text Char";
	mso-style-priority:99;
	mso-style-link:"Plain Text";
	font-family:"Calibri","sans-serif";}
.MsoChpDefault
	{mso-style-type:export-only;
	font-family:"Calibri","sans-serif";}
@page WordSection1
	{size:8.5in 11.0in;
	margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
	{page:WordSection1;}
/* List Definitions */
@list l0
	{mso-list-id:11424535;
	mso-list-type:hybrid;
	mso-list-template-ids:827492486 67698705 67698713 67698715 67698703 676987=
13 67698715 67698703 67698713 67698715;}
@list l0:level1
	{mso-level-text:"%1\)";
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level2
	{mso-level-number-format:alpha-lower;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level3
	{mso-level-number-format:roman-lower;
	mso-level-tab-stop:none;
	mso-level-number-position:right;
	text-indent:-9.0pt;}
@list l0:level4
	{mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level5
	{mso-level-number-format:alpha-lower;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level6
	{mso-level-number-format:roman-lower;
	mso-level-tab-stop:none;
	mso-level-number-position:right;
	text-indent:-9.0pt;}
@list l0:level7
	{mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level8
	{mso-level-number-format:alpha-lower;
	mso-level-tab-stop:none;
	mso-level-number-position:left;
	text-indent:-.25in;}
@list l0:level9
	{mso-level-number-format:roman-lower;
	mso-level-tab-stop:none;
	mso-level-number-position:right;
	text-indent:-9.0pt;}
ol
	{margin-bottom:0in;}
ul
	{margin-bottom:0in;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext=3D"edit" spidmax=3D"1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext=3D"edit">
<o:idmap v:ext=3D"edit" data=3D"1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang=3D"EN-US" link=3D"blue" vlink=3D"purple">
<div class=3D"WordSection1">
<p class=3D"MsoPlainText">I have reviewed this document as part of the secu=
rity directorate's ongoing effort to review all IETF documents being proces=
sed by the IESG.&nbsp; These comments were written primarily for the benefi=
t of the security area directors.&nbsp; Document
 editors and WG chairs should treat these comments just like any other last=
 call comments.<o:p></o:p></p>
<p class=3D"MsoPlainText"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoPlainText">Summary: Ready with issues<o:p></o:p></p>
<p class=3D"MsoPlainText"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoPlainText">The security considerations section of this docum=
ent seems reasonable and correct. However, I&#8217;m concerned that bringin=
g ICCP into the PON has some security risks that have not been properly cal=
led out and mitigated. As the Security Considerations
 section says:<o:p></o:p></p>
<p class=3D"MsoPlainText"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoPlainText">&nbsp;&nbsp; In many passive optical networks the=
 optical paths between<o:p></o:p></p>
<p class=3D"MsoPlainText">&nbsp;&nbsp; OLT and ONTs traverse publicly acces=
sible facilities including<o:p></o:p></p>
<p class=3D"MsoPlainText">&nbsp;&nbsp; public attachments (e.g. telephone p=
oles)<o:p></o:p></p>
<p class=3D"MsoPlainText"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoPlainText">Note that I&#8217;m pretty sure that ONT should b=
e ONU here and in several other places in the Security Considerations.<o:p>=
</o:p></p>
<p class=3D"MsoPlainText"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoPlainText">My concern is that section 4.2 says that &#8220;W=
hen a fault is detected on its working PW (e.g., by VCCV BFD), a working OL=
T SHOULD turn off its associated PON interface and then send an ICCP messag=
e with PON State TLV with local PON Port
 State being set to notify the protection OLT of the PON fault.&#8221; Sinc=
e the working PW has failed, the working OLT will presumably send this ICCP=
 message to the protection PW over the PON. That means that any other autho=
rized or unauthorized party on the PON
 could also send such a message. I&#8217;m not very familiar with ICCP but =
the Security Considerations section of RFC 7275 seems to say that ICCP mess=
ages are frequently authenticated only by looking at the source IP address,=
 which is an exceedingly weak method of
 authentication susceptible to easy forgery. Using MD5 (as permitted by RFC=
 7275) isn&#8217;t much better. The impact of permitting attackers to easil=
y forge ICCP messages on the PON is not clear to me but it seems that the a=
ttacker could at least prevent proper
 failover and maybe force failover or even force failure. Of course, an att=
acker on the PON could also just jam the PON by setting their laser always =
on but ICCP attacks would be much trickier to detect. I would suggest that =
TCP-AO be required at least for
 ICCP messages sent or received on the PON.<o:p></o:p></p>
<p class=3D"MsoPlainText"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">Two minor typos:<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoListParagraph" style=3D"text-indent:-.25in;mso-list:l0 level=
1 lfo1"><![if !supportLists]><span style=3D"mso-list:Ignore">1)<span style=
=3D"font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
</span></span><![endif]>In section 3, &#8220;protect OLTs&#8221; should be =
&#8220;protection OLTs&#8221;.<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoListParagraph" style=3D"text-indent:-.25in;mso-list:l0 level=
1 lfo1"><![if !supportLists]><span style=3D"mso-list:Ignore">2)<span style=
=3D"font:7.0pt &quot;Times New Roman&quot;">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
</span></span><![endif]>In the Security Considerations, &#8220;ONT&#8221; s=
hould be &#8220;ONU&#8221;. At least, I assume so. If not, ONT should be de=
fined.<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">Thanks for your consideration,<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">Steve<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
</div>
</body>
</html>

--_000_9ca53ebc0523442d9be8ab8a1bc3b45eMUCSE609infineoncom_--

