]> git.puffer.fish Git - matthieu/frr.git/commit
bgpd: relax ORF capability length handling
authorDavid Lamparter <equinox@opensourcerouting.org>
Wed, 23 Jan 2013 04:50:24 +0000 (05:50 +0100)
committerDavid Lamparter <equinox@opensourcerouting.org>
Fri, 1 Feb 2013 16:55:04 +0000 (17:55 +0100)
commit5e728e929942d39ce5a4ab3d01c33f7b688c4e3f
tree6f2b2413fc182b75b589fdb340c813d7da944771
parentf47e5a18b5beb00d6b5b94965e305dadb5aa5bad
bgpd: relax ORF capability length handling

commit fe9bb64... "bgpd: CVE-2012-1820, DoS in bgp_capability_orf()"
made the length test in bgp_capability_orf_entry() stricter and is now
causing us to refuse (with CEASE) ORF capabilites carrying any excess
data.  This does not conform to the robustness principle as laid out by
RFC1122 ("be liberal in what you accept").

Even worse, RFC5291 is quite unclear on how to use the ORF capability
with multiple AFI/SAFIs.  It can be interpreted as either "use one
instance, stuff everything in" but also as "use multiple instances".
So, if not for applying robustness, we end up clearing sessions from
implementations going by the former interpretation.  (or if anyone dares
add a byte of padding...)

Cc: Denis Ovsienko <infrastation@yandex.ru>
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
bgpd/bgp_open.c