[secdir] SECDIR review draft-ietf-mpls-tp-cc-cv-rdi-05.txt

Donald Eastlake <d3e3e3@gmail.com> Wed, 13 July 2011 15:51 UTC

Return-Path: <d3e3e3@gmail.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 E179911E8132; Wed, 13 Jul 2011 08:51:50 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -104.477
X-Spam-Level:
X-Spam-Status: No, score=-104.477 tagged_above=-999 required=5 tests=[AWL=-0.878, BAYES_00=-2.599, RCVD_IN_DNSWL_LOW=-1, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id mt0lDbR7gunX; Wed, 13 Jul 2011 08:51:50 -0700 (PDT)
Received: from mail-yx0-f172.google.com (mail-yx0-f172.google.com [209.85.213.172]) by ietfa.amsl.com (Postfix) with ESMTP id 9A41421F8686; Wed, 13 Jul 2011 08:51:48 -0700 (PDT)
Received: by yxp4 with SMTP id 4so3058116yxp.31 for <multiple recipients>; Wed, 13 Jul 2011 08:51:48 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:from:date:message-id:subject:to:content-type :content-transfer-encoding; bh=gaWsgtT9wf5BF/k3wRLBhy9VY+sv1KC34xqNVYWixgc=; b=HoX8PE+5U/SDdFlEqSdMiybMmBDemBcjFPQ3/POpcwp9mk0Uo+qmgw5gbwfzVTKlyP 0hmpPMxbWgG5sdbfV6ubtYAXjSgXQFMk/PjLiEzp8YlBnpiQEEeASh64I6YYyFXtT3Z/ cKnu33x/SFaPZtUJA+2Mj9ePjRUI4LhkY4t7I=
Received: by 10.150.12.9 with SMTP id 9mr1338953ybl.328.1310572308092; Wed, 13 Jul 2011 08:51:48 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.151.144.3 with HTTP; Wed, 13 Jul 2011 08:51:28 -0700 (PDT)
From: Donald Eastlake <d3e3e3@gmail.com>
Date: Wed, 13 Jul 2011 11:51:28 -0400
Message-ID: <CAF4+nEHEJ0Z2m6Th39wmdE-a_TsZwWss+tXGLaz5=GDEw0WPpg@mail.gmail.com>
To: iesg@ietf.org, secdir@ietf.org, draft-ietf-mpls-tp-cc-cv-rdi.all@tools.ietf.org
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: quoted-printable
Subject: [secdir] SECDIR review draft-ietf-mpls-tp-cc-cv-rdi-05.txt
X-BeenThere: secdir@ietf.org
X-Mailman-Version: 2.1.12
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: <http://www.ietf.org/mail-archive/web/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: Wed, 13 Jul 2011 15:51:51 -0000

I have reviewed this document as part of the security directorate's
ongoing effort to review all IETF documents being processed by the
IESG. Document editors and WG chairs should treat these comments just
like any other last call comments.

SECURITY

The Security Considerations section is brief but covers most of the
considerations that specifically occurred to me in reading this draft.

In Section 3.5.2 there are various fields that MUST be unique. Are
there security consequences if they are not?

MINOR

A reference to RFC 6291 should probably be included.

Section 2.1: CC is not listed. P/F is not listed.

Section 3.7.4.1: I believe all the figure numbers in this section are wrong.

EDITORIAL

Abstract: "integrity of the continuity" seems redundant. Just
"continuity" is better.

Abstract: "any loss of continuity defect". So you lost a "continuity
defect", did you? Slipper little guys, aren't they? Maybe you mean
"any loss-of-continuity defect".

Introduction: I don't get the reason for the double references like
"[12][12]" and "[13][13]".

Introduction: Missing commas: "the same
   continuity check (CC) proactive continuity verification (CV) and
   remote defect indication (RDI) capabilities" should be "the same
   continuity check (CC), proactive continuity verification (CV). and
   remote defect indication (RDI) capabilities".

Section 2.1: This is just a personal preference of mine but I think it
is best to explain a little more than you think you need to. So I
would include entries for MPLS, OAM, and PDU.

Figure 4, Figure 6: There should be a blank line after the Figure label.

Figure 5, Figure 7, Figure 8: Figures should not be broken over page boundaries.

Section 4, Section 6: No blank line before Section header.

Section 4: Ends with a list of length 1. List constructs should not be
used for lists of length one.

Overall: As in many such documents, I believe that acronyms are
overused and the document would be improved by more frequently
spelling things out. For example, p2p occurs only twice in the
document, the first time when it is also spelled out and only one
other use. I believe such rarely used acronyms should generally be
spelled out for all of their tiny number of uses.

Thanks,
Donald
=============================
 Donald E. Eastlake 3rd   +1-508-333-2270 (cell)
 155 Beaver Street
 Milford, MA 01757 USA
 d3e3e3@gmail.com