Re: [TLS] Secdir review of draft-ietf-tls-session-hash-04

Karthikeyan Bhargavan <karthik.bhargavan@gmail.com> Thu, 16 April 2015 07:07 UTC

Return-Path: <karthik.bhargavan@gmail.com>
X-Original-To: expand-draft-ietf-tls-session-hash.all@virtual.ietf.org
Delivered-To: tls@ietfa.amsl.com
Received: by ietfa.amsl.com (Postfix, from userid 65534) id 841841B2AF2; Thu, 16 Apr 2015 00:07:12 -0700 (PDT)
X-Original-To: xfilter-draft-ietf-tls-session-hash.all@ietfa.amsl.com
Delivered-To: xfilter-draft-ietf-tls-session-hash.all@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 3B9EB1B2AF1 for <xfilter-draft-ietf-tls-session-hash.all@ietfa.amsl.com>; Thu, 16 Apr 2015 00:07:12 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.883
X-Spam-Level:
X-Spam-Status: No, score=-0.883 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_ADSP_CUSTOM_MED=0.001, FREEMAIL_FROM=0.001, HELO_EQ_FR=0.35, SPF_SOFTFAIL=0.665] autolearn=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 2U-q6G9anXek for <xfilter-draft-ietf-tls-session-hash.all@ietfa.amsl.com>; Thu, 16 Apr 2015 00:07:11 -0700 (PDT)
Received: from zinfandel.tools.ietf.org (zinfandel.tools.ietf.org [IPv6:2001:1890:123a::1:2a]) (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 371DB1B2AEF for <draft-ietf-tls-session-hash.all@ietf.org>; Thu, 16 Apr 2015 00:07:11 -0700 (PDT)
Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:41475) by zinfandel.tools.ietf.org with esmtps (TLS1.0:RSA_ARCFOUR_128_SHA1:128) (Exim 4.82_1-5b7a7c0-XX) (envelope-from <karthik.bhargavan@gmail.com>) id 1YidtB-0000Kn-Uh for draft-ietf-tls-session-hash.all@tools.ietf.org; Thu, 16 Apr 2015 00:07:10 -0700
X-IronPort-AV: E=Sophos;i="5.11,586,1422918000"; d="scan'208";a="111699762"
Received: from 166.92.69.86.rev.sfr.net (HELO [192.168.1.44]) ([86.69.92.166]) by mail3-relais-sop.national.inria.fr with ESMTP/TLS/AES128-SHA; 16 Apr 2015 09:06:43 +0200
Content-Type: text/plain; charset=windows-1252
Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\))
From: Karthikeyan Bhargavan <karthik.bhargavan@gmail.com>
In-Reply-To: <CAFOuuo7MA9Rd7zbsic8pJgShYF8NRgnddyC5JHfA6hGcGkkAfA@mail.gmail.com>
Date: Thu, 16 Apr 2015 09:06:42 +0200
Content-Transfer-Encoding: quoted-printable
Message-Id: <BB1534CC-76ED-482E-BA9B-D11E8B52C7AE@gmail.com>
References: <CAFOuuo7MA9Rd7zbsic8pJgShYF8NRgnddyC5JHfA6hGcGkkAfA@mail.gmail.com>
To: Radia Perlman <radiaperlman@gmail.com>
X-Mailer: Apple Mail (2.1878.6)
X-SA-Exim-Connect-IP: 192.134.164.104
X-SA-Exim-Rcpt-To: draft-ietf-tls-session-hash.all@tools.ietf.org
X-SA-Exim-Mail-From: karthik.bhargavan@gmail.com
X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000)
X-SA-Exim-Scanned: Yes (on zinfandel.tools.ietf.org)
Resent-To: draft-ietf-tls-session-hash.all@ietf.org
Resent-Message-Id: <20150416070711.371DB1B2AEF@ietfa.amsl.com>
Resent-Date: Thu, 16 Apr 2015 00:07:11 -0700 (PDT)
Resent-From: karthik.bhargavan@gmail.com
Archived-At: <http://mailarchive.ietf.org/arch/msg/draft-ietf-tls-session-hash.all@tools/5y4Zyjzkbx0j-6FEtD93fMeFhT0>
Archived-At: <http://mailarchive.ietf.org/arch/msg/tls/Hvh_k4FMk-4zYynveJMWPWDtZJE>
X-Mailman-Approved-At: Thu, 16 Apr 2015 07:32:49 -0700
Cc: draft-ietf-tls-session-hash.all@tools.ietf.org, The IESG <iesg@ietf.org>, "secdir@ietf.org" <secdir@ietf.org>
Subject: Re: [TLS] Secdir review of draft-ietf-tls-session-hash-04
X-BeenThere: tls@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "This is the mailing list for the Transport Layer Security working group of the IETF." <tls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tls>, <mailto:tls-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/tls/>
List-Post: <mailto:tls@ietf.org>
List-Help: <mailto:tls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tls>, <mailto:tls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 16 Apr 2015 07:07:12 -0000

Hi Radia,

Thanks for the comments. I made many of the smaller changes, and have some
questions about the interop concerns.

> In particular, section 5.2 paragraph 4 & 5 say:
> 
> "If the server receives a ClientHello without the extension, it SHOULD abort
> the handshake. If it chooses to continue, then it MUST NOT include the
> extension in the ServerHello."
> 
> "If a client receives a ServerHello without the extension, it SHOULD abort
> the handshake.”

The style we chose was to say that the client/server SHOULD abort the handshake,
but we also say in paragraph 7: 

"If the client or server choose to continue a full handshake without the extension, they use the legacy master secret derivation for the new session. In this case, the considerations in section 5.4 apply.”

That is, we do offer them the choice of opting out of this draft for interoperability, by following
the recommendations in 5.4.

We could make this forward pointer more explicit by saying instead:

“In order to interoperate with legacy peers, some lients and servers may choose to 
 to continue a full handshake without the extension. In this case, they use
 the  legacy master secret derivation for the new session, and the 
 considerations in section 5.4 apply.”

Would this assuage the security/interoperability concern?


> In section 5.3, there is similar language to disable session resumption when
> the extension is not present. While this may be more acceptable because it
> will not break interoperability, it *will* cause a significant performance
> degradation in some important scenarios. I would again recommend that it be
> a configuration parameter so that no one will refuse to deploy this change
> because of performance concerns.

During resumption we use similar language to point forward to section 5.2
if the client or server chooses to continue without the extension.

> Section 5.4 also mandates breaking behavior (and here it is a MUST) in
> scenarios where there is most likely to be a triple handshake vulnerability.
> Given that there are cases with no such vulnerability and given that people
> will be reluctant to deploy software that turns a subtle security
> vulnerability into non-interoperability, I believe this should be a matter
> of configuration.

This case seems most difficult, because section 5.4 is the defense of 
last resort. Weakening the MUST requirements here would, in our mind,
be tantamount to disabling the extension altogether. 

Recall that in  the triple handshake attack, the man-in-the-middle is a valid peer
who is trying to fool both the client and the server. Hence, we cannot
rely on this man-in-the-middle sending the extension. If we allow
handshakes without the extension, the attack cannot be prevented, only mitigated.
The MUSTs in this section try to enforce these mitigations.

Best regards,
Karthik


> 
> Section 5.4 paragraph 4 seems to be undermining earlier requirements, saying
> that it MAY be safe (to break rules stated earlier). Taking this to heart, I
> would propose that the "MUST abort" clauses in sections 5.2 and 5.3 be
> softened to at least allow - and perhaps recommend - implementations to
> support this scenario.
> 
> -------
> 
> Nits:
> 
> In section 6.2, it is claimed that the security of TLS depends on the hash
> function used being collision resistant, use of MD5 and SHA1 is NOT
> RECOMMENDED. That would rule out use of TLS 1.0 and TLS 1.1. While a worthy
> goal, I don't believe this enhancement makes migration away from TLS 1.0 and
> TLS 1.1 any more urgent. I also don't believe that TLS depends on the hash
> function being collision resistant (but only that it is second preimage
> resistant).
> 
> P9: "each other identity" -> "each other's identities"
> P10: "that where hence" -> "that were hence"
> P11: "server/ Hence," -> "server. Hence,