Re: Last Call: <draft-ietf-6man-udpchecksums-06.txt> (IPv6 and UDP Checksums for Tunneled Packets) to Proposed Standard

Magnus Westerlund <magnus.westerlund@ericsson.com> Tue, 18 December 2012 10:31 UTC

Return-Path: <magnus.westerlund@ericsson.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 9742B21F884C for <ietf@ietfa.amsl.com>; Tue, 18 Dec 2012 02:31:54 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -106.213
X-Spam-Level:
X-Spam-Status: No, score=-106.213 tagged_above=-999 required=5 tests=[AWL=0.036, BAYES_00=-2.599, HELO_EQ_SE=0.35, RCVD_IN_DNSWL_MED=-4, 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 y9Wk29uFxsUp for <ietf@ietfa.amsl.com>; Tue, 18 Dec 2012 02:31:54 -0800 (PST)
Received: from mailgw7.ericsson.se (mailgw7.ericsson.se [193.180.251.48]) by ietfa.amsl.com (Postfix) with ESMTP id 819CD21F8840 for <ietf@ietf.org>; Tue, 18 Dec 2012 02:31:53 -0800 (PST)
X-AuditID: c1b4fb30-b7f736d0000010de-31-50d046122af8
Received: from esessmw0197.eemea.ericsson.se (Unknown_Domain [153.88.253.124]) by mailgw7.ericsson.se (Symantec Mail Security) with SMTP id CD.39.04318.21640D05; Tue, 18 Dec 2012 11:31:46 +0100 (CET)
Received: from [127.0.0.1] (153.88.115.8) by esessmw0197.eemea.ericsson.se (153.88.115.88) with Microsoft SMTP Server id 8.3.279.1; Tue, 18 Dec 2012 11:31:46 +0100
Message-ID: <50D04611.5040807@ericsson.com>
Date: Tue, 18 Dec 2012 11:31:45 +0100
From: Magnus Westerlund <magnus.westerlund@ericsson.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:17.0) Gecko/17.0 Thunderbird/17.0
MIME-Version: 1.0
To: John William Atwood <william.atwood@concordia.ca>
Subject: Re: Last Call: <draft-ietf-6man-udpchecksums-06.txt> (IPv6 and UDP Checksums for Tunneled Packets) to Proposed Standard
References: <20121212011555.15698.49063.idtracker@ietfa.amsl.com> <50CFB119.1010205@concordia.ca>
In-Reply-To: <50CFB119.1010205@concordia.ca>
X-Enigmail-Version: 1.4.6
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 8bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrCLMWRmVeSWpSXmKPExsUyM+Jvja6Q24UAg8n7BS2ebZzPYnFn81pm i94HM1ktmhZfYXVg8fhy5BWjx85Zd9k9liz5yeRxd75VAEsUl01Kak5mWWqRvl0CV8axD9eY Cz5KVcx4oNjAeFy0i5GTQ0LAROLc0+vMELaYxIV769m6GLk4hAROMkr8bnkG5SxnlPi0aScb SBWvgLbE6cdL2UFsFgFVia9zZ4DF2QQsJG7+aASzRQV8JWbt+cUEUS8ocXLmExYQW0TAVOLS tdtg25gFCiU2XJzKCLJAWKCJUeL1qwdgzUICiRKbzreBNXAK6Eicfr+DCeI8SYlF0zpZIJr1 JKZcbWGEsOUlmrfOZobo1ZZoaOpgncAoNAvJ7llIWmYhaVnAyLyKkT03MTMnvdx8EyMwpA9u +W2wg3HTfbFDjNIcLErivOGuFwKEBNITS1KzU1MLUovii0pzUosPMTJxcEoBA1F62+lXn575 10980rZ+fptZzNqVXgy5Fw5lsphlXDk17Vfcs797FzA7PD212NLxrbril+uJ06ayb5f7vV+K +0wiy8LzH/xnHL/sP1f7/D2Duc8fmRpVreCxlJ7fEtk/8xa38ez3ARdNFS7mpaaWeZU9eV+r cNe6QvP3ze/TLrlfVrPN3X+seZsSS3FGoqEWc1FxIgBkv7mYNwIAAA==
Cc: ietf@ietf.org, Philip Chimento <Philip.Chimento@jhuapl.edu>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 18 Dec 2012 10:31:54 -0000

Thanks John

The editorial nits we will take care of in an update. Many of these are
new due to the significant changes, and this is in fact the first chance
to comment on them. Thus, do not feel bad for it being IETF last call.


On 2012-12-18 00:56, John William Atwood wrote:
> Comments on draft-ietf-6man-udpchecksums-06
> 
> 

> 
> Bullet 3, sub-bullet 1.  What does it mean, "the 5-tuple with a zero
> checksum enabled"?  I _believe_ that you are trying to indicate that the
> delivery mechanism has a data structure with the 5-tuple and an
> indication that this context has enabled zero checksums and that the
> incoming packet has a zero checksum and that the 5-tuple matches the one
> in the data structure.  I am not sure what to suggest, but the potential
> for erroneous coding seems high to me.

A packet matching an existing context which has zero checksum enables
receives a packet not intended for this context, but due to some
corruption event of the address information it arrives in this context.

You are correct that there is basically nothing you can do against this
unless you actually have a checksum to detect the corruption. In the
case of a tunneling protocol is is also likely that the tunneling
protocol is so thin, i.e. no or few headers that using them to detect
that this packet is not intended for this context is difficult.
Correctly I think this is something you have to accept when using
non-checksum protected receivers. It can occur but more than 10000 times
as common when using Internet checksums.

> 
> Bullet 3, sub-bullet 1.  If a payload is matched to the wrong context,
> why is it delivered as if it were correct?  Is there something that I am
> missing here?

The receiving context interprets the packet according to the rules of
the existing context. If nothing in this processing is tripped then what
was in that packet will be used as a valid packet in that context.

To be clear here, we are describing the cases using full knowledge of
what occurs, but the receiving node can't see the difference between a
corrupted zero-checksummed packet and a correct one. That is what it
requires the checksum to do.

> 
> Page 9, replacement text, para 1, lines 6-9.  This sentence has no
> subject.  I suggest the following text:
> 
> "As an alternative usage for some protocols, such as protocols that use
> UDP as a tunnel encapsulation, the zero-checksum mode MAY be enabled for
> specific sets of ports."
> 
> I note, however, that the phrase "zero-checksum mode" is probably not
> defined anywhere in RFC2460, so it may be necessary to be more specific
> here: "a packet with a zero checksum MAY be allowed for specific sets of
> ports."

The zero-checksum mode is intended to be defined by this text, where the
zero-checksum mode is defined by the rules in draft-ietf-6man-udpzero. I
have some difficulties determining what changes would improve this.

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
----------------------------------------------------------------------