Merge pull request #2540 from LabNConsulting/working/master/bgp-twice-leaked-nht

bgpd: don't nexthop-track twice-leaked routes that came from zebra

Merge pull request #2542 from pacovn/Coverity_1452552_Out-of-bounds_access

isisd: out-of-bounds access (Coverity 1452552)

Merge pull request #2545 from pacovn/Coverity_1468413_Explicit_null_dereferenced

bgpd: null check (Coverity 1468413)

Merge pull request #2547 from pacovn/Coverity_1458168_Dereference_null_return_value

eigrpd ospfd: null check (Coverity 1458168 1455335)

Merge pull request #2535 from donaldsharp/netlink_talk_fun

Netlink talk fun

Merge pull request #2549 from pacovn/Coverity_1452317_Explicit_null_dereferenced

ldpd: null check (Coverity 1452317)

Merge pull request #2552 from qlyoung/remove-defun-deprecated

Remove DEFUN_DEPRECATED

Merge pull request #2548 from pacovn/Coverity_1453456_Unchecked_return_value_from_library

lib: check return value (Coverity 1453456)

lib: check return value (Coverity 1453456)

Signed-off-by: F. Aragon <paco@voltanet.io>

Merge pull request #2546 from pacovn/Coverity_1465491_Untrusted_value_as_argument_2

pimd: untrusted argument (2) (Coverity 1465491)

Merge pull request #2530 from pacovn/Coverity_1399295_Out-of-bounds_read

Coverity 1399295 out of bounds read

Merge pull request #2536 from pacovn/Coverity_1470150_Dereference_null_return_value

lib: null check (Coverity 1470150)

Merge pull request #2550 from pacovn/Coverity_1399284_Dereference_null_return_value

ospfd: null check (Coverity 1399284)

Merge pull request #2544 from pacovn/Coverity_1468510_Dereference_null_return_value

ospf6d: null check (Coverity 1468510)

lib: remove DEFUN_DEPRECATED

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

bgpd: remove DEFUN_DEPRECATED usage

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

Merge pull request #2543 from pacovn/Coverity_1399304_Out-of-bounds_access

ospfd: out-of-bounds access (Coverity 1399304 1399286)

Merge pull request #2541 from pacovn/Coverity_1221445_1221448_Out-of-bounds_access

ospfd: OoB access (Coverity 1221445 1221448)

*: remove deprecated 'log trap' commands

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

pimd: untrusted argument (2) (Coverity 1465491)

Additional fix over 18e994a0437cfba9f4c09bd62293e13e49ea774b (PR #2457)

Previous correction was not enough for fixing the Coverity warning. Now we
ensure we don't overflow the buffer.

Signed-off-by: F. Aragon <paco@voltanet.io>

ospfd: null check (Coverity 1399284)

Signed-off-by: F. Aragon <paco@voltanet.io>

ospfd: OoB access (Coverity 1399304 1399286)

Signed-off-by: F. Aragon <paco@voltanet.io>

ldpd: null check (Coverity 1452317)

Signed-off-by: F. Aragon <paco@voltanet.io>

eigrpd ospfd: null chk (Coverity 1458168 1455335)

Signed-off-by: F. Aragon <paco@voltanet.io>

zebra: Log how we got to this failure.

When a filter function fails to work correctly, we get an
error message that something has gone wrong.  Unfortunately
we may not have any clues as to where the decode failure
happened.  Add a backtrace to give us a clue.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

zebra: Add error messages in places where we have a decode problem

When we receive a netlink message from the kernel we have
handler functions for when we send a netlink command, if these
return a failure ( < 0 ) then we output that we had a parse
issue.  But if all we get is:

2018-06-21T23:47:45.298156+00:00 qct-ix1-08 zebra[1484]: netlink-cmd (NS 0) filter function error

Then it is not very useful to figure out *where* the error happened.

Add more error code when in a decode path to hopefully allow us
to figure out where this message is coming from.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: null check (Coverity 1468413)

Signed-off-by: F. Aragon <paco@voltanet.io>

ospf6d: null check (Coverity 1468510)

Signed-off-by: F. Aragon <paco@voltanet.io>

isisd: out-of-bounds access (Coverity 1452552)

Signed-off-by: F. Aragon <paco@voltanet.io>

ospfd: OoB access (Coverity 1221445 1221448)

Signed-off-by: F. Aragon <paco@voltanet.io>

bgpd: don't nexthop-track twice-leaked routes that came from zebra

Issue 2381: interface based routes not marked "up" when they originate
in zebra, redistributed to bgp vrf, then imported to vpn and then
imported by another vrf.

Routes that are redistributed into BGP from zebra should not get
nexthop tracking (the assumption is that the originating protocol
is responsible to export or withdraw the route according to its own
notion of nexthop status).

The vpn-vrf route-leaking code checks the source route sub_type to
decide whether to use nexthop tracking on the resulting leaked route.

A route that is redistributed from zebra into bgp will have
sub_type==BGP_ROUTE_REDISTRIBUTE. If it is leaked to the vpn RIB,
the resulting vpn RIB route will have sub_type==BGP_ROUTE_IMPORTED.
If THAT vpn route is leaked to another vrf, the original code will
examine only the leak-source route sub_type and, since it is
not BGP_ROUTE_REDISTRIBUTE, will wrongly try to use nexthop tracking
on the new route in the final vrf.

This change modifies the leak function to track back up the
parent links to the ultimate parent of the leak source route
and look at that route's sub_type instead.

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

lib: null check (Coverity 1470150)

Signed-off-by: F. Aragon <paco@voltanet.io>

Merge pull request #2520 from donaldsharp/privs_smivs

lib: Fix privs when using HAVE_CAPABILITIES

Merge pull request #2512 from pacovn/Coverity_1399200_Unchecked_return_value_from_library

vtysh: return value check (Coverity 1399200)

ripd: out-of-bounds read (Coverity 1399295)

Signed-off-by: F. Aragon <paco@voltanet.io>

Merge pull request #2449 from donaldsharp/lib_delayed_read

Lib delayed read

vtysh: return value check (Coverity 1399200)

Signed-off-by: F. Aragon <paco@voltanet.io>

Merge pull request #2501 from pacovn/infer_unused1

bgpd isisd ldpd lib ospfd pimd: redundancy (infer, grouped)

Merge pull request #2505 from pacovn/Coverity_1143220_Argument_cannot_be_negative

tools: return check (2) (Coverity 1143220)

Merge pull request #2511 from pacovn/Coverity_1452287_et_al_Explicit_null_dereferenced

ldpd, lib: null check (Coverity 1452287 and 20 alike)

Merge pull request #2408 from ajones-rvbd/ajones-issue-2403

tests: use BUILT_SOURCES in tests/Makefile.am for automake 1.16.1

Merge pull request #2502 from pacovn/Coverity_1221459_Use_after_free

ospf6d: fix use after free (2) (Coverity 1221459)

Merge pull request #2513 from pacovn/Coverity_1455334_et_al_Dereference_null_return_value

eigrpd: null check (Coverity 1455334 and 4 alike)

Merge pull request #2518 from pacovn/Coverity_23106_Explicit_null_dereferenced

bgpd: null check (Coverity 23106)

Merge pull request #2519 from pacovn/Coverity_1399238_Logically_dead_code

bgpd: dead code (Coverity 1399238)

Merge pull request #2522 from pacovn/Coverity_1399277_Explicit_null_dereferenced

lib: null check (Coverity 1399277)

Merge pull request #2523 from pacovn/Coverity_1455380_Dereference_null_return_value

bgpd: null check (Coverity 1455380)

bgpd: null check (Coverity 1455380)

Signed-off-by: F. Aragon <paco@voltanet.io>

lib: null check (Coverity 1399277)

Signed-off-by: F. Aragon <paco@voltanet.io>

sharpd: Does not need any special priv's

Remove the unnecessary privs that we were requesting
for the sharp daemon.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: dead code (Coverity 1399238)

Signed-off-by: F. Aragon <paco@voltanet.io>

lib: Fix privs when using HAVE_CAPABILITIES

If your daemon does not need any special privileges
and you are compiling with HAVE_CAPABILIES, the
zprivs->change pointer will end up NULL due
to the way zprivs_caps_init.  So as a check
let's add a NULL check for zprivs->change
and set it to a function that will do nothing.

This change prevents a crash if you raise privileges
when your daemon needs no special privileges.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

Merge pull request #2510 from donaldsharp/extended_ack

zebra: Add extended Ack error parsing to FRR

bgpd: null check (Coverity 23106)

Signed-off-by: F. Aragon <paco@voltanet.io>

spf6d: fix use after free (2) (Coverity 1221459)

Previous fix was incomplete, as calling ospf6_lsa_unlock() frees 'req' but
it does not put it to zero, so it was called ospf6_lsdb_remove() afterwards
even being 'req' already freed.

Signed-off-by: F. Aragon <paco@voltanet.io>

include: add netlink.h to our build system

For compilation on platforms that do not have the
linux headers installed.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

zebra: Add extended Ack error parsing to FRR

Add code to request and read in extended ack information
to provide a bit more context of what went wrong when
a failure is detected in the kernel.

Example of a failed delete:
Jun 20 21:19:25 robot zebra[11878]: Extended Error: Invalid prefix for given prefix length
Jun 20 21:19:25 robot zebra[11878]: netlink-cmd (NS 0) error: Invalid argument, type=RTM_DELROUTE(25), seq=8, pid=4078403400
Jun 20 21:19:25 robot zebra[11878]: 0:4.3.2.0/24: Route Deletion failure

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

eigrpd: null check (Coverity 1455334 and 4 alike)

Coverity issues fixed with this commit:

1455334 1455335 1455336 1455337 1455339

Signed-off-by: F. Aragon <paco@voltanet.io>

Merge pull request #2504 from pacovn/Coverity_23110_Dereference_after_null_check

ospfd: null check (Coverity 23110)

Merge pull request #2507 from pacovn/Coverity_1436344_1451361_Dereference_after_null_check

bgpd, lib: null check (Coverity 1436344, 1451361)

Merge pull request #2508 from pacovn/Coverity_1470113_Untrusted_array_index_write

zebra: untrusted array index (Coverity 1470113)

Merge pull request #2506 from pacovn/Coverity_1221453_1461297_Dereference_after_null_check

bgpd ospf6d: null check (Coverity 1221453 1461297)

ldpd lib: null check (Coverity 1452287 + 20 alike)

Coverity issues fixed with this commit:

1452287 1452291 1452307 1452310 1452317 1452321 1452327 1452330 1452331 1452336
1452337 1452340 1452352 1452354 1452358
(originated at ldpd/ldpd_vty_cmds_clippy.c)

1448388 1448390 1448392 1448397 1448404 1448408
(originated at lib/plist_clippy.c)

Signed-off-by: F. Aragon <paco@voltanet.io>

bgpd isisd ldpd lib ospfd pimd: redundancy (infer)

Signed-off-by: F. Aragon <paco@voltanet.io>

tools: return check (2) (Coverity 1143220)

Previous correction (657f9948ed593387d29d391832c18dc04326a5a9) was incomplete.

Signed-off-by: F. Aragon <paco@voltanet.io>

zebra: untrusted array index (Coverity 1470113)

This is a correction over 32ac96b2ba9693696d2f1156af1b80985d4e55bb, so
removing the forced string null termination doesn't involve a worse situation
than before (the underflow check should protect for the case of receiving
an incomplete buffer, which would be the cause of non-zero terminated string)

Signed-off-by: F. Aragon <paco@voltanet.io>

bgpd, lib: null check (Coverity 1436344, 1451361)

Signed-off-by: F. Aragon <paco@voltanet.io>

bgpd ospf6d: null check (Coverity 1221453 1461297)

ospfd: null check (Coverity 23110)

Signed-off-by: F. Aragon <paco@voltanet.io>

Merge pull request #2498 from pacovn/Coverity_1470098_Logically_dead_code

bgpd: json output fix (Coverity 1470098)

bgpd: json output fix (Coverity 1470098)

Signed-off-by: F. Aragon <paco@voltanet.io>

Merge pull request #2415 from qlyoung/pbr-table-range

PBR table range configuration + misc cleanups

Merge pull request #2462 from pacovn/Coverity_1465494_String_not_null_terminated

zebra: string null termination (Coverity 1465494)

Merge pull request #2485 from pacovn/cppcheck_ldpd_redundant_condition

ldpd: redundant condition (cppcheck)

Merge pull request #2461 from pacovn/Coverity_1399380_Untrusted_value_as_argument

bgpd: Untrusted argument (Coverity 1399380)

Merge pull request #2495 from pacovn/fixme_all_digit

bgpd, lib, ospfd, zebra: all_digit moved to frrstr

Merge pull request #2496 from pacovn/fixme_group1

isisd, zebra: FIXME fixes (grouped)

Merge pull request #2499 from donaldsharp/routemap_kisses

Some route-map prep

Merge pull request #2500 from msablic/pim_doc2

doc: added parameters to two PIM commands

Merge pull request #2446 from pacovn/Coverity_1452454_1452455_1452456_Out-of-bounds_access

ospfd: OoB access (Cov 1452454 1452455 1452456)

doc: added parameters to two PIM commands

ip igmp join and mtrace have parameters which were missing from
the command spec.

Signed-off-by: Mladen Sablic <mladen.sablic@gmail.com>

bgpd, lib, zebra: Remove unnecessary return

The route_map_walk_update_list callback function
never uses the return code, so just remove it.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

lib: Don't expose a function that is never used

route_map_clear_updated is only used by routemap.c,
don't expose it too be used by the outside world.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

zebra: Add initial framework to keep track of changed route-maps

Add some basic code for zebra to start to keep track
of route-maps that have changed.  At this point we
are not doing anything.  As we fix code to handle
route-maps better, code will be shifted around.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

pimd: Remove unused code

pim does not even use route-maps, remove the
unneded code.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd, lib, pimd: Remove unused variable.

route_map_mark_updated has a `int del_later` variable
that is passed in but never used.  Just remove it.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

isisd, zebra: FIXME fixes

Signed-off-by: F. Aragon <paco@voltanet.io>

bgpd, lib, ospfd, zebra: all_digit moved to frrstr

This solves a pending FIXME

Signed-off-by: F. Aragon <paco@voltanet.io>

Merge pull request #2459 from pacovn/Coverity_1469898_Uninitialized_scalar_variable

lib: uninitialized variable (Coverity 1469898)

Merge pull request #2373 from rubenk/specfile-cleanups

Red Hat spec file cleanup

doc: Add --log-level documentation

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

lib: Add --log-level to daemons

Add the ability to specify the designated log level at startup.

--log-level <emergencies|alerts|critical|errors|warnings|notifications|informational|debugging>

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Untrusted argument (Coverity 1399380)

Signed-off-by: F. Aragon <paco@voltanet.io>

zebra: string null termination (Coverity 1465494)

Signed-off-by: F. Aragon <paco@voltanet.io>

lib: Remove special case code to use syslog

Remove the special case code to use syslog for Cumulus.
They can specify this via startup now instead of having
a special compile flag for this option.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

doc: Add some documentation for cli logging

Add some basic documentation for the new cli added to all daemons.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

lib: Add new cli to specify where to output logs on startup

When we are starting a daemon, allow the user to specify:

--log <stdout|syslog|file:file_name>

This can be used on early startup to put the log files
where the end user wants them to show up.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

vtysh: Fix 'no log syslog ..' to be correct

The vtysh version of `no log syslog...` was out
of sync with what is actually correct.  Fix.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

ldpd: Schedule application of config till after read-in

With commit e94b38d94b5 we are now scheduling the read
of vty config until after the startup of main thread
processing.  It now becomes necessary to move the
application of the config until after the read
in of the config from a file if we are using a
non-integrated config.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

lib: Create a thread for reading in the cli

The read in of cli was happening prior to thread
event handling for non-integrated configs.  This
is interesting for 2 reasons:

1) Read-in of integrated configs was after thread
event loop startup, so we had a difference of behavior

2) Read-in can cause a series of events that cause
us to attempt to communicate with zebra.  The zebra
zapi connection only happens after the thread event
loop has been started.  This can cause data that
is being written down to zebra to be lost and
no real way to notice that this has happened and
to recover gracefully.

Modify the code to create a thread event for read
in of client config.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>