[XCON] Re: tsv-dir review of draft-ietf-xcon-bfcp-connection-04

Gonzalo Camarillo <Gonzalo.Camarillo@ericsson.com> Sun, 17 June 2007 09:52 UTC

Return-path: <xcon-bounces@ietf.org>
Received: from [127.0.0.1] (helo=stiedprmman1.va.neustar.com) by megatron.ietf.org with esmtp (Exim 4.43) id 1HzrQm-00032Q-Pl; Sun, 17 Jun 2007 05:52:24 -0400
Received: from [10.91.34.44] (helo=ietf-mx.ietf.org) by megatron.ietf.org with esmtp (Exim 4.43) id 1HzrQg-0002p7-T1; Sun, 17 Jun 2007 05:52:18 -0400
Received: from mailgw4.ericsson.se ([193.180.251.62]) by ietf-mx.ietf.org with esmtp (Exim 4.43) id 1HzrQd-0008VJ-JT; Sun, 17 Jun 2007 05:52:18 -0400
Received: from mailgw4.ericsson.se (unknown [127.0.0.1]) by mailgw4.ericsson.se (Symantec Mail Security) with ESMTP id B693C201D9; Sun, 17 Jun 2007 11:52:14 +0200 (CEST)
X-AuditID: c1b4fb3e-af833bb0000007e1-c5-4675044e2f82
Received: from esealmw128.eemea.ericsson.se (unknown [153.88.254.121]) by mailgw4.ericsson.se (Symantec Mail Security) with ESMTP id 8A8EA20165; Sun, 17 Jun 2007 11:52:14 +0200 (CEST)
Received: from esealmw128.eemea.ericsson.se ([153.88.254.176]) by esealmw128.eemea.ericsson.se with Microsoft SMTPSVC(6.0.3790.1830); Sun, 17 Jun 2007 11:52:14 +0200
Received: from mail.lmf.ericsson.se ([131.160.11.50]) by esealmw128.eemea.ericsson.se with Microsoft SMTPSVC(6.0.3790.1830); Sun, 17 Jun 2007 11:52:13 +0200
Received: from [131.160.126.54] (rvi2-126-54.lmf.ericsson.se [131.160.126.54]) by mail.lmf.ericsson.se (Postfix) with ESMTP id 8C0B6248C; Sun, 17 Jun 2007 12:52:13 +0300 (EEST)
Message-ID: <4675044C.9040004@ericsson.com>
Date: Sun, 17 Jun 2007 12:52:12 +0300
From: Gonzalo Camarillo <Gonzalo.Camarillo@ericsson.com>
User-Agent: Thunderbird 1.5.0.12 (Windows/20070509)
MIME-Version: 1.0
To: Black_David@emc.com
References: <F222151D3323874393F83102D614E055068B9132@CORPUSMX20A.corp.emc.com>
In-Reply-To: <F222151D3323874393F83102D614E055068B9132@CORPUSMX20A.corp.emc.com>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
X-OriginalArrivalTime: 17 Jun 2007 09:52:14.0027 (UTC) FILETIME=[2E9A79B0:01C7B0C5]
X-Brightmail-Tracker: AAAAAA==
X-Spam-Score: 0.0 (/)
X-Scan-Signature: 848ed35f2a4fc0638fa89629cb640f48
Cc: Cullen Jennings <fluffy@cisco.com>, tsv-dir@ietf.org, ietf@ietf.org, xcon@ietf.org
Subject: [XCON] Re: tsv-dir review of draft-ietf-xcon-bfcp-connection-04
X-BeenThere: xcon@ietf.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Centralized Conferencing <xcon.ietf.org>
List-Unsubscribe: <https://www1.ietf.org/mailman/listinfo/xcon>, <mailto:xcon-request@ietf.org?subject=unsubscribe>
List-Post: <mailto:xcon@ietf.org>
List-Help: <mailto:xcon-request@ietf.org?subject=help>
List-Subscribe: <https://www1.ietf.org/mailman/listinfo/xcon>, <mailto:xcon-request@ietf.org?subject=subscribe>
Errors-To: xcon-bounces@ietf.org

Hi David,

thanks for your comments. Answers inline:

Black_David@emc.com wrote:
> I've reviewed this document as part of the transport area
> directorate's ongoing effort to review key IETF documents.
> These comments were written primarily for the transport
> area directors, but are copied to the document's authors
> for their information and to allow them to address any
> issues raised.
> 
> This is a relatively straightforward draft about how a
> client opens a TCP connection to a BFCP server when it
> has the server's transport address information.
> 
> Section 3 ventures into the area of IP address selection -
> it references RFC 3484 (which is good) and then goes on to
> make additional comments and recommendations on usage and
> state of deployment of the techniques specified in RFC 3484.
> While there's nothing technically wrong with this text, the
> additional comments and recommendations are not specific
> to BFCP, and may belong in a more generic document.

Given that such a generic document does not exist, we need to give these 
recommendations here.


> Section 4 starts with "All BFCP entities implement TLS ..."
> That is correct, as RFC 4582 requires this, but it would be
> better to cite RFC 4582 as part of this statement, e.g.,
> "[RFC 4582] requires that all BFCP entities implement TLS ..."

What about performing the following change?

OLD:

All BFCP entities implement TLS [7] and SHOULD use it in all their
    connections.

NEW:

[RFC 4582] requires that all BFCP entities implement TLS and recommends 
that they use it in all their connections.


> In the second paragraph of Section 4, I would change
> "can request the use of TLS" to "SHOULD request the use
> of TLS".

OK, I will fix it.

> Section 5.1 specifies that SubjectAltName identities in
> certificates are to be preferred to Subject identities.
> Is this specific to BFCP or more general?

We got that recommendation from our security adviser. I do not know 
whether this will be documented in a generic document at some point.

> The following text appears to be an oversight:
> 
>    If the client knows the server's IP address, the iPAddress
>    subjectAltName must be present in the certificate and must
>    exactly match the IP address known to the client.
> 
> The client *always* knows the server's IP address (e.g.,
> DNS returns it).  I think the intended case here is that
> the client contacts the server using the server's IP address
> and as a result does not know the server's hostname.  Rephrasing
> in that sort of fashion would also express a preference for
> hostname as certificate identity, which I believe is desirable.

What about performing the following change?:

OLD:
    If the client knows the server's IP address, the iPAddress
    subjectAltName must be present in the certificate and must exactly
    match the IP address known to the client.

NEW:
    If the client does not know the server's hostname and contacts the
    server directly using the server's IP address, the iPAddress
    subjectAltName must be present in the certificate and must exactly
    match the IP address known to the client.


> Section 6 disturbingly shifts between "password" and
> "pre-shared key" and appears to get a few things wrong in
> the process.  To begin with, the statement that "TLS PSK mode
> is subject to offline dictionary attacks." is false when
> the PSK is high-entropy.  OTOH, it is correct when the PSK
> is low-entropy (e.g., a password, or derived from a password
> without introduction of additional entropy).  The discussion
> in Section 7.2 of RFC 4279 applies, especially the last
> paragraph about PSK generation.  The section needs to be
> carefully revised to distinguish between "password" and
> "pre-shared key", especially given the mention of use of
> PBKDF2 to generate the latter from the former.

what about performing the following change?:

OLD:

    TLS PSK mode is subject to offline dictionary attacks.  In DHE and
    RSA modes, an attacker who can mount a single man-in-the-middle
    attack on a client/server pair can then mount a dictionary attack on
    the password.  In modes without DHE or RSA, an attacker who can
    record communications between a client/server pair can mount a
    dictionary attack on the password.  Accordingly, it is RECOMMENDED
    that where possible clients use certificate-based server
    authentication ciphersuites with PSK in order to defend against
    dictionary attacks.

    In addition, passwords SHOULD be chosen with enough entropy to
    provide some protection against dictionary attacks.  Because the
    entropy of text varies dramatically and is generally far less than
    that of an equivalent random bitstring, no hard and fast rules about
    password length are possible.  However, in general passwords SHOULD
    be chosen to be at least 8 characters and selected from a pool
    containing both upper and lower case, numbers, and special keyboard
    characters (note that an 8-character ASCII password has a maximum
    entropy of 56 bits and in general far lower).  FIPS PUB 112 [11]
    provides some guidance on the relevant issues.  If possible,
    passphrases are preferable to passwords.  In addition, a cooperating
    client and server pair MAY choose to derive the TLS PSK shared key
    from the passphrase via a password-based key derivation function such
    as PBKDF2 [2].


NEW:

    TLS PSK simply relies on a pre-shared key without specifying the
    nature of the key. In practice, such keys have two sources: text
    passwords and randomly generated binary keys. When keys are derived
    from passwords, TLS PSK mode is subject to offline dictionary
    attacks. In DHE and RSA modes, an attacker who can mount a single
    man-in-the-middle attack on a client/server pair can then mount a
    dictionary attack on the password. In modes without DHE or RSA, an
    attacker who can record communications between a client/server pair
    can mount a dictionary attack on the password. Accordingly, it is
    RECOMMENDED that where possible clients use certificate-based
    server authentication ciphersuites with password-derived PSKs,
    in order to defend against dictionary attacks.

    In addition, passwords SHOULD be chosen with enough entropy to
    provide some protection against dictionary attacks.  Because the
    entropy of text varies dramatically and is generally far less than
    that of an equivalent random bitstring, no hard and fast rules about
    password length are possible.  However, in general passwords SHOULD
    be chosen to be at least 8 characters and selected from a pool
    containing both upper and lower case, numbers, and special keyboard
    characters (note that an 8-character ASCII password has a maximum
    entropy of 56 bits and in general far lower).  FIPS PUB 112 [11]
    provides some guidance on the relevant issues.  If possible,
    passphrases are preferable to passwords.  In addition, a cooperating
    client and server pair MAY choose to derive the TLS PSK shared key
    from the passphrase via a password-based key derivation function such
    as PBKDF2 [2]. Because such key derivation functions may incorporate
    iteration functions for key strengthening they provide some
    additional security against dictionary attacks.

    When the keys are randomly generated and of sufficient length,
    these attacks do not obtain. Where possible, keys SHOULD be
    generated using a strong random number generator as specified
    in RFC 4086 [REF]. A minimum key length of 80 bits SHOULD be used.


Thanks,

Gonzalo


_______________________________________________
XCON mailing list
XCON@ietf.org
https://www1.ietf.org/mailman/listinfo/xcon