Re: Benjamin Kaduk's Discuss on draft-ietf-bfd-yang-16: (with DISCUSS and COMMENT)

Jeffrey Haas <jhaas@pfrc.org> Wed, 11 July 2018 14:36 UTC

Return-Path: <jhaas@pfrc.org>
X-Original-To: rtg-bfd@ietfa.amsl.com
Delivered-To: rtg-bfd@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 82287130F76; Wed, 11 Jul 2018 07:36:55 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_PASS=-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 hS2spwMjWKf3; Wed, 11 Jul 2018 07:36:53 -0700 (PDT)
Received: from slice.pfrc.org (slice.pfrc.org [67.207.130.108]) by ietfa.amsl.com (Postfix) with ESMTP id 8CD3B130E19; Wed, 11 Jul 2018 07:36:50 -0700 (PDT)
Received: from dresden.attlocal.net (99-59-193-67.lightspeed.livnmi.sbcglobal.net [99.59.193.67]) by slice.pfrc.org (Postfix) with ESMTPSA id 161261E281; Wed, 11 Jul 2018 10:36:41 -0400 (EDT)
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\))
Subject: Re: Benjamin Kaduk's Discuss on draft-ietf-bfd-yang-16: (with DISCUSS and COMMENT)
From: Jeffrey Haas <jhaas@pfrc.org>
In-Reply-To: <EB68C226-A9F7-4254-97D8-042FB6FB6DFF@cisco.com>
Date: Wed, 11 Jul 2018 10:36:49 -0400
Cc: "Acee Lindem (acee)" <acee@cisco.com>, Benjamin Kaduk <kaduk@mit.edu>, "rtg-bfd@ietf.org" <rtg-bfd@ietf.org>, "draft-ietf-bfd-yang@ietf.org" <draft-ietf-bfd-yang@ietf.org>, The IESG <iesg@ietf.org>, "bfd-chairs@ietf.org" <bfd-chairs@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <B14B0368-0795-4163-8A6D-A4D03B4568A1@pfrc.org>
References: <153064928232.4913.5177531623706237853.idtracker@ietfa.amsl.com> <69638E39-860F-4D2F-AE2B-3B0B0A7BA855@cisco.com> <20180704035647.GF60996@kduck.kaduk.org> <20180710234621.GD12853@pfrc.org> <DBDBAFD1-8280-4389-B7F9-08785FC01BF8@cisco.com> <650FCD2E-A555-447A-A7AA-87D67B4E2BDE@cisco.com> <D4C67FBF-10C0-4F22-B8C6-1D9DD9CFE7B0@cisco.com> <EB68C226-A9F7-4254-97D8-042FB6FB6DFF@cisco.com>
To: Reshad Rahman <rrahman@cisco.com>
X-Mailer: Apple Mail (2.3273)
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-bfd/BRU9RqtBKbEw0QKKqzndflcMfyw>
X-BeenThere: rtg-bfd@ietf.org
X-Mailman-Version: 2.1.27
Precedence: list
List-Id: "RTG Area: Bidirectional Forwarding Detection DT" <rtg-bfd.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-bfd>, <mailto:rtg-bfd-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-bfd/>
List-Post: <mailto:rtg-bfd@ietf.org>
List-Help: <mailto:rtg-bfd-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-bfd>, <mailto:rtg-bfd-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 11 Jul 2018 14:37:02 -0000

Even that may be overkill in some circumstances.

Consider the case where an operator may decide that they'll permit BFD to be de-configured for protocols without also giving operators permission to similarly de-configure the underlying client protocols.  Examples of this may be that BFD is experiencing issues that are leading to false negatives and tearing down valid sessions.

While the above example would seem unusual, it was more of a concern in the more subtle use cases for BFD-on-LAG where links may be managed by one provisioning group and protocols by another.

-- Jeff

> On Jul 11, 2018, at 10:31 AM, Reshad Rahman (rrahman) <rrahman@cisco.com> wrote:
> 
> Hi Acee,
> 
> That and a statement saying the BFD clients should be authenticated.
> 
> Regards,
> Reshad.
> 
> On 2018-07-11, 10:10 AM, "Acee Lindem (acee)" <acee@cisco.com> wrote:
> 
>    Hi Reshad, 
> 
>    Ok - so are you saying that all that is being asked for is that the NACM rules on IGP BFD configuration be at least as strict as BFD since the IGPs are instantiated BFD sessions? I'd be ok with that. 
> 
>    Thanks,
>    Acee 
> 
>    On 7/11/18, 9:25 AM, "Reshad Rahman (rrahman)" <rrahman@cisco.com> wrote:
> 
>        Hi,
> 
>        My read on the DISCUSS is not just wrt spoofing of BFD clients but also making sure that the proper access restriction (NACM) is used for the BFD clients. 
> 
>        I didn't interpret Benjamin's comments as requiring a security boundary between BFD clients (BGP, IPGPs) and BFD running on the same dveice, which I agree would be preposterous.
> 
>        Regards,
>        Reshad.
> 
>        On 2018-07-10, 10:17 PM, "Acee Lindem (acee)" <acee@cisco.com> wrote:
> 
> 
> 
>            On 7/10/18, 7:46 PM, "Rtg-bfd on behalf of Jeffrey Haas" <rtg-bfd-bounces@ietf.org on behalf of jhaas@pfrc.org> wrote:
> 
>                On Tue, Jul 03, 2018 at 10:56:49PM -0500, Benjamin Kaduk wrote:
>> On Wed, Jul 04, 2018 at 03:20:42AM +0000, Reshad Rahman (rrahman) wrote:
>>> <RR> I am not 100% sure I understand the point being made. Is it a question of underlying the importance of having the IGPs authenticated since the IGPs can create/destroy BFD sessions via the local API?
>> 
>> That's the crux of the matter, yes.  Since (in this case) the IGP state
>> changes are being translated directly into BFD configuration changes,
>> the NETCONF/RESTCONF authentication is not really used.  So, any
>> authentication/authorization decisions that are made must be on the basis
>> of authentication at the IGP level.  This does not necessarily mean a hard
>> requirement for IGP authentication, though using unauthenticated IGP would
>> then be equivalent (for the purposes of this document) to allowing
>> anonymous NETCONF/RESTCONF access.
>> 
>> I'd be happy to just have a note in the security considerations that "when
>> BFD clients such as IGPs are used to modify BFD configuration, any
>> authentication and authorization for the configuration changes take place
>> in the BFD client, such as by using authenticated IGPs".  But feel free to
>> reword in a better fashion; this is really just about acknowledging the new
>> access mechanism (since the boilerplate covers SSH/TLS for
>> NETCONF/RESTCONF).
> 
>                I must admit to being somewhat perplexed by the request here.
> 
>                In the cases where BFD as a top level module is not the creator of a BFD
>                session, you seem to be concerned that BFD can be used to attack a resource
>                by spoofing that non-BFD client.
> 
>                While this is perhaps logically true, it also means that you have a far
>                greater problem of being able to spoof the underlying BFD clients.  To give
>                some real-world examples:
>                - BGP typically requires explicit configuration for its endpoints.
>                - Both OSPF and ISIS will require a matched speaker with acceptable
>                  configuration parameters for a session to come up.
>                - Static routes with BFD sessions will require explicit configuration.
> 
>                In each of these cases, a client protocol that also wants to use BFD, the
>                simple spoofing of the protocol endpoints is already a massive disaster
>                since it will allow injection of control plane or forwarding state into the
>                device.  This is so much worse than convincing a BFD session to try to come
>                up with its default one packet per mode that ... well, I'm boggled we're
>                even talking about this. :-)
> 
>                My request would be that we not complicate the security considerations of
>                this module for such cases.
> 
>            I agree. This is DISCUSS is just preposterous - imposing some sort of security boundary between the IGP modules and the BFD module running on the same networking device.
> 
>            Thanks,
>            Acee (LSR WG Co-Chair) 
> 
> 
> 
>                -- Jeff
> 
> 
> 
> 
> 
> 
> 
> 
>