bgpd: fix clang warning

stlv_last is initialized with the loops. No need to reset it.
Its scope is local to the use with the loops.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

Merge pull request #1307 from vjardin6WIND/clean

Fix many clang warning

eigrpd: fix clang warning

ei shall never be NULL once we do the lookup. If it is NULL
bad things could happen.
For instance ei->t_distribute may not be a valid reference.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

eigrpd: fix clang warning

nbr cannot be NULL. Doing such test does confuse clang since
we do not have execution case if nbr == NULL.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

bgpd: fix clang warning

Same fix than c31a793, alwas get the string so clang does not complain that
it can be NULL.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

vtysh: fix clang warning

cmd->line shall never be null for these sections.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

zebra: fix clang warning

Set to 0 to avoid "Branch condition evaluates to a garbage value".

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

lib: fix clang warning

Properly initialize to avoid "Branch condition evaluates to a garbage value"
warning.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

bgpd: fix clang warning

Remove stored value to llen that is never used.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

rfapi: fix clang warning

When DEBUG_L2_EXTRA is not set, clang emits a warning because rc is not
used.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

eirgpd: fix clang warning

opcode is not used.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

ripngd: fix clang warning

this value len is not used for this vty_out() case.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

ospfd: fix clang warning

us is clearly not used, but let's keep it to have a readable logic.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

ldpd: fix clang warning

size is not used for further parsing. Keep it updated but tell
to the compiler that we know it is not used just in case one
needs to extend the parsing somedays.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

isisd: fix clang warning

tlvs is not used after extracting the last TLV. So,
let's tell to the compiler that it is expected.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

ospfd: fix clang warning

Value stored to 'nbraddr' during its initialization is never read

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

ospf6d: fix clang warning

troute and tinfo needs to be 0'd

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

ospf6d: fix clang warning, NULL() calls

Avoid show() methods from being a NULL function when it is called.
Meanwhile, I did const'ify the handlers so only its debug field can be changed.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

rfapi: avoid calling NULL() function.

Keep the execution flow, bug log errors.

Signed-off-by: Vincent Jardin <vincent.jardin@6wind.com>

Merge pull request #1303 from opensourcerouting/ldpd-scan-build

ldpd: sprinkle in some asserts to make scan-build happy

ldpd: sprinkle in some asserts to make scan-build happy

While here, fix the other SA warnings.

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

Merge pull request #1295 from donaldsharp/more_1270

More 1270

Merge pull request #1299 from opensourcerouting/small_fixes

Small fixes

zebra: fix display of negative route metrics and tags

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

*: use argv[]->text instead of argv[]->arg when appropriate

Incomplete commands like "debug ospf6 route mem" were being ignored. The
changes in ripd and ripngd are intended to make the code easier to read,
no bugs were fixed in these two daemons.

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

lib, ldpd: fix "argument cannot be negative" coverity warnings

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

Merge pull request #1264 from donaldsharp/linklist

Linklist api cleanup

Merge pull request #1293 from donaldsharp/pim_boundary_oil

Pim boundary oil

*: Convert list_free usage to list_delete

list_free is occassionally being used to delete the
list and accidently not deleting all the nodes.
We keep running across this usage pattern.  Let's
remove the temptation and only allow list_delete
to handle list deletion.

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

*: Convert list_delete(struct list *) to ** to allow nulling

Convert the list_delete(struct list *) function to use
struct list **.  This is to allow the list pointer to be nulled.

I keep running into uses of this list_delete function where we
forget to set the returned pointer to NULL and attempt to use
it and then experience a crash, usually after the developer
has long since left the building.

Let's make the api explicit in it setting the list pointer
to null.

Cynical Prediction:  This code will expose a attempt
to use the NULL'ed list pointer in some obscure bit
of code.

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

vtysh: Fix ordering of multicast boundary command

The multicast boundary command must be shown after
the `ip pim sm` command.  So add a new config_add_line_end
and make it the last one.

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

pimd: Fix multicast boundary command

1) Fix missing newlines
2) Write the correct cli out.

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

bgpd: Fixup json with RouteDistinguishers

The json option for displaying a bgp table
with route Distinguishers in it was not properly
working.  This code cleans this issue up.

Additionally attempt to make the code a bit
easier to read and handle.

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

bgpd: Cleanup bgp_show_table to be easier on the eyes

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

bgpd: fix 'show bgp ipvX vpn rd' command

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

bgpd: fix show bgp ipvX vpn all rd

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

bgpd: remove old/broken vpnv4 from of show bgp

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

bgpd: unify 'show bgp' with RD with normal unicast bgp show

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

Merge pull request #1244 from donaldsharp/flush_routes

tools: Fix route flush

Merge pull request #1280 from donaldsharp/eigrp_interface

Eigrp interface cleanup

Merge pull request #1292 from chiragshah6/mdev

ospf6d: add debug ospf6 route memory to config

*: Add some documentation about how to handle new daemons

Add some breadcrumbs for when we add a new routing daemon
so things `just work`.

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

tools: Add NHRP to route flushing

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

tools: Fix babel RT_PROT #

We had mistakenly created another RT_PROT # for babel.
This is incorrect.

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

tools: Fix route flush

This commit does two things:

1) Flush by proto number not string
   This is useful because not all systems might have the proto
   values installed, or a version of iproute2 that they might
   be installed with.
2) Flush missing routes that might have been installed( eigrp
   and nhrp )

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

Merge pull request #1286 from opensourcerouting/isis-lsp-queue-improv

isisd: optimize per interface lsp send-queue creation

Merge pull request #1291 from sebageek/debian-pkg-add-iproute2-alternative

Add iproute2 as dependency alternative in Debian package

Merge pull request #1288 from donaldsharp/1287

doc: Added python-ipaddr dependancy

ospf6d: add debug ospf6 route memory to config

vty config write to display 'debug ospf6 route memory'.

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

Merge pull request #1289 from sebageek/fix-defualt-typo

Fix defualt-default typo in bgp evpn help text

debian: Add iproute2 as dependency alternative in Debian package

iproute has been a transitional package that only depends on iproute2
since Debian Jessie or Ubuntu 14.04. To avoid installing this transitional
dummy package on newer installations we add iproute2 as a dependency
alternative to iproute. The iproute dependency can be dropped when
wheezy / 12.04 support is no longer needed.

Signed-off-by: Sebastian Lohff <seba@someserver.de>

bgpd: Fix defualt-default typo in bgp evpn help text

Signed-off-by: Sebastian Lohff <seba@someserver.de>

doc: Added python-ipaddr dependancy

Added python-ipaddr apt requirement to documentation.
Added clarification to install systemd service section
as commands assume are in frr directory.

Signed-off-by: Will McLendon <wimclend@gmail.com>

Merge pull request #1283 from donaldsharp/ldp_bite_me

ldpd: Fix compiler warning about uninitialized rt

ldpd: Fix compiler warning about uninitialized rt

Certain compilers cannot recognize that rt is
actually being init'ed, but let's set it to
NULL 'till we get them updated.

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

Merge pull request #1282 from opensourcerouting/ldpd-minor-issues

ldpd minor issues

Merge pull request #1267 from opensourcerouting/ldpd_cli_show

ldpd: add more filtering options in some "show" commands

ldpd: detach stdin/stdout/stderr from the child processes

Doing a "ssh user@node 'ldpd -d'" was making the SSH session hang. In
the original OpenBSD's ldpd(8) daemon, the daemon function takes care
of connecting stdin/stdout/stderr to /dev/null. In the FRR port, this
only happens in the frr_run() function, after all children have been
forked. Ideally we could try to rearrange libfrr.c and ldpd.c in a way
that start_child() is called only after the parent connects the standard
I/O streams to /dev/null. But since this issue needs an immediate
fix, let's do this workaround for now. Note: even when running on the
foreground, all log messages from the child processes are sent to the
parent process, which then prints the messages to stdout/stderr and/or
to a log file.

Reported-by: Martin Winter <mwinter@opensourcerouting.org>
Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

ldpd: fix issue when displaying the running configuration

More than a cosmetic fix, this should prevent issues with frr-reload.py.

Before:

mpls ldp
 router-id 1.1.1.1
 !
 address-family ipv4
  discovery transport-address 1.1.1.1
  !
  interface rt1-eth0
   discovery hello holdtime 10
   discovery hello interval 3
 exit-address-family
 !
!

After:

mpls ldp
 router-id 1.1.1.1
 !
 address-family ipv4
  discovery transport-address 1.1.1.1
  !
  interface rt1-eth0
   discovery hello holdtime 10
   discovery hello interval 3
  !
 exit-address-family
 !
!

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

ldpd: remove the "discovery targeted-hello" commands from the interface node

These commands belong only in the global LDP node and in the LDP
address-family nodes.

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

ldpd: fix wrong indentation in the help menu

The -n option was not aligned with the other ones:

% ldpd --help
[snip]
  -A, --vty_addr     Set vty's bind address
  -P, --vty_port     Set vty's port number
      --ctl_socket   Override ctl socket path
-n,   --instance     Instance id

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

Merge pull request #1111 from chiragshah6/mdev

ospfd: OSPFv2 VRF Support

Merge pull request #1281 from donaldsharp/boo_memset

lib: Fix memset usage

ospfd: OSPFv2 VRF remove iflist from master

Remove instances of ospf master's iflist and use vrf_list
with ospf vrf_id.

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

ospfd: fix vrf bind sock non-linux system

For non GNU_LINUX like open Bsd avoid sending packet
to non vrf aware device using in_pktinfo.

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

ospfd: Bind socket to vrf device

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

ospfd: OSPFv2 VRF fixesI

Fix ospf interface running config to display vrf
fix clang warnings

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

ospfd: OSPFv2 VRF Changes II

Accomodate review comments

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

ospfd: fix clang warning & ci-ospf test

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

ospfd: OSPFv2 VRF, fix CLAN SA warning

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

ospfd: OSPFv2 VRF Support

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

lib: Fix memset usage

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

Merge pull request #1278 from chiragshah6/ospf_vrf_dev

ospfd: Fix maxage_lsa lookup key

Merge pull request #1263 from vjardin6WIND/cleanup

Cleanup - scan-build

Merge pull request #1262 from dwalton76/vtysh-conft-available2

tools: frr-reload.py should exit(1) if vtysh config mode is locked

isisd: optimize per interface lsp send-queue creation

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

eigrpd: Remove unnecessary macro definitions

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

eigrpd: Remove ei mapping to connected routes

We need one struct eigrp_interface per ifp structure
not a ifp->info structure with a ei per connected.

Some minor code cleanup as well with macros and
their weird usage.

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

eigrpd: Start fixing eigrp_interface

The EIGRP interface handling code created 2 levels
of interface information, the ei( created per *connected*
address?????) and the ifp->info pointer.

Start the combination of pushing these together and
cleaning up the weirdness that is these two data
structures.

This commit probably breaks eigrp's handling of
interfaces.

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

ospfd: Fix maxage_lsa lookup key

maxage_lsa using prefix structure to store in route_table.
The lsa pointer is 4-8 bytes depending on arch, but prefix member field
would be 1 byte. Use ptr field of prefix structure to store lsa pointer.
Also memset to initialize to avoid crash on ARM platform as same LSA is
referenced from multiple times during shutdown of ospf.

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

Merge pull request #1259 from qlyoung/vtysh-warn-mi

vtysh: 'x not running' redux

Merge pull request #1250 from chiragshah6/ospf_vrf_dev

ospfd: fix virtual-link config command

Merge pull request #1277 from donaldsharp/zebra_crash

Zebra crash and valgrind issues

ospfd: ospf use macro for authentication

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

Merge pull request #1254 from donaldsharp/irdp_no_crash

zebra: Fix irdp so it doesn't crash when looked at

zebra: Cleanup ptm memory *after* we tell the sockets to clean up

The zebra_ptm_finish() code was being called before the
client_list deletion.  The client_list deletion is
attempting to call the ptm daemon and shut down the connection.
We should not be doing this *after* we shut down memory associated
with it as that we were writing into memory in random spots
in this case.

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

zebra: Properly initialize memory for rtadv

The adata pointer was not properly being set to
0 before being used.  In addition notice malloc
failure and hard exit.  If we have no memory on
startup something terrible has gone wrong and
we were going to crash shortly here anyways.

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

Merge pull request #1276 from opensourcerouting/isis-fix-vertex-queue

isisd: fix vertex queue comparator

zebra: Fix valgrind report of unintialized data

When calling the kernel we are not properly initializing
iov and msg on function startup.

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

isisd: fix vertex queue comparator

While vertizes should be strictly ordered on insertion, deletion
will of course encouter equality.

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

scan-build: fix warning

Do not change the implementation, just fix scan-build/clang warning.

ospf_packet: scan-build/clang clean up

get the code safer using snprintf().

scan-build: cleanup some warnings

Current cleanup is for unset values or variables that are not used anymore.

Regarding ospfd/ospf_vty.c: argv_find()
  we'll never get it NULL, so get coststr = argv[idx]->arg;

sha256: fix clang warning

t0 and t1 are not used, but want do really want them to be reset to secure
the stack.
Suggested by Quentin: let's use memset() to be consistent.

Merge pull request #1260 from donaldsharp/logrotate

debian: Fix logrotate when FRR is not running

Merge pull request #1265 from donaldsharp/pim_fill

pimd: Add 'show ip mroute fill' command

Merge pull request #1271 from donaldsharp/multiplier

lib: Fix abbreviation

ospfd: fix area virtual-link command

Adjust to review comments

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

ospfd: ospf virtual-link fix

Initialize idx to 0 and remove reassignment of idx

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

Merge pull request #1255 from qlyoung/vtysh-remove-double-slash

vtysh: remove extra / in config path