redhat: Fix redhat startup to actually read in unified config

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

Merge pull request #1362 from dslicenc/cm18309-label-delete

zebra: only pass mpls proto type if doing install

Merge pull request #1358 from opensourcerouting/isis-lsp_tick-fixes

Isis lsp_tick fix and improve perfomance for processing LSP updates

zebra: only pass mpls proto type if doing install

Problem reported with not deleting LSPs from the zebra kernal mpls table
when a delete occurred in bgp.  Found that we were exiting the delete
process incorrectly due to not being able to derive the route_type from
the best_nhlre on the lsp while deleting. Since this info was only
needed for route installation, removed this early exit in the case of
deleting the lsp.

Signed-off-by: Don Slice <dslice@cumulusnetworks.com>
Ticket: CM-18309
Reviewed By: CCR-6781
Testing Done:  Manual testing looks good.  mpls tests successful

Merge pull request #1361 from opensourcerouting/vtysh_mem_leak

vtysh: properly fix memory leak

vtysh: properly fix memory leak

Commit 44f12f20 fixed the memory leak in the wrong way and introduced a
"uninitialized variable" warning.

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

Merge pull request #1357 from opensourcerouting/coverity_fixes

Coverity fixes

lib: optimize sockunion_connect()

This function is only called with non-blocking sockets [1], so there's
no need to worry about setting O_NONBLOCK and unsetting it later if the
given fd was a blocking socket. This saves us 4 syscalls per connect,
which is not much but is something.

Also, remove an outdated comment about the return values of this
function. It returns a 'connect_result' enum now, whose values are
self-explanatory (connect_error, connect_success and connect_in_progress).

This also fixes a coverity scan warning where we weren't checking the
return value of the fcntl() syscall.

[1] bgp_connect() and pim_msdp_sock_connect().

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

*: fix coverity warnings - error handling issues

Ignore the return value of some functions in the places we know they
can't fail, and other small fixes.

Regarding the change in bgpd/rfapi/rfapi_rib.c, asserting that
rfapiRaddr2Qprefix() didn't fail is the common idiom inside the rfapi
code.

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

*: fix coverity warnings - resource leaks

These are mostly trivial fixes for leaks in the error path of some functions.

The changes in bgpd/bgp_mpath.c deserves a bit of explanation though. In
the bgp_info_mpath_aggregate_update() function, we were allocating memory
for the lcomm variable but doing nothing with it. Since the code for
communities, extended communities and large communities is pretty much
the same in this function, it's clear that this was a copy and paste
error where most of the ext. community code was copied but not all of
it as it should have been.

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

ospfd: fix coverity warnings - security best practices violations

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

*: add missing \n in some help strings

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

lib: fix coverity warnings introduced by the iface rb-tree conversion

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

Merge pull request #1346 from donaldsharp/apply_routemap

bgpd: Note routemap information even if peer not activated.

Merge pull request #1337 from donaldsharp/debug_nht_bgp

bgpd: Allow turn off of 'debug bgp nht' from 'no debug bgp'

Merge pull request #1296 from donaldsharp/eigrp_table

eigrpd: Convert topology list to a table

eigrpd: Convert topology list to a table

The EIGRP topology list is an extremely inefficient
way to store data about the known routes.  Convert
to using a table.

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

Merge pull request #1330 from donaldsharp/zclient_shenanigans

Zclient shenanigans

Merge pull request #1354 from chiragshah6/mdev1

ospfd: show comand for ospf packet stats

bgpd: Use stored ifindex for zebra_announce

In some situations we already know the ifp and by extension
the ifindex there is no need to look it up for every
route we send to zebra.

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

zebra: Allow zebra_client_read to handle up to 10 messages

The zebra_client_read functionality was reading 1 message
from a peer at a time.  Modify the code so that we can
read up to 10 at a time.

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

*: Modify zclient_init to require privs data

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

ripngd: Make 'struct zebra_privs_t' available

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

ripd: Make 'struct zebra_privs_t' available

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

ospf6d: Make 'struct zebra_privs_t' available

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

ospfd: Make 'struct zebra_privs_t' available.

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

isisd: Allow struct zebra_privs_t to be available

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

eigrpd: Make struct zebra_privs_t available

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

nhrpd: Let struct zebra_privs_t be available

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

babeld: Make the zebra_privs_t available for use

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

zebra: Set zapi send/receive buffer to a larger value

Allow a bit more to buffer before we stop zebra
from sending/receiving.

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

lib: Cleanup some missed reformat

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

zebra: Add a function handler for zserv commands

The zserv command handlers make an already long function
even longer.  Isolate this code so that we can rearrange
the zebra_client_read function.

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

zebra: Make all zserv handler functions use zvrf instead of vrf_id

Some handler functions were using vrf_id, standardize onto
zvrf being passed around instead.

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

zebra: No need to pass sock descriptor around.

Standardize the api to read zapi requests from protocols.

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

zebra: Always read full zapi message in ptm code

The ptm code when it encountered an error situation
was not fully reading all the data in the stream
meant for it.  Ensure that this is read.

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

ospfd: reset ospf interface protocol stats

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

ospfd: show comand for ospf packet stats

Display OSPFv2 Protocol packets stats per interface.

tor-1# show ip ospf vrf all interface traffic

Interface    HELLO    DB-Desc   LS-Req LS-Update   LS-Ack
             Rx/Tx    Rx/Tx     Rx/Tx  Rx/Tx       Rx/Tx
------------------------------------------------------------
swp1         1/0      2/3       1/1    2/2         1/1
swp2         6/0      2/7       1/1    1/4         3/2

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

Merge pull request #1341 from chiragshah6/ospf_vrf_dev

ospf6d: Divide LSupdate to keep size small

Merge pull request #1353 from opensourcerouting/rpm-fix-master

Rpm fix master

Merge pull request #1349 from opensourcerouting/bgpd_listen_queue

bgpd: bump listen() backlog

redhat: Fix doc by removing outdated package information and point to main doc/ directory for it

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

redhat: Fix rpm scripts to correctly restart daemons on upgrade

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

redhat: Add fix to correct watchfrr config during upgrade

- /etc/frr/daemon is a config file and won't be replaced, do quick sed fix
- Updated changelog

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>
Conflicts:
redhat/frr.spec.in

redhat: systemd file should not be executable

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

redhat: Always reference main frr start/stop script in /usr/lib/frr

* Fixes issue where daemon default config tried to use /etc/init.d/frr script which doesn't exist on systemd systems

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>
Conflicts:
redhat/daemons

ospf6d: Divide LSupdate to keep size small

Within OSPFv3 area, Disect Router LSA and
Intra-prefix LSA in order to keep LSA size Small.
Each LSA has unique Link State ID assigned.

Intra-Area-Prefix LSA:
Spread prefixes across multiple intra-area-prefix-LSAs.

Ticket:CM-18069
Testing Done:
Tested 92 ospf6 enabled (point-to-point) interfaces
between two routers.
92 adajancy comes up with Full Neighborship.
Validated 'show ipv6 ospf6 database router detail' &
'show ipv6 ospf6 database intra-prefix detail', each adv-router
has two distinct LSA of such catgory.

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

bgpd: bump listen() backlog

Handle better stress situations when multiple peers are trying to
connect at the same time by bumping the TCP connection backlog limit.

This reduces the convergence time of BGPerf stress test.

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

Merge pull request #1347 from medallia/all-routes-to-fpm

Send all routes to FPM, including VRF and multicast

bgpd: Note routemap information even if peer not activated.

If upon bgp startup we have this config:

router bgp 64540
 neighbor 192.168.201.134 remote-as external
 !
 address-family ipv4 unicast
  no neighbor 192.168.201.134 activate
  neighbor 192.168.201.134 route-map NEXTHOP in
 exit-address-family

The route-map map pointer for the incoming(or outgoing)
filter was not being saved due to a pre-mature optimization
of not handling the routemap callback if the peer is not
activated.  The function that handles the peers route-maps
is making sure that the peer is in established state
before attempting to actually apply anything so just
call it to set the map pointer.

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

Send all routes to FPM, including VRF and multicast

Signed-off-by: Thorvald Natvig <thorvald@medallia.com>

Merge pull request #1331 from donaldsharp/hash_speedup

lib: Allow hash_get to sidestep expensive hash key generation in some…

Merge pull request #1332 from chiragshah6/ospf_vrf_dev

ospf6d: Handle lsupdate upto max ospf6 payload

Merge pull request #1272 from donaldsharp/peer_group_ordering

bgpd: Fix peer group copying of data for late activation

Merge pull request #1339 from donaldsharp/bgp_compile_issues

Bgp compile issues

bgpd: Fix 'not initialized' possibly on older compilers

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

bgpd: Fix off by one error introduced by 8c9cc7bbf657e3440d9bc758fe45aef5f43c989f

Commit 8c9cc7bbf657e3440d9bc758fe45aef5f43c989f changed the size
of the `struct bgp_attr_encap_subtlv` type to be a zero length
array at the end instead of having a 1 byte.  All memory allocations
for this subsuquently were off by 1 byte since those were not
adjusted either.

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

bgpd: Allow turn off of 'debug bgp nht' from 'no debug bgp'

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

Merge pull request #1333 from donaldsharp/command_py_leak

lib: Fix small memory leak when using command_py.c

Merge pull request #1334 from donaldsharp/large_number

Large number

bgpd: Display unsigned ints

We are storing data as an unsigned int and we should not
display it as a signed number.

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

lib: Display unsigned instead of signed

When displaying thread cpu data, display unsigned instead
of signed data when we get really really really large
numbers of invocations.

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

lib: Fix small memory leak when using command_py.c

When free'ing memory associated with the wgraph, also
free memory malloced during the initialization.

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

Merge pull request #1308 from chiragshah6/mdev1

ospfd: fix ospf nssa command

ospf6d: Handle lsupdate upto max ospf6 payload

While traversing LSUpdate list, if LSUpdate message
fills up (w/ LSAs) to the interface MTU size,
Send the existing filled LSUpdate message if max
payload exceeds max ospf6 payload,
reset sendbuf (buffer) to fill rest of
the remaining LSAs.

Add relevant fields to debugs.

Ticket:CM-18069

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

Merge pull request #1290 from qlyoung/doc-commit-msgs

doc: add docs for commit message format

Merge pull request #1300 from chiragshah6/mdev

ospf6d: fix ospf6d crash during sigterm/shutdown

Merge pull request #1324 from donaldsharp/bgp_aspath

bgpd: Add ability to display information about bestpath in json

lib: No need to call apply_mask 2 times

route_node_set is only called by route_node_get
which calls apply_mask.  There is no need to do
this again.

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

lib: Allow hash_get to sidestep expensive hash key generation in some cases

There is no need to generate a hash key *if* the hash_alloc_function
is NULL and the hash is empty.

This changed showed a measurable increase in performance for
table hash lookup for tables that were meant to be empty in
bgp( the distance commands ).

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

ospfd: fix ospf nssa command

-Fix ordering of nssa command with translate options
and no-summary option.
Just like ospf stub no-summary keep the order order
of nssa no-summary.
- Fix NSSA options.
- Avoid displaying translate-candiate (default) option
in running-config.

cumulus(config-router)# area 2.2.2.2 nssa
  <cr>
  no-summary           Do not inject inter-area routes into nssa
  translate-always     Configure NSSA-ABR to always translate
  translate-candidate  Configure NSSA-ABR for translate election (default)
  translate-never      Configure NSSA-ABR to never translate

Running-config output:
router ospf
 area 2.2.2.2 nssa translate-always
 area 2.2.2.2 nssa no-summary

Ticket:CM-8312

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

Merge pull request #1313 from LabNConsulting/working/master/patch-set/block-non-core-fix-assert

bgpd: only allow unicast|multicast config commands in non-core BGP instances

Merge pull request #1298 from opensourcerouting/iface-rb-tree

Use rb-trees to store interfaces instead of linked-lists

Merge pull request #1305 from donaldsharp/workqueue

lib: Free workqueue memory leak on free

Merge pull request #1317 from donaldsharp/babel_mem

babeld: Do not copy beyond end of data

lib: Free workqueue memory leak on free

When free'ing the workqueue if you have items
on the workqueue you should free the memory associated
with it.

Additionally move the work_queue_item_remove function
to allow for static to be awesome

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

Merge pull request #1322 from donaldsharp/ospf6_proto

ospf6d: Make ospf6_prefix have an empty v6 addr at end

ospf6d: Make ospf6_prefix have an empty v6 addr at end

The `struct ospf6_prefix` type expects to have space allocated
at the end of the structure for a v6 prefix.  So let's tell
the compiler that there might be more there.  This is to fix
a coverity scan warning.

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

bgpd: Add ability to display information about bestpath in json

When retrieving information about neighbors or summary in json
add some information about what bestPath particulars we have for
this bgp session.

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

Merge pull request #1120 from donaldsharp/branch_million

configure: Have --enable-dev turn on better debugging.

Merge pull request #1323 from donaldsharp/ospf_list_delete

ospfd: Fix ships in the night PR's and list_delete

ospfd: Do not delete write_q 2 times either

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

ospfd: We should not be deleting ospf->oiflist 2 times

The ospf->oiflist deletion is happening 2 times now.
Let's delete it once only.

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

ospfd: Fix ships in the night PR's and list_delete

list_delete_and_null should be used.

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

Merge pull request #1304 from chiragshah6/ospf_vrf_dev

ospfd: OSPFv2 VRF, add vrf_name in show command output

Merge pull request #1316 from donaldsharp/high_stuff

High stuff

Merge pull request #1319 from opensourcerouting/ldpd-use-after-free

ldpd: fix heap-use-after-free at exit

ospfd: OSPFv2 VRF abstact api

Create an api to display vrf name in vty or json format.
Use VRF Name for vtysh and vrfName for json output.

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

Merge pull request #1314 from donaldsharp/pim_rpf_address

Pim rpf address

ospfd: OSPFv2 VRF, vrf in show command output

Add VRF name in both vtysh and json format of
show commands.

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

ospfd: Free memory associated with ospf instance startup

We have memory that is initialized upon ospf instance
and area startup.  Free it up on shutdown.

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

ripngd: Fix possible memory leak of route_node

There exists a path where we do not save the route_node
after creation.  Allow cleanup.

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

lib: Add some documentation about argv_find

We expect that the index value passed in for argv_find
should be initially set to 0.  This way if the cli
ever changes there is no need to modify the initial
value.

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

lib: Fix missing va_end

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

ospfd: Fix leak of route_node

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

ospfd: Fix leak of stream

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

ospfd: Fix memory leak

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

ospfd: Fix leak of FD

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

pimd: Fix uninited idx for argv_find

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

isisd: Fix leaked memory in error case

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