Re: [AVTCORE] WG last call for draft-ietf-avtcore-srtp-ekt-01

"Peck, Michael A" <mpeck@mitre.org> Wed, 13 November 2013 19:09 UTC

Return-Path: <mpeck@mitre.org>
X-Original-To: avt@ietfa.amsl.com
Delivered-To: avt@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D4A8521E80CA for <avt@ietfa.amsl.com>; Wed, 13 Nov 2013 11:09:23 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.598
X-Spam-Level:
X-Spam-Status: No, score=-6.598 tagged_above=-999 required=5 tests=[AWL=0.001, BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 8UAdH78bV0lC for <avt@ietfa.amsl.com>; Wed, 13 Nov 2013 11:09:16 -0800 (PST)
Received: from smtpksrv1.mitre.org (smtpksrv1.mitre.org [198.49.146.77]) by ietfa.amsl.com (Postfix) with ESMTP id 1DC6921E80A8 for <avt@ietf.org>; Wed, 13 Nov 2013 11:09:16 -0800 (PST)
Received: from smtpksrv1.mitre.org (localhost.localdomain [127.0.0.1]) by localhost (Postfix) with SMTP id A32771F083B; Wed, 13 Nov 2013 14:09:15 -0500 (EST)
Received: from IMCCAS03.MITRE.ORG (imccas03.mitre.org [129.83.29.80]) by smtpksrv1.mitre.org (Postfix) with ESMTP id 8E7D61F06FE; Wed, 13 Nov 2013 14:09:15 -0500 (EST)
Received: from IMCMBX04.MITRE.ORG ([169.254.4.201]) by IMCCAS03.MITRE.ORG ([129.83.29.80]) with mapi id 14.03.0158.001; Wed, 13 Nov 2013 14:09:15 -0500
From: "Peck, Michael A" <mpeck@mitre.org>
To: Magnus Westerlund <magnus.westerlund@ericsson.com>, IETF AVTCore WG <avt@ietf.org>
Thread-Topic: [AVTCORE] WG last call for draft-ietf-avtcore-srtp-ekt-01
Thread-Index: AQHO0Wg0RjntMMFOlkmogX2Nb7ZcD5ojpN2A
Date: Wed, 13 Nov 2013 19:09:14 +0000
Message-ID: <CEA93521.9BBF%mpeck@mitre.org>
In-Reply-To: <526A3FF9.1060607@ericsson.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/14.3.8.130913
x-originating-ip: [128.29.194.119]
Content-Type: text/plain; charset="iso-8859-1"
Content-ID: <7E10E20F86E2AA4991D2DF2CBBF9C879@imc.mitre.org>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Cc: "draft-ietf-avtcore-srtp-ekt@tools.ietf.org" <draft-ietf-avtcore-srtp-ekt@tools.ietf.org>
Subject: Re: [AVTCORE] WG last call for draft-ietf-avtcore-srtp-ekt-01
X-BeenThere: avt@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Audio/Video Transport Core Maintenance <avt.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/avt>, <mailto:avt-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/avt>
List-Post: <mailto:avt@ietf.org>
List-Help: <mailto:avt-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/avt>, <mailto:avt-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 13 Nov 2013 19:09:25 -0000

Hi - I have a few minor comments:

There are several references to AES_128 throughout the document (I counted
6) including a statement that "AES_128 is the default value for the EKT
cipher."  It looks like these are leftovers from a previous draft and
should be removed, as AES_128 is no longer a defined EKT cipher.  Section
3.4.1 states that AESKW_128 is now the default cipher.

Section 2.1:
The ROC and ISN paragraphs both state "It is included in the EKT
plaintext, but does not appear on the wire."
The phrase "but does not appear on the wire" seems confusing to me - I'd
suggest removing it?
The EKT_Plaintext paragraph already states that the EKT plaintext doesn't
appear on the wire.
Besides, they sort-of do appear on the wire, but they're in ciphertext
form?

Section 2.2:
"for use in processing inbound SRTCP traffic"
Should this be "for use in processing inbound SRTCP and SRTP traffic"?

Section 2.2.2
typo: "containing and EKT" should be "containing an EKT"
typo: "Steps 2, 3, 4 5" should be "Steps 2, 3, 4, 5"

Section 3.4.1:
typo: "with in" should be "within"
Same sentence: should "SRTCP packets" be "SRTCP and SRTP packets"?

Section 3.4.2:
Should "SRTCP packets" be "SRTCP and SRTP packets"?

Section 3.5:
typo: "Section (newline) Section 3.4" should just be "Section 3.4"

Mike Peck

On 10/25/13 5:55 AM, "Magnus Westerlund" <magnus.westerlund@ericsson.com>
wrote:

>WG,
>
>This starts a 3-week (due to the IETF meeting) WG last call of Encrypted
>Key Transport for Secure RTP with intended status Proposed Standard.
>Please provide any feedback by the 15th of November.
>
>The draft
>https://datatracker.ietf.org/doc/draft-ietf-avtcore-srtp-ekt/
>
>Cheers
>
>Magnus Westerlund
>
>----------------------------------------------------------------------
>Multimedia Technologies, Ericsson Research EAB/TVM
>----------------------------------------------------------------------
>Ericsson AB                | Phone  +46 10 7148287
>Färögatan 6                | Mobile +46 73 0949079
>SE-164 80 Stockholm, Sweden| mailto: magnus.westerlund@ericsson.com
>----------------------------------------------------------------------
>
>_______________________________________________
>Audio/Video Transport Core Maintenance
>avt@ietf.org
>https://www.ietf.org/mailman/listinfo/avt