Re: [Gen-art] Gen-ART review for draft-ietf-ipsecme-esp-null-heuristics-05.txt

Tero Kivinen <kivinen@iki.fi> Fri, 26 February 2010 12:08 UTC

Return-Path: <kivinen@iki.fi>
X-Original-To: gen-art@core3.amsl.com
Delivered-To: gen-art@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id A73BE3A7984; Fri, 26 Feb 2010 04:08:23 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.522
X-Spam-Level:
X-Spam-Status: No, score=-2.522 tagged_above=-999 required=5 tests=[AWL=0.077, BAYES_00=-2.599]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 7QyKrmDAiI9Q; Fri, 26 Feb 2010 04:08:22 -0800 (PST)
Received: from mail.kivinen.iki.fi (fireball.acr.fi [83.145.195.1]) by core3.amsl.com (Postfix) with ESMTP id 086B63A878A; Fri, 26 Feb 2010 04:08:21 -0800 (PST)
Received: from fireball.kivinen.iki.fi (localhost [127.0.0.1]) by mail.kivinen.iki.fi (8.14.3/8.14.3) with ESMTP id o1QCAWR4012499 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Fri, 26 Feb 2010 14:10:32 +0200 (EET)
Received: (from kivinen@localhost) by fireball.kivinen.iki.fi (8.14.3/8.12.11) id o1QCAV5x027064; Fri, 26 Feb 2010 14:10:31 +0200 (EET)
X-Authentication-Warning: fireball.kivinen.iki.fi: kivinen set sender to kivinen@iki.fi using -f
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Message-ID: <19335.47671.472054.767654@fireball.kivinen.iki.fi>
Date: Fri, 26 Feb 2010 14:10:31 +0200
From: Tero Kivinen <kivinen@iki.fi>
To: Spencer Dawkins <spencer@wonderhamster.org>
In-Reply-To: <F482F25C6B0B49159A83DBDB0D0045AB@china.huawei.com>
References: <7B2204AACA664DA2988E253FE2FCF568@china.huawei.com> <19323.49404.562705.833947@fireball.kivinen.iki.fi> <F482F25C6B0B49159A83DBDB0D0045AB@china.huawei.com>
X-Mailer: VM 7.19 under Emacs 21.4.1
X-Edit-Time: 11 min
X-Total-Time: 15 min
Cc: General Area Review Team <gen-art@ietf.org>, ietf@ietf.org, draft-ietf-ipsecme-esp-null-heuristics@tools.ietf.org
Subject: Re: [Gen-art] Gen-ART review for draft-ietf-ipsecme-esp-null-heuristics-05.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 26 Feb 2010 12:08:23 -0000

Spencer Dawkins writes:
> I don't feel strongly about this, but do suggest s/uses the same policy/uses 
> the same policy, and that changes to that single policy can be coordinated 
> throughout the administrative domain/, to capture what you said in your 
> response, which I found helpful.

Changed that sentence as you suggested and the full sentence is now:

    Also, such a solution might require some kind of centralized
    policy management to make sure everybody in an administrative
    domain uses the same policy, and that changes to that single
    policy can be coordinated throughout the administrative
    domain.

> I saw that this isn't a 2119 document, but it's hard for people who are 
> familiar with 2119 language to ignore the words when they are in lower case 
> :D
> 
> I really liked the explanation you gave in your response here. I suggest 
> picking one of "can/will/should", probably "can", and including your 
> response in the document.
> 
> The resulting text (with some additional edits) might look something like 
> "As ESP-NULL heuristics need to know the same protocols as a deep inspection 
> device, an ESP-NULL instance of an unknown protocol can be handled the same 
> way as a cleartext instance of the same unknown protocol.

Changed to the text you suggested.

> OK, that's the part that was missing for me. I would suggest "the packet has 
> already transited a NAT box which changed the IP addresses but assumed any 
> ESP payload was encryped and did not recalculate the transport checksums 
> with the new IP addresses. Thus"

Made the changes you requested, but used "fix" instead "recalculate"
as most of the nats do not recalculate checksum, but do incremental
update on it. The whole text section is now:

      The most obvious field, TCP checksum, might not be usable, as it
      is possible that the packet has already transited a NAT box
      which changed the IP addresses but assumed any ESP payload was
      encrypted and did not fix the transport checksums with the new
      IP addresses. Thus the IP numbers used in the checksum are
      wrong, thus the checksum is wrong.

> This explanation is helpful. I'd suggest adding "This hueristic is most 
> helpful when only one packet is outstanding. For example, if a TCP SYN 
> packet is lost, the next packet would always be a retransmission of the SYN 
> packet".

Changed (with minor edits) as you suggested. The full text is now:

      One good method of detection is if a packet is dropped then the
      next packet will most likely be a retransmission of the previous
      packet. Thus if two packets are received with the same source,
      and destination port numbers, and where sequence numbers are
      either same or right after each other, then it's likely a TCP
      packet has been correctly detected. This heuristics is most
      helpful when only one packet is outstanding. For example, if a
      TCP SYN packet is lost (or dropped because of policy), the next
      packet would always be a retransmission of the same TCP SYN
      packet.

> Your explanation was very helpful. I'd suggest
> 
> "Forcing use of ESP-NULL everywhere inside the enterprise, so that 
> accounting, logging, network monitoring, and intrusion detection all work, 
> increases the risk of sending confidential information where eavesdroppers 
> can see it" 

Changed to use your text.

I updated the draft and submitted 06 version which includes these
changes.
-- 
kivinen@iki.fi