Merge pull request #2512 from pacovn/Coverity_1399200_Unchecked_return_value_from_library

vtysh: return value check (Coverity 1399200)

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>

bgpd: dead code (Coverity 1399238)

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

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>

lib: Add ability to retry if backup is specified

If we fail to read in the config file and we have
specified a backup of the backup, attempt to
read that information.

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

lib: Add ability to know if we have read anything in

When reading the config file add an ability to know
if we have properly read in anything.  So that a daemon
can make fallback plans.

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

Merge pull request #2445 from ppmathis/feature/bgp-pg-overrides

bgpd: Improve peer-group override implementation

Merge pull request #2450 from ppmathis/bugfix/bgp-filtered-routes

bgpd: Fix crash when showing filtered routes

Merge pull request #2465 from qlyoung/fix-build-am-cppflags-warning

tests: fixup minor autotools nit

Merge pull request #2468 from qlyoung/docuser

doc: update workflow.rst

Merge pull request #2470 from chiragshah6/mdev

zebra: Hide default vrf instance of l3vni cmd

Merge pull request #2474 from donaldsharp/vty_thread_cancel_writes

Add 'show thread poll'

Merge pull request #2478 from pacovn/Coverity_1408766_Dereference_after_null_check

bgpd: null check (Coverity 1408766)

Merge pull request #2479 from dslicenc/zebra-vrf-statics

zebra: re-install static routes needed vrf when the vrf intf comes up

Merge pull request #2481 from pacovn/security_vpn_null_check

bgpd: vpn null check

Merge pull request #2483 from pacovn/clang_scan_bgpd_updgrp_deref

bgpd: null check (Clang scan-build)

Merge pull request #2491 from pacovn/cppcheck_no_effect

eigrpd, isisd, lib, ospfd: no effect (cppcheck)

Merge pull request #2492 from pacovn/cppcheck_security1

eigrpd, lib, tests, vtysh: security (cppcheck)

eigrpd, lib, tests, vtysh: security (cppcheck)

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

eigrpd, isisd, lib, ospfd: no effect (cppcheck)

Assignment of function parameter has no effect outside the function.

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

redhat: fix else cases in spec file

As pointed out during review, disable the options when the macros are
false.

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: spec file: specify file only once

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: spec file: no need for custom attr for service file

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: add missing dependencies on chkconfig in spec file

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: remove a few superfluous Requires from spec file

rpm adds these automatically

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: remove a few superfluous BuildRequires from spec file

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: depend on info package instead of binary in specfile

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: install-info is not called from %pre in spec file

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: remove wrong comment from spec file

systemd support needs watchfrr

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: spec file: all supported Fedora versions use systemd

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: use default permissions for logrotate file in spec file

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: spec file: we ship only one logrotate file

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: be consistent in adding ./configure options in spec file

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: use Github for SOURCE0 in spec file

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: Use https for URL in spec file

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: remove trailing whitespace from spec file

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

redhat: fix tabs vs spaces in spec file

rpmlint reports:
redhat/frr.spec:282: W: mixed-use-of-spaces-and-tabs (spaces: line 13, tab: line 282)

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

Merge pull request #2476 from pacovn/Coverity_1433544_Explicit_null_dereferenced

bgpd: null check (Coverity 1433544, 1433543, 1433542)

Merge pull request #2482 from pacovn/clang_scan_bgpd_bgp_rpki_deref

bgpd: null check (Clang scan-build)

bgpd: null check (Clang scan-build)

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