Re: [secdir] Security review of draft-ietf-rtcweb-fec-08 (was Re: Security review of draft-ietf-perc-srtp-ekt-diet-08)

Justin Uberti <justin@uberti.name> Sat, 09 February 2019 01:20 UTC

Return-Path: <juberti@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 052AB131069; Fri, 8 Feb 2019 17:20:57 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.879
X-Spam-Level:
X-Spam-Status: No, score=-1.879 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, FREEMAIL_FORGED_FROMDOMAIN=0.018, FREEMAIL_FROM=0.001, HEADER_FROM_DIFFERENT_DOMAINS=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, URIBL_BLOCKED=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 nTaJnYpocV80; Fri, 8 Feb 2019 17:20:54 -0800 (PST)
Received: from mail-lf1-f51.google.com (mail-lf1-f51.google.com [209.85.167.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 61605130EE9; Fri, 8 Feb 2019 17:20:54 -0800 (PST)
Received: by mail-lf1-f51.google.com with SMTP id e27so3887990lfj.8; Fri, 08 Feb 2019 17:20:54 -0800 (PST)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=S+4dh1fFQE5b+xjVzp6thWnWkDWGaOm4t3qlv1cdaiw=; b=EoD7TDcrWErY3XHjLso4Bad4H2FrK4dyYpJZV6ET0KKGh+BK1yq3qxfbUoBCDxF2G+ DIUkIq1RS6NO31eXfauR5BkLAmnjWZL1v8/qRAbRLPkeE8wZ93Wo8OTOQr6RV52Rw92x 1vAnI6ZUTZJrWn5jsKWmSgn0rIa9uiP5+DX0HtL30oESppOPj2zENWT0xWt8yRjIF3RX 5uQlkLVCAmLjqd9kh7ELsffkGS8UxPbT3OFQorDXWdz7HewIoWjsieADwnZpvdZpZfs7 h+0a/aDND5O+E4dODNeXsLN0bfwuL2fHMA54/ORhJsopQtjRqzE+jIhtfH+XBlqqgwJI SWZQ==
X-Gm-Message-State: AHQUAuah2kiH4rT8wf4sHNDNXinLzVjgeJeiJigZ4HrOKNN2u9RHuV0b uyj3dVC2bFsvOXFRfC/wruyllZRC2SfrAcZMXU8=
X-Google-Smtp-Source: AHgI3IbMyMxx2o2BK02ZwXnMzLaN3iLBjfgybjsuT+qrtoC54AFpHYqnNEv0jwuJdp9fHp4Tjxd9MSljLT/gmvHStrM=
X-Received: by 2002:ac2:4474:: with SMTP id y20mr3382017lfl.40.1549675252259; Fri, 08 Feb 2019 17:20:52 -0800 (PST)
MIME-Version: 1.0
References: <201902010742.x117gdGm030846@rumpleteazer.rhmr.com> <F44FA6A0-4599-4BFF-8BEB-C67774714762@nostrum.com>
In-Reply-To: <F44FA6A0-4599-4BFF-8BEB-C67774714762@nostrum.com>
From: Justin Uberti <justin@uberti.name>
Date: Fri, 08 Feb 2019 17:20:41 -0800
Message-ID: <CALe60zD=OeTfjof3Q5UqnJRHsAQC-kS1oZYaQZ5HahAVOJgVKQ@mail.gmail.com>
To: Ben Campbell <ben@nostrum.com>
Cc: Hilarie Orman <hilarie@purplestreak.com>, The IESG <iesg@ietf.org>, secdir@ietf.org, draft-ietf-rtcweb-fec.all@tools.ietf.org
Content-Type: multipart/alternative; boundary="000000000000eda39405816be2c0"
Archived-At: <https://mailarchive.ietf.org/arch/msg/secdir/q1pDusf--Zr6TjyXJeVU-aUueEs>
Subject: Re: [secdir] Security review of draft-ietf-rtcweb-fec-08 (was Re: Security review of draft-ietf-perc-srtp-ekt-diet-08)
X-BeenThere: secdir@ietf.org
X-Mailman-Version: 2.1.29
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: Sat, 09 Feb 2019 01:20:57 -0000

On Fri, Feb 1, 2019 at 2:49 PM Ben Campbell <ben@nostrum.com> wrote:

> Hi,
>
> Please note that this review is for draft-ietf-rtcweb-fec-08, not the PERC
> draft referenced in the subject.
>
> Thanks!
>
> Ben.
>
>
> > On Feb 1, 2019, at 1:42 AM, Hilarie Orman <hilarie@purplestreak.com>
> wrote:
> >
> > Security Review of WebRTC Forward Error Correction Requirements
> > draft-ietf-rtcweb-fec-08
> >
> > Do not be alarmed.  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 comments were written primarily
> > for the benefit of the security area directors.  Document editors and
> > WG chairs should treat these comments just like any other last call
> > comments.
> >
> > The document describes the appropriate uses of FEC for web content when
> > using WebRTC.  It also describes how to indicate that FEC is being used.
> >
> > The Security Considerations mention the possibility of additional network
> > congestion when using FEC.  Although this can be a problem, I do not
> think
> > it is a security issue, thus it does not belong in this section.
>

Understood. I think this paragraph could easily be moved to the preceding
section.

> >
> > There is a security-related issue wrt to FEC and encryption.  If the
> > error model is that message blocks may be lost but not altered in
> > transit, then FEC with encryption is fine.  But if FEC is added for
> > the purpose of correcting corrupted bits in a message block, then it
> > is important that FEC is done after encryption.  The draft seems to
> > ignore the issue, and it also seems to recommend a processing scheme
> > that would result in encryption of the FEC data.  If there is a body
> > of practice for other IETF FEC protocols that explains these issues,
> > an explicit reference to it in the Security Considerations would be
> > very helpful.
>
> FEC is added specifically to protect against lost blocks. Any corruption
of the blocks will be detected by the decryption procedure, and such blocks
will be discarded.

There is a reference to RFC 3711, which stipulates the fec-then-encrypt
ordering. RFC 3711 is admittedly terse on this subject, but it is quite
clear about the ordering.