bgpd: Convert to using LIB_ERR_XXX where possible

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

zebra, lib: error references for zebra

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

isisd: Cleanup compile issue

cleanup compile with missnamed enum usage.

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

isisd: Add isis_errors and generate custom Error Codes

Generate appropriate error codes for ISIS.

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

isisd: Convert to use LIB_ERR_XXX

Where an obvious choice could be made about converting
to a LIB_ERR_XXX function do so.

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

babeld: Convert all zlog_err to zlog_ferr and add appropriate info

Convert babeld to use zlog_ferr and add appropriate BABEL_ERR_XXX
information.

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

babeld: Add BABEL_ERR_XXX error messages.

Add some BABEL_ERR_XXX error messages and convert over to using
zlog_ferr instead of zlog_err.

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

pimd: Add pim_errors and define some pim specific errors

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

bgpd: Cleanup initialization of bgp_errors.c

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

bgpd: implement zlog_ferr facility for enhance error messages in bgp

Signed-off-by: Don Slice <dslice@cumulusnetworks.com<

ldpd: Convert to using LIB_ERR_XXX and zlog_ferr

convert ldpd over to using zlog_ferr and LIB_ERR_XXX
where appropriate.

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

eigrpd: Convert to use LIB_ERR_XXX for zlog_err

Convert, where appropriate, to using LIB_ERR_XXX for zlog_ferr

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

lib: Update error ranges to include all daemons

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

pimd: Convert over to use LIB_ERR_XXX for zlog_err

Convert, where appropriate, the zlog_err functions to zlog_ferr
for the LIB_ERR_XXX enums

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

ripngd: Convert over to using LIB_ERR_XXXX for zlog_ferr

Convert over those zlog_err functions to zlog_ferr that
are appropriate for the LIB_ERR_XXX.

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

ripd: Convert zlog_err to zlog_ferr for LIB_ERR_XXX

For those errors that can be traced to LIB_ERR_XXX
convert to using those in ripd.

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

ospf6d: Update OSPFv3 to use new zlog_ferr

Update OSPFv3 to use the new zlog_ferr messages

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

lib: Cleanup some missed LIB_ERR_SYSTEM_CALL

libfrr.c had some detection for system call issues.

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

lib: Add LIB_ERR_ZMQ

Add new error event for when we detect an issue within
the ZMQ subsystem.

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

lib: Add LIB_ERR_DEVELOPMENT

Sometimes a error state is detected when we have added
new code to FRR, but not updated all the places that
we should have.  Consider this a developmental escape
that needs to be fixed.

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

lib: Add LIB_ERR_NS to error subsystem

Add additional information about NameSpace errors that
may happen.

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

lib, zebra: Add LIB_ERR_INTERFACE

Add a error type that allows us to track bad interface states.

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

bgpd, lib, zebra: Convert LIB_ERR_PRIVILEGES

For all the places we zlog_err about raising/lowering privileges, use
zlog_ferr.

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

lib: Add LIB_ERR_SNMP

Add code to indicate a SNMP error of some sort that needs to be
handled and addressed.

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

lib: Add LIB_ERR_VTY

Add a error code for when the vty subsystem detects
an error.

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

lib: Add LIB_ERR_SYSTEM_CALL and convert VRF_SOCKET to SOCKET

Add a new error code LIB_ERR_SYSTEM_CALL to the ferr subsystem.
Additionally convert LIB_ERR_VRF_SOCKET to a more generic
LIB_ERR_SOCKET.

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

lib: When logging commands do not use zlog_err

When we are logging a commands via the `log commands`
cli, use zlog_notice instead of zlog_err, since that
this is not an actual error situation.

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

lib: Convert to zlog_ferr for zclient.c

Convert the zclient.c file to use zlog_ferr.

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

lib, bgpd: Add code to make lib auto create the ferr infrastructure

Add code to auto-create the ferr infrastructure as well as add
some initial error handling for vrf.c

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

lib: Allow adding arrays of ferr's

Simplify addition of new messages to the system by allow passage of
arrays of data, instead of one at a time.

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

lib: add error reference system

* Add zlog_* function to log with a reference code
* Add ability to track reference cards for errors to ferr.[ch]
* Assign some reference code ranges

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

doc: Add missing flowspec.rst to distribution tar file

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

Merge pull request #2805 from opensourcerouting/malloc-size

lib: track total memory per MTYPE if possible

Merge pull request #2294 from opensourcerouting/bfd-final

bfdd: add BFD support

Merge branch 'master' into bfd-final

Merge pull request #2823 from opensourcerouting/snap-staticd

Updates to snap package to support staticd

Merge pull request #2833 from opensourcerouting/assorted-fixes

Assorted fixes (pre-northbound)

Merge pull request #2832 from opensourcerouting/rm-debug-leftovers

zebra: remove unguarded debugging leftovers

lib, vtysh: fix inconsistent interface commands in vtysh

The definition of the interface commands in vtysh.c were outdated.
Currently, all daemons that call if_cmd_init() will have the "no interface
IFNAME" command and the "[no] description" commands as well, so there's
no need to define exceptions for these commands anymore.

To fix this, make extract.pl parse the if.c file so that vtysh can get the
interface commands from there automatically. Only the "interface IFNAME
[vrf NAME]" must be kept in vtysh.c because it changes the vty node and
thus needs special treatment.

Finally, make pimd and pbrd display interface descriptions on "sh run"
when they are configured.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

lib, vtysh: fix inconsistent VRF commands in vtysh

* Only zebra and pimd call vrf_cmd_init(), so these are the only daemons
  that should receive VRF commands from vtysh;
* "netns NAME" and "no netns NAME" are available only in zebra, write
  custom DEFSHs in vtysh to make it aware of that;
* Remove the "no vrf NAME" definition from vtysh.c and expose the
  original command to vtysh by converting the DEFUN_NOSH to a simple
  DEFUN. This command doesn't change the vty node so there's no need to
  special case it.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

zebra: fix "no pseudowire IFNAME" on vtysh

We must hide only "pseudowire IFNAME" from vtysh, the "no" form of the
command should be made available to the extract.pl script. Split the
command into two to fix this problem.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

ldpd: use DEFPY_NOSH whenever possible

All ldpd commands were written using DEFPY except the ones that needed
to be ignored by vtysh, in which case we were using DEFUN_NOSH. Now that
DEFPY_NOSH is available, convert the remaining commands for consistency
and to simplify the code a little bit.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

lib: implement DEFPY_NOSH

This is the DEFPY equivalent of DEFUN_NOSH.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

babeld: avoid reading the configuration file twice

The frr_config_fork() function called by all FRR daemons during
initialization already takes care of reading the startup configuration
file.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

lib: add listnode_add_head()

Provide a new convenience function that adds an element to the beginning
of a list.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

snapcraft: Add new staticd to snap package

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

vtysh: Fix missing tailing / in --config_dir option parsing

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

zebra: remove unguarded debugging leftovers

These debug messages were committed by accident.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

Merge pull request #2830 from pacovn/Coverity_1221459_revert

ospf6d: revert fix for Coverity 1221459

Merge pull request #2820 from opensourcerouting/err-prep

remove alloc return value checks + minor babeld cleanup

ospf6d: revert fix for Coverity 1221459

The correction in commit 7edb6aa (PR #2502) was wrong, as it is was not
taking in consideration the unlock counter. Thanks to @eqvinox for noticing
it.

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

Merge pull request #2827 from pacovn/Coverity_1472310_Unchecked_return_value

bgpd: return value check (Coverity 1472310)

Merge pull request #2828 from pacovn/Coverity_1472311_null_check

staticd: null check (Coverity 1472311)

Merge pull request #2825 from dslicenc/frr-reload-vni

tools: correct frr-reload.py handling of bgp vni/exit-vni config

Merge pull request #2824 from vishaldhingra/reg_case

peer_delete did not trigger ZEBRA_NEXTHOP_UNREGISTER to ZEBRA

staticd: null check (Coverity 1472311)

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

bgpd: return value check (Coverity 1472310)

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

peer_delete did not trigger ZEBRA_NEXTHOP_UNREGISTER to ZEBRA

bgp_unlink_nexthop_check() PEER should be NULL and bgp_info list count should be 0

Signed-off-by: vishaldhingra <vdhingra@vmware.com>

doc: Add missing flowspec.rst to distribution tar file

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

babeld: Use enum instead of #defines for kernel route type information

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

tests: Cleanup assumption that ALLOC could fail.

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

lib: Remove memory check test

Found a missed memory check test that can just be removed.

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

ripd: re-add check for rip_create fail

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

*: ALLOC calls cannot fail

There is no need to check for failure of a ALLOC call
as that any failure to do so will result in a assert
happening.  So we can safely remove all of this code.

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

Merge pull request #2809 from opensourcerouting/routemap-rpki-fix

bgpd: Fix bgp rpki to match route-map update from PR #2697

tools/checkpatch: fix some bogus macro warnings

checkpatch was throwing an error for "#define FOO , bar"

Signed-off-by: David Lamparter <equinox@diac24.net>

tools: correct frr-reload.py handling of bgp vni/exit-vni config

Problem reported that when a peer-group was added in certain
configurations, it would be rejected because of the order of the
commands put in by nclu. Issued turned out to be how frr-reload.py
was handling the sub-sub-context of the vni under the address-family
and subsequently how it handled the following exit-vni.

Ticket: CM-21996
Signed-off-by: Don Slice <dslice@cumulusnetworks.com>

doc: add "show memory" user documentation

Signed-off-by: David Lamparter <equinox@diac24.net>

Merge pull request #2806 from opensourcerouting/vty-term-fd

lib: fix "-t" command line option

Merge pull request #2812 from manuhalo/fix-staticd-coverity

staticd: Fix null pointer access (coverity 1472311)

staticd: Fix null pointer access (coverity 1472311)

Aded a check to ensure that if_is_vrf() is not called on a NULL if pointer.
Incidentally this was causing PR #2805 to fail.

Signed-off-by: Emanuele Di Pascale <emanuele@voltanet.io>

Merge pull request #2807 from mjstapp/configure_help_fixes

configure.ac : fix help text

bgpd: Fix bgp rpki to match route-map update from PR #2697

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

bgpd/ospf(6)d/pimd: hide BFD commands with timers

Don't show BFD commands with timers since it might confuse users
("show running-config" won't display timers in client daemons anymore),
but keep accepting this command from previous configurations.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: fix IPv6 peers using link-local address

When using link-local address we must specify the scope-id for the
address in order to bind to the interface.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: clean-up bfd clients data on shutdown

On `zebra` / `bfdd` shutdown we now clean up all client data to avoid
memory leaks (ghost clients). This also prevents 'slow' shutdown on
`zebra` sparing us from seeing some rare topotests shutdown failures
(signal handler getting stopped by signal).

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: improve logging messages

Show a little more details, remove some duplicated calls and remove the
macro compatibility with old debugging functions.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: clean up header inclusion

Most of the headers we need are included by zebra.h, so lets simplify
this.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: code refactory for simplification

Simplify code and remove unnecessary log messages. The old log messages
are going to be shown by the caller anyway.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: free zebra clients data on unregistration

Avoid a memory leak on client daemons restart by getting rid of old
registrations.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: send replay request on zebra connection

This will make `bfdd` synchronize with its client when zebra dies or
bfdd is restarted.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: show single hop local-address

Show local-address on single hop when configured.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: smooth configuration transition

After configuring a new value set the polling bit to negotiate speeds
again next transmission cycle.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: show the correct transmission speed

When configured transmission speed doesn't match the actual speed, show
the difference in the output.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: add documentation

Add BFD daemon documentation:
* commands;
* man page;
* manual / description;

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bgpd/ospf(6)d/pimd: don't show BFD timers

When BFD timers are configured, don't show it anymore in the daemon
side. This will help us migrate the timers command from daemons to
`bfdd`.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: re-route PTM-BFD daemon messages

When `bfdd` is enabled - which it is by default - re-route the PTM-BFD
messages to the FRR's internal BFD daemon instead of the external
PTM daemon.

This will help the migration of BFD implementations and avoid
duplicating code.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: add vty shell commands

Implement vty shell integration and allow `bfdd` to be configured
through FRR's vtysh.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: wire to the build process

Add BFD daemon to the build process and packaging instructions.

Currently the bfdd daemon does nothing, this is just to document how the
daemon insertion step occured.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: imported new daemon source code

Import source code from external `bfdd` daemon ported from Cumulus PTM.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

lib: import new JSON iteration macro

Save a few lines when iterating over JSON objects using the new
JSON_FOREACH macro.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

configure: fix help text

Correct help text in a couple of places; add configure switch
for new staticd daemon.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

lib: fix "-t" command line option

was_stdio was getting set for fd == -1 (config file read), thus
prematurely closing the stdio vty.

Signed-off-by: David Lamparter <equinox@diac24.net>

Merge pull request #2790 from lyq140/branch1

don't send packets when auth_str is empty (via keychain.)

Merge pull request #2794 from netravnen/feature/docs/set-origin

doc: set origin

Merge pull request #2799 from adharkar/frr-zebra_cli

Zebra: Changes to "show ip route" json commands

lib: count total memory allocation per MTYPE

If malloc_usable_size() or malloc_size() are available, we can count
total usage of a particular MTYPE.  (Without the functions, we don't
know how much to subtract on free.)

Signed-off-by: David Lamparter <equinox@diac24.net>

build: rework mallinfo test & find malloc_size

Signed-off-by: David Lamparter <equinox@diac24.net>

Merge pull request #2122 from donaldsharp/zebra_nhs

Create Staticd

staticd: fix tiny whitespace issue

Just add a linebreak.

Signed-off-by: David Lamparter <equinox@diac24.net>