lib: parser: remove forgotten line

MTYPE_CMD_TOKEN_DATA isn't used anymore, I forgot to ditch the line.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

Merge pull request #531 from qlyoung/fix-stack-ref

lib: fix access to stack value

doc: fix VARIABLE <> WORD confusion in cli.md

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

bgpd: autocomplete peer-groups & interface peers

This wasn't quite correct in the previous patch, leading to peer-groups
& interface peers breaking in autocompletion.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

lib: cli: fix IPv6 address partial matching

A partially-entered IPv6 address would never return a "partly_match",
meaning some possible completions weren't listed by the matcher.

This specifically breaks autocompleting BGP IPv6 neighbor addresses.
Before:
aegaeon# show ip bg ne 2001:<?>
  WORD      Neighbor on BGP configured interface

After:
aegaeon# show ip bg ne 2001:<?>
  WORD      Neighbor on BGP configured interface
  X:X::X:X  Neighbor to display information about
      2001:db8::2

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

doc: update CLI documentation

Fix markdown formatting & add variable names.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

bgpd: autocomplete neighbor names

Add autocompletion for neighbor IP/IPv6/peer-group names.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

vtysh: autocomplete variables

This asks the connected daemons for their variable completions through a
hidden CLI command.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

lib: cli: autocomplete variables

Shows known values in the appropriate naming domain when the user hits
<?> or <Tab>.  This patch only works in the telnet CLI, the next patch
adds vtysh support.

Included completions:
- interface names
- route-map names
- prefix-list names

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

lib: parser: add pre-merge varname propagation step

Fills token->varname based on context.  WORD tokens use the WORD - if it
isn't actually "WORD".  Other than that, a preceding constant token is
used as name.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

lib: parser: split off & rename graph handling

Put core CLI graph stuff in lib/command_graph.[ch] and consistently
prefix all function names with "cmd_".

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

lib: parser: add named variables in CLI

struct cmd_token now has a "varname" field which is derived from the
DEFUN's string definition.  It can be manually specified with "$name"
after some token, e.g. "foo WORD$var".  A later commit adds code to
automatically fill the value if nothing is specified.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

Merge pull request #520 from opensourcerouting/bgpd-fixes-master

bgpd: assorted fixes (master)

Merge pull request #527 from LabNConsulting/working/master/patch-set/issue471+473

Master: Restore label distribution for VPN safi

Merge pull request #526 from donaldsharp/renato_eigrp

Renato eigrp

lib: fix access to stack value

Passing stack value to thread_add_* causes thread->ref to become an
invalid pointer when the value goes out of scope

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

Merge remote-tracking branch 'origin/stable/3.0'

bgpd: fix table leak introduced in labeled unicast (Issue #473)
      Fix tested for vpn and encap ribs/safi.  Labeled unicast
      may still leak as not tested.

Signed-off-by: Lou Berger <lberger@labn.net>

bgpd: fix v6 encap safi config - more labeled unicast breakage (Issue #473)

Signed-off-by: Lou Berger <lberger@labn.net>

bgpd: fix rfapi label origination (Issue #473)

Signed-off-by: Lou Berger <lberger@labn.net>

bgpd: fix VPN SAFI label distribution (Issue #473)

Signed-off-by: Lou Berger <lberger@labn.net>

bgpd: add epvn as labeled safi type (ISSUE #471)

Signed-off-by: Lou Berger <lberger@labn.net>

eigrpd: Diverse Fixes

* Correct the metric calculation as well as the default metrics;
* Do not show invalid routes in the "show ip eigrp topology".
* Add support to VRFs;
* When downloading a neighbor remove the related routes;
* Fix bugs in the parser of packages they were creating
Invalid default routes;
* Add and remove routes in the zebra;
* Add command "on router eigrp AS";
* Make "delay" and "bandwitch" commands work as well as
Display them in running config;
* Add "no" version of several commands;
* Fix a serious momory leaks;
* Fix segfault when there is no 'successor' route to a
Given prefix;
* Other minor corrections;

Signed-off-by: Renato Westphal <renatowestphal@gmail.com>
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

eigrpd: Some Basic Corrections

* Advertise routes to zebra
* Connected routes should not have a nexthop address
* Fix segfaut on exit

Signed-off-by: Renato Westphal <renatowestphal@gmail.com>

lib: Cleanup command parsing a tiny bit

The CLI changes now make it impossible for numbers
outside the range specified in the cli to make it to
this code.  No need to check for it again.

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

Merge pull request #521 from opensourcerouting/bgpd-fixes-stable

bgpd: assorted fixes (stable/3.0)

Merge pull request #508 from LabNConsulting/working/3.0/patch/issue198

bgpd: Issue #198 - clean up duplicate encap SAFI commands

Merge pull request #525 from qlyoung/frr-script-fix

tools: $valgrind --> "$valgrind"

Merge pull request #524 from qlyoung/blublublub3.0

lib: use %p specifier for pointers

tools: $valgrind --> "$valgrind"

String cmp missing quotes

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

lib: use %p specifier for pointers

Fixes builds on ARM

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

README: add references to EIGRP and NHRP.

Merge pull request #518 from dwalton76/zebra-mpls-lsp-uninstall

zebra: zebra_mpls_lsp_uninstall was not being called

Merge pull request #495 from donaldsharp/mpls_fixes

Mpls fixes

bgpd: fix segfault with '[no] bgp network import-check'

These commands were causing bgpd to crash if a static VPN route was
configured.

While here, fix a bug in bgp_static_add() and bgp_static_delete().

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

bgpd: fix duplicate install of command

Only happens when FRR is configured with --enable-oldvpn-commands.

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

bgpd: fix 'show bgp [...] longer-prefixes'

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

bgpd: remove unused bgp_attr_default_intern() function

This function is not used anymore since commit 137446f9.

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

lib: remove gap between AFI_IP6 and AFI_L2VPN

We have several pieces of code like this in FRR:

  for (afi = AFI_IP; afi < AFI_MAX; afi++)
    for (safi = SAFI_UNICAST; safi < SAFI_MAX; safi++)
      bgp_distance_table[afi][safi] = bgp_table_init (afi, safi);

We were creating a lot of useless garbage in the code because of this
gap. Fix this.

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

bgpd: fix segfault with '[no] bgp network import-check'

These commands were causing bgpd to crash if a static VPN route was
configured.

While here, fix a bug in bgp_static_add() and bgp_static_delete().

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

bgpd: fix duplicate install of command

Only happens when FRR is configured with --enable-oldvpn-commands.

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

bgpd: removed unused parameter from bgp_dump_attr()

Originally we used the 'peer' parameter for this:

  if (peer_sort (peer) == BGP_PEER_IBGP)
    snprintf (buf + strlen (buf), size - strlen (buf), ", localpref %d",
              attr->local_pref);

Now we have this:

  if (CHECK_FLAG (attr->flag, ATTR_FLAG_BIT (BGP_ATTR_LOCAL_PREF)))
    snprintf (buf + strlen (buf), size - strlen (buf), ", localpref %u",
              attr->local_pref);

Remove the now useless 'peer' parameter.

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

lib: add labeled-unicast in safi2str()

While here, move SAFI_ENCAP above SAFI_MPLS_VPN to keep the ordering
consistent.

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

bgpd: fix 'show bgp [...] longer-prefixes'

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

zebra: zebra_mpls_lsp_uninstall was not being called

Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>
Old entries were not being removed from the MPLS table.

Merge pull request #511 from donaldsharp/netlink_dump

zebra: Fix kernel netlink hex dump of information

zebra: Fix kernel netlink hex dump of information

1) Fix the kernel send and recv hex dump of what we
are sending to the kernel

2) When configuring the debugs under 'conf t' display
it too.

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

Merge pull request #510 from opensourcerouting/update-tools-gitignore

tools: update .gitignore

tools: update .gitignore

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

bgpd: Issue #198 - clean up duplicate command resulting from CLI refactoring

Signed-off-by: Lou Berger <lberger@labn.net>

bgpd: fixup bgpd: allow VPN next hop to be different AFI than NLRI next
hop for EVPN

In the case of EVPN, add the nexthop attribute.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

added troubleshooting section to debian 8 build doc

Merge pull request #503 from pguibert6WIND/issue_473

bgpd: fixup bgpd: allow VPN next hop to be different AFI than NLRI nexthop for EVPN

Merge pull request #500 from qlyoung/fix-router-rip-3.0-3.0

vtysh: add \n to 'router rip[ng]' docstrings (3.0)

Merge pull request #496 from donaldsharp/vtysh_pager

*: Allow configure to control VTYSH_PAGER used

Merge pull request #502 from opensourcerouting/show-ip-fib

add 'show <ip|ipv6> fib' command

zebra: add 'show ipv6 fib' command

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

zebra: add 'show ip fib' commands

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Address the error "Dead assignment" of static analysif

- Refer to https://ci1.netdef.org/browse/FRR-FRR4-44/artifact/shared/static_analysis/index.html
- Remove unused variable

Signed-off-by: Hung-Weic Chiu <sppsorrg@gmail.com>

- Solve the Coverity Scan PW.ASSIGN_WHERE_COMPARE_MEANT

- Minor refactoring

Signed-off-by: Hung-Weic Chiu <sppsorrg@gmail.com>

vtysh: add \n to 'router rip[ng]' docstrings

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

Address the error "Dead assignment" of static analysif

- Refer to https://ci1.netdef.org/browse/FRR-FRR4-44/artifact/shared/static_analysis/index.html
- Remove unused variable

Signed-off-by: Hung-Weic Chiu <sppsorrg@gmail.com>

- Solve the Coverity Scan PW.ASSIGN_WHERE_COMPARE_MEANT

- Minor refactoring

Signed-off-by: Hung-Weic Chiu <sppsorrg@gmail.com>

Merge pull request #408 from qlyoung/remove-thread-macros

*: remove THREAD_ON macros, add nullity check

bgpd: fixup bgpd: allow VPN next hop to be different AFI than NLRI next
hop for EVPN

In the case of EVPN, add the nexthop attribute.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

*: Allow configure to control VTYSH_PAGER used

Signed-off-by: Donald Sharp

eigrpd: Fix compilation errors for some versions of gcc

Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>

zebra: do not crash if zebrad.lsp_process_q is NULL

Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>

zebra: lsp_install() failed due to ZEBRA_FLAG_SELECTED check

ZEBRA_FLAG_SELECTED hasn't been set yet by the time lsp_install is
called.  The call path is:
rib_process -> rib_process_add_fib -> zebra_mpls_lsp_install -> lsp_install

but ZEBRA_FLAG_SELECTED is set in rib_process after it calls
rib_process_add_fib.  I can't think of anything that it would hurt to
install the LSP regardless of whether ZEBRA_FLAG_SELECTED is set later.

I also cleaned up some UI (json and display the pretty label names
instead of their numeric values).

Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>

zebra: ZEBRA_MIN_FEC_LENGTH should be 5

If the client registers for 0.0.0.0/0 the length will be 5

Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>

bgpd: add ENHE support for "address-family ipv4 labeled-unicast"

Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>

Merge pull request #494 from donaldsharp/community.md

*: Modify documentation to show people where to work

*: Modify documentation to show people where to work

Give some guidance to people on where they should work for
bug fixese and features.

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

Merge remote-tracking branch 'origin/stable/3.0'

Merge pull request #486 from LabNConsulting/working/3.0/patch/issue483

bgp rfapi: rfapi shouldn't be called (yet) for BGP VRF instances.

Merge pull request #480 from donaldsharp/agentx

Agentx

Merge pull request #488 from donaldsharp/sudoers3

*: Remove ability to install frr_sudoers for 3.0

eigrpd: spatch thread_add* calls and macros

As described in previous commits

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

ospf6d, zebra: remove unnecessary asserts

These pointers are now guaranteed to be null, so asserting the opposite
will crash.

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

*: update thread_add_* calls

Pass pointer to pointer instead of assigning by return value. See
previous commit message.

To ensure that the behavior stays functionally correct, any assignments
with the result of a thread_add* function have been transformed to set
the pointer to null before passing it. These can be removed wherever the
pointer is known to already be null.

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

lib: safely check & set thread pointers

When scheduling a thread, the scheduling function returns a pointer to
the struct thread that was placed on one of the scheduling queues in the
associated thread master. This pointer is used to check whether or not
the thread is scheduled, and is passed to thread_cancel() should the
daemon need to cancel that particular task.

The thread_fetch() function is called to retrieve the next thread to
execute. However, when it returns, the aforementioned pointer is not
updated. As a result, in order for the above use cases to work, every
thread handler function must set the associated pointer to NULL. This is
bug prone, and moreover, not thread safe.

This patch changes the thread scheduling functions to return void. If
the caller needs a reference to the scheduled thread, it must pass in a
pointer to store the pointer to the thread struct in. Subsequent calls
to thread_cancel(), thread_cancel_event() or thread_fetch() will result
in that pointer being nulled before return. These operations occur
within the thread_master critical sections.

Overall this should avoid bugs introduced by thread handler funcs
forgetting to null the associated pointer, double-scheduling caused by
overwriting pointers to currently scheduled threads without performing a
nullity check, and the introduction of true kernel threads causing race
conditions within the userspace threading world.

Also removes the return value for thread_execute since it always returns
null...

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

*: remove THREAD_ON macros, add nullity check

The way thread.c is written, a caller who wishes to be able to cancel a
thread or avoid scheduling it twice must keep a reference to the thread.
Typically this is done with a long lived pointer whose value is checked
for null in order to know if the thread is currently scheduled.  The
check-and-schedule idiom is so common that several wrapper macros in
thread.h existed solely to provide it.

This patch removes those macros and adds a new parameter to all
thread_add_* functions which is a pointer to the struct thread * to
store the result of a scheduling call. If the value passed is non-null,
the thread will only be scheduled if the value is null. This helps with
consistency.

A Coccinelle spatch has been used to transform code of the form:

  if (t == NULL)
    t = thread_add_* (...)

to the form

  thread_add_* (..., &t)

The THREAD_ON macros have also been transformed to the underlying
thread.c calls.

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

*: Remove ability to install frr_sudoers

If the user were to uncomment last line
and allow VTYSH_SHOW to be used as a non-root
account, this would allow arbitrary command completion
inside of vtysh via multiple -c ... -c .... lines

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

Merge pull request #485 from LabNConsulting/working/3.0/patch-set/missing1

Pick up a couple of patches to master that are needed on 3.0

bgp rfapi: rfapi shouldn't be called (yet) for BGP VRF instances.

bgpd: restore  -S, --skip_runas options

Signed-off-by: Lou Berger <lberger@labn.net>

bgp: fix a couple of instances of bm being used before init'ed

Signed-off-by: Lou Berger <lberger@labn.net>

Merge pull request #478 from opensourcerouting/test-extension

tests: allow to use exit_cleanly as test method

Merge pull request #469 from qlyoung/fix-list-perm-3.0

lib: fix 'list permutations' (3.0)

Merge pull request #454 from dwalton76/hide-router-bgp-peer-ipv4-unicast-commands

bgpd: hide "router bgp" commands also under "address-family ipv4 unic…

Merge pull request #468 from qlyoung/fix-list-perm

lib: fix 'list permutations'

Merge pull request #435 from chiragshah6/pim_dev

pimd: Introduce show command for protocol counters

debian: Mark FRR as conflicting with Quagga

FRR when compiled with this debian packaging will cause
some issue with the system to work properly due to the
sudoers file conflicts.

So to prevent some level of stupidity, mark them as
conflicting.

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

*: Allow agentx to be displayed as part of vtysh

Modify code so that that vtysh can now accept the
agentx command for snmp support

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

Merge remote-tracking branch 'origin/stable/3.0'

Merge pull request #475 from chiragshah6/pim_dev_3_0

Pim related defect fixes

Merge pull request #464 from donaldsharp/datacenter

Datacenter

Merge remote-tracking branch 'origin/master' into datacenter

Merge pull request #477 from opensourcerouting/gitignore

*: ignore libtool library anotation files

*: ignore libtool library anotation files

Signed-off-by: Christian Franke <chris@opensourcerouting.org>