Merge pull request #2637 from donaldsharp/pim_malloc_me

pimd: Remove unnecessary alloc failures

Merge pull request #2642 from donaldsharp/deprecated

ospfd: Remove deprecated code

Merge pull request #2632 from ppmathis/bugfix/bgp-autoshut

bgpd: Fix bgp auto-shutdown behavior

ospfd: Remove deprecated code

This code has been deprecated for over a year, let's remove it now.

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

pimd: Remove unnecessary alloc failures

Remove from pim unnecessary alloc failure testing
as that alloc failure will cause an assert.

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

bgpd: Fix bgp auto-shutdown behavior

The current behavior of the `bgp default shutdown` command is to set the
state of all newly configured peers to shutdown. This leads to a problem
when restarting bgpd, because all peers will then be seen as newly
configured, which leads to all peers being set to shutdown after each
restart.

This behavior is undesired and not common when comparing the
implementation against other vendors. This commit moves the `bgp default
shutdown` configuration underneath the peer-group and peer
configuration, to ensure that existing peers will not be set to shutdown
after a daemon restart.

Signed-off-by: Pascal Mathis <mail@pascalmathis.com>

Merge pull request #2630 from donaldsharp/hashables

revert hash_walk/iterate optimizations

Merge pull request #2628 from pguibert6WIND/realpath_debug

lib: add the pathname to get the realpath in case of failure

Merge pull request #2618 from pacovn/PVS-Studio_library_warnings

pimd: standard library usage (PVS-Studio)

Merge pull request #2605 from pacovn/PVS-Studio_null_check

ospfd  pimd: null check (PVS-Studio)

vtysh: Remove unused variable

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

lib: Add some documentation

Add some documentation to hash_walk/iterate to tell people
to not do something stupid.

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

Revert "lib: A small optimization for the hash iterate and walk functions"

This reverts commit fc61644e440c875eefa222ab34d726c6281ca806.

lib: add the pathname to get the realpath in case of failure

Sometimes, the file under /var/run/netns may not be authorised to be
read ( because it is not read permission for frr user, for instance).
so it is good to know what happened.

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

pimd: standard library usage (PVS-Studio)

fread() function was not used properly.

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

ospfd pimd: null check (PVS-Studio)

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

Merge pull request #2602 from pacovn/PVS-Studio_element_overflow

bgpd zebra: element overflow (PVS-Studio)

Merge pull request #2623 from pacovn/PVS-Studio_memcpy_source_underflow

ldpd: buffer underflow, thread safety (PVS-Studio)

Merge pull request #2568 from pguibert6WIND/doc_mpls

doc: add MPLS configuration commands

Merge pull request #2624 from donaldsharp/PIM_ZOMILY_ZOM

pimd: Use generated interface name

doc: add MPLS configuration commands

Add MPLS information to statically handle label from zebra.

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

pimd: Use generated interface name

When we have a generated interface name, use it instead of the actual
pointer.

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

Merge pull request #2614 from rtrlib/2018-07-02-master-bugfix

bgpd: fix rpki validation for ipv6 (master)

ldpd: buffer underflow, thread safety (PVS-Studio)

This commit fixes two issues:
- memcpy() using containers of different sizes when using addr2sa(), mixing
  'struct sockaddr_storage' and 'union sockunion'.
- addr2sa() function not being thread safe (using a local static variable as
  container.

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

Merge pull request #2620 from pacovn/PVS-Studio_null_check_2

bgpd lib ospfd pimd ripngd: null chk (PVS-Studio)

Merge pull request #2610 from qlyoung/fix-zebra-pthread-crash-shutdown-min

zebra: remove asserts on task nullity

Merge pull request #2619 from pguibert6WIND/show_ipv6_bgp_wrong

doc: bgp wrong command displayed

Merge pull request #2615 from chiragshah6/evpn_dev

bgpd: l3vni del to free ip prefix routes from vrf

doc: clean up flowspec.rst

* Fix broken citations
* Remove trailing whitespace
* Rewrap to 80 lines
* Tweak capitalization of section headers
* Clean up a few indented blocks

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

Merge pull request #2598 from dslicenc/cm21429-vrf-dup

zebra: duplicate nexthops for cross-vrf static routes on vrf bounce

Merge pull request #2599 from pacovn/PVS-Studio_Array_overrun

ospfd zebra: array overrun fixes (PVS-Studio)

bgpd: fix rpki validation for ipv6

Signed-off-by: Marcel Röthke <marcel.roethke@haw-hamburg.de>

bgpd lib ospfd pimd ripngd: null chk (PVS-Studio)

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

doc: bgp wrong command displayed

This command does not exist. Move it to the right one.

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

ospfd zebra: array overrun fixes (PVS-Studio)

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

Merge pull request #2516 from rubenk/more-spec-cleanups

More spec cleanups

Merge pull request #2609 from pacovn/clang_scan_nhrpd_vici_initialization

nhrpd: uninitialized variable (Clang scan)

bgpd: l3vni del to free ip prefix routes from vrf

In Symmetric routing case, L3VNI stores evpn MAC_IP routes
as IP_PREFIX routes in associated bgp_vrf and fib.

When vxlan device for l3vni goes down, triggers l3vni delete
in bgp.
As part l3vni delete, evpn ip prefix routes associated
with the vni need to be withdrawn from zebra as well
bgpinfo needs to be freed.
bgp_delete does not free bgp_info associated
to evpn ip prefix routes (link to bgp_vrf).
Call to uninstall_evpn_route_entry_in_vrf() properly
cleanup bgp_info as well triggers appropriate updates.

Ticket:CM-21443
Testing Done:
On DUT, bringup symmetric routing configuration, learn
EVPN Type-2 and Type-3 Routes.
Type-2 MAC_IP routes will be stored as ip_prefix in vrf table
during l3vni bring up.

Remove L3vni, deletes all ip_prefix routes from the zebra, kernel
vrf route table and bgp_info is freed.

Check the show bgp memory stats for bgp_info post l3vni flap.

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

Merge pull request #2604 from pacovn/PVS-Studio_odd_operator_usage

nhrpd: odd operator usage fix (PVS-Studio)

Merge pull request #2603 from pacovn/PVS-Studio_format_warnings

eigrpd nhrpd ospfd pimd: fomat fixes (PVS-Studio)

nhrpd: uninitialized variable (Clang scan)

Fix over 0a939f4f24fa34fea688482fbf57fb16eaf2a081 (there was a case not
covered by previous fix)

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

zebra: remove asserts on task nullity

While ZAPI I/O threads make a best effort to kill any scheduled tasks on
their threadmasters, after death another pthread can continue to
schedule onto the threadmaster. This isn't a problem per se since the
tasks will never run, but it also means that asserting that it hasn't
happened is pointless.

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

bgpd zebra: element overflow (PVS-Studio)

The warning given by PVS-Studio is related to per-element overflow (there is
no real overflow, because of how elements are mapped in the union). This
same warning is typically reported by Coverity, too.

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

Merge pull request #2601 from pacovn/PVS-Studio_scope_overlap

ripngd: variable scope overlap fix (PVS-Studio)

Merge pull request #2600 from pacovn/PVS-Studio_sizeof_and_dead_code

eigrpd: sizeof() usage + dead code (PVS-Studio)

eigrpd nhrpd ospfd pimd: fomat fixes (PVS-Studio)

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

Merge pull request #2533 from pguibert6WIND/doc_flowspec

doc: add flowspec information from user point of view

nhrpd: odd operator usage fix (PVS-Studio)

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

ripngd: variable scope overlap fix (PVS-Studio)

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

eigrpd: sizeof() usage + dead code (PVS-Studio)

Two issues:

1) Removed an unnecesary null check of 'ei'
2) Fixed the usage of sizeof(), as it was used the size of a pointer instead
   of the size of the structure

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

zebra: duplicate nexthops for cross-vrf static routes on vrf bounce

Problem created by the fix for cm-21306 (inactive cross-vrf static routes
when vrfs were bounced.)  Determined that in another case, that fix would
cause duplicate nexthops to appear in the table. Resolved the problem by
removing the vrf static route process from the zebra "add" process leaving
it in the zebra " if up" process as added in cm-21306 since that's the point
that the vrf device is now functional.

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

doc: fix missing bgp keyword in debug bgp command

Some debug bgp commands were missing bgp keyword.

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

doc: add flowspec information from user point of view

Add some information for BGP flowspec.

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

Merge pull request #2475 from LabNConsulting/working/master/no_vrf_socket_4l3mdev

Don't open per vrf sockets when net.ipv4.tcp|udp_l3mdev_accept != 0

Merge pull request #2593 from rtrlib/2018-07-01-master-bugfix

bgpd: fix rpki segfault

bgpd: fix rpki segfault

If a cache server was added after rpki was started it's tr_socket would
not be initialized. This would lead to a segfault if the rtr manager
ever decides to switch to that socket or if rpki support is stopped.

Signed-off-by: Marcel Röthke <marcel.roethke@haw-hamburg.de>

Merge pull request #2592 from tigranmartirosyan/master

fixed #2567: Json status of BGP peer is not updated properly

fixed #2567: Json status of BGP peer is not updated properly

Merge pull request #2574 from pacovn/Coverity_1399202_Copy-paste_error

bgpd: copy-paste error (Coverity 1399202)

Merge pull request #2582 from donaldsharp/work_smarter_not_slower

bgpd: Remove HAVE_CUMULUS from evpn commands

Merge pull request #2588 from pacovn/Clang_dereference1

bgpd isisd: null check (Clang scan)

Merge pull request #2577 from pacovn/Coverity_1399377_Structurally_dead_code

bgpd: dead code (Coverity 1399377)

bgpd: dead code (Coverity 1399377)

Replacing "continue" with "break" because the loop is a "while (0)"

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

Merge pull request #2517 from pguibert6WIND/flowspec_other_options

Flowspec support other options

Merge pull request #2532 from donaldsharp/various_stuff

Redistribution and some extra developer debug code

Merge pull request #2583 from donaldsharp/more_warnings

zebra: Fix use of uninited variable

Merge pull request #2581 from qlyoung/fix-vtysh-autocomplete

vtysh: fix autocomplete garbage printouts

Merge pull request #2578 from pacovn/Coverity_1453455_Dereference_null_return_value

bgpd: null check (Coverity 1453455)

Merge pull request #2573 from pacovn/Coverity_23065_Explicit_null_dereferenced

bgpd: null check (Coverity 23065)

Merge pull request #2514 from pacovn/Coverity_1462467_1465497_Control_flow_issues

zebra: control flow issues (Coverity 1462467 1465497)

Merge pull request #2586 from pacovn/clang_scan_nhrpd_vici_initialization

nhrpd: uninitialized variable (Clang scan)

Merge pull request #2587 from pacovn/Clang_scan_dead_code

isisd zebra: dead code (Clang scan)

bgpd isisd: null check (Clang scan)

This correction fixes three bugs detected by Clang scan:

Bug Group: Logic error
Bug Type: Dereference of null pointer

File: bgpd/bgp_evpn.c
Function: bgp_evpn_unconfigure_import_rt_for_vrf
Line: 4246

File: isisd/isis_spf.c
Function: isis_print_paths
Line: 69 (two bugs of same type in one line)

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

Merge pull request #2562 from pacovn/Coverity_1221444_1221446_Out-of-bounds_read

ospf6d: OoB read (Coverity 1221444 1221446)

isisd zebra: dead code (Clang scan)

This correction fixes two bugs detected by Clang scan:

Bug Group: Dead store

Bug Type: Dead assignment
File: zebra/kernel_netlink.c
Function: netlink_parse_extended_ack
Line: 548

Bug Type: Dead increment
File: isisd/isis_lsp.c
Function: lsp_bits2string
Line: 625

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

nhrpd: uninitialized variable (Clang scan)

This correction fixes two bugs detected by Clang scan:

Bug Group: Logic error
Bug Type: Assigned value is garbage or undefined
File: nhrpd/vici.c
Function: vici_parse_message
Lines: 100, 105

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

Merge pull request #2575 from pacovn/Coverity_1399219_Copy-paste_error

bgpd: copy-paste error (Coverity 1399219)

bgpd: null check (Coverity 1453455)

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

bgpd: null check (Coverity 23065)

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

ospf6d: OoB read (Coverity 1221444 1221446)

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

bgpd: Add some asserts because of our linklist stuff

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

zebra: Fix use of uninited variable

Allow compiler to think the value is actually set to
something useful in an impossible case.

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

bgpd: Fix some build issues from removing HAVE_CUMULUS

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

bgpd: Use correct data size

a vni_t is 32 bit, so we only need to use a normal
integer for json commands.

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

bgpd: Remove HAVE_CUMULUS from evpn commands

In order to make EVPN behavior work without special casing
the code, bring the evpn commands under HAVE_CUMULUS into
the fold.

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

Merge pull request #2561 from pacovn/Coverity_1399231_Logically_dead_code

lib ospf6d: dead code (Coverity 1399231)

vtysh: fix autocomplete garbage printouts

The semantics for suppressing output received from daemons changed
slightly when pipe actions were introduced, causing raw autocomplete
output to be printed where it shouldn't have been.

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

Merge pull request #2576 from pacovn/Coverity_1399228_Logically_dead_code

bgpd: dead code (Coverity 1399228)

Merge pull request #2572 from pacovn/Coverity_1399203_Unchecked_return_value

bgpd: return value (Coverity 1399203)

Merge pull request #2579 from pacovn/Coverity_1424370_Unchecked_return_value_from_library

isisd: return check (Coverity 1424370)

Merge pull request #2580 from pacovn/Coverity_1424529_Dereference_after_null_check

isisd: null check (Coverity 1424529)

isisd: null check (Coverity 1424529)

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

isisd: return check (Coverity 1424370)

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

bgpd: dead code (Coverity 1399228)

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

bgpd: copy-paste error (Coverity 1399219)

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

bgpd: copy-paste error (Coverity 1399202)

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

bgpd: return value (Coverity 1399203)

The return value can be ignored because in case of error both the 'afi'
and 'safi' variables are set to the invalid values (AFI_MAX, SAFI_MAX),
and the invalid values are handled properly afterwards in the 'default'
blocks.

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

ospf6d: dead code (Coverity 1399231)

It should have the same behavior when debug not enabled, and slightly
different behavior when debug enabled (previously dead code now should
show debug messages in debug mode)

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

bgpd: rework enumerate function, handle not values

The handling of reverse values is in a separate function.

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

bgpd: add an icmp flag for flowspec icmp entries

Some values for icmp type/code can not be encoded like port source or
port destination. This is the case of 0 value that is authorized for
icmp.

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