| Age | Commit message (Collapse) | Author |
|
zebra: disallow negative rtadv intvl, fix overflow
|
|
zebra: ensure ipset name is null terminated
|
|
Fix zebra zapi crashes
|
|
zebra: add a debug category for nexthops
|
|
isisd, yang: set default is-type to level-1-2
|
|
zebra: route changes via notify path should trigger nht
|
|
zebra: fix kernel-route's deletion on vrf
|
|
lib: dont decode more nexthops than we can handle
|
|
zebra: Fix zapi pbr foo, pseudowire ifname termination
|
|
lib: fix ifindex comparison overflow
|
|
If someone provides us more nexthops than our configured multipath
setting, drop the rest of them
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
bgpd: Time to deprecate bgpTimerUp
|
|
docker: Use proper container name when creating CentOS 8 container
|
|
packaging: Redirect stderr to /dev/null in %post action (frr.spec.in)
|
|
bgpd: Print json output for show_ip_bgp_regexp_cmd
|
|
Changes to a route via the dataplane notify path should
trigger nht and mpls lsp processing.
Signed-off-by: Mark Stapp <mjs@voltanet.io>
|
|
configure.ac: make msan & ubsan flags additive
|
|
zebra: fix undefined bitshifts in netlink stuff
|
|
bgpd: fix large route-distinguisher's format
|
|
This commit is about #5629 's issue.
Before this commit, bgpd creates format string of
bgp-route-distinguisher as int32, but correctly format
is uint32. current bgpd's sh-run-cli generate int32 rd,
so if user sets the rd as 1:4294967295(0x1:0xffffffff),
sh-run cli generates 1: -1 as running-config. This
commit fix that issue.
Signed-off-by: Hiroki Shirokura <slank.dev@gmail.com>
|
|
alpine: fix build on current 'edge' version of Alpine
|
|
Pim rb
|
|
Solves:
```
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
```
This happens because `grep -q` returns immediately after first match
and closes the pipe while sed has more output to write.
Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
|
|
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
We read an ifname from the wire but don't make sure its null terminated,
fix it
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
- Fix iptable freeing code to free malloc'd list
- malloc iptable in zapi handler and use those functions to free it when
done to fix a linked list memleak
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
We copy a fixed length buffer from the wire but don't ensure it is null
terminated. Then print it as a c-string. Lul.
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
- Disallow RA interval < 0
- Fix integer overflow issue converting interval to seconds from
milliseconds
- Add missing "m" to "ms"
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
Very small (negative!) ifindexes, when subtracted, can overflow.
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
|
|
Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
|
|
bgpd: fix too much no listen doc string
|
|
These were setting = SAN_FLAGS, we want to add to the variable. Note
MSAN can't be used with any other sanitizer (except UBSan) but the
compiler will complain about that if you use it wrong so we don't need
to enforce it here.
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
configure.ac: add --enable-undefined-sanitizer
|
|
doc: Add rfc8212 to supported RFCs list
|
|
further down we hash the src & dst ip, which asserts that the afi is one
of the well known ones, given the field names i assume the correct afis
here are af_inet[6]
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
Do not warn when we receive a multicast address that
matches 224.0.0.0/24.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
|
Convert the upstream_list and hash to a rb tree, Significant
time was being spent in the listnode_add_sort. This reduces
this time greatly.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
|
The channel_oil_list and hash are taking significant
cpu at scale when adding to the sorted list. Replace
with a RB_TREE.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
|
Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
|
|
a bool is not 4 bytes
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
doc: Updating FRR supported RFC list
|
|
1. Added subsections for BGP, MPLS, ISIS, OSPF etc.
2. Updated RFCs for BGP, MPLS, OSPF.
Signed-off-by: Sri Mohana Singamsetty <msingamsetty@vmware.com>
|
|
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
For UBSan
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
docker: Create build for CentOS 8
|
|
packaging: Add CentOS 8 support
|
|
Rip redistribute fixes
|
|
ripd was using zclient_redistribute for installation but not
for removal. As such the lib/zclient.c was not properly tracking
add/removal. I think it would be best to just let rip to track
this instead of zclient.
Fixes: #5599
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
|
ripngd was using zclient_redistribute for installation but not
for removal. As such the lib/zclient.c was not properly tracking
add/removal. I think it would be best to just let ripng to track
this instead of zclient.
Fixes: #5599
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|