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>

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

lib: Fix abbreviation

The word Multiplier has been abbreviated to 'Mul' in
the output.  This apparently is causing people
angst.  Write word out.

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

lib: remove extra / in other paths

Non-integrated config paths need the treatment as well.

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

Merge pull request #1236 from donaldsharp/interface_startup

zebra: Do not allow same rib_dest_t be queued multiple times to meta …

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

Being able to filter the output of some "show" commands is super useful
when troubleshooting large MPLS networks.

Examples:

% show mpls ldp ipv4 binding 10.0.0.48/29 longer-prefixes neighbor 192.168.0.5
AF   Destination          Nexthop         Local Label Remote Label  In Use
ipv4 10.0.0.48/30         192.168.0.5     24          26                no
ipv4 10.0.0.52/30         192.168.0.5     25          27                no

% show mpls ldp ipv4 binding 10.0.0.48/29 longer-prefixes neighbor 192.168.0.5 detail json
{
  "10.0.0.48\/30":{
    "localLabel":"24",
    "advertisedTo":[
      {
        "neighborId":"192.168.0.5"
      }
    ],
    "remoteLabels":[
      {
        "neighborId":"192.168.0.5",
        "label":"26",
        "inUse":0
      }
    ]
  },
  "10.0.0.52\/30":{
    "localLabel":"25",
    "advertisedTo":[
      {
        "neighborId":"192.168.0.5"
      }
    ],
    "remoteLabels":[
      {
        "neighborId":"192.168.0.5",
        "label":"27",
        "inUse":0
      }
    ]
  }
}

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

ldpd: improve ordering of interfaces on user output

Before:

debian# show mpls ldp interface
AF   Interface   State  Uptime   Hello Timers  ac
ipv4 rt0-eth0    ACTIVE 00:00:05 5/15           1
ipv4 rt0-eth1    ACTIVE 00:00:05 5/15           1
ipv4 rt0-eth10   ACTIVE 00:00:05 5/15           1
ipv4 rt0-eth11   ACTIVE 00:00:05 5/15           1
ipv4 rt0-eth2    ACTIVE 00:00:05 5/15           1
ipv4 rt0-eth3    ACTIVE 00:00:05 5/15           1
ipv4 rt0-eth4    ACTIVE 00:00:05 5/15           1
ipv4 rt0-eth5    ACTIVE 00:00:05 5/15           1
ipv4 rt0-eth6    ACTIVE 00:00:05 5/15           1
ipv4 rt0-eth7    ACTIVE 00:00:05 5/15           1
ipv4 rt0-eth8    ACTIVE 00:00:05 5/15           1
ipv4 rt0-eth9    ACTIVE 00:00:05 5/15           1

After:

debian# show mpls ldp interface
AF   Interface   State  Uptime   Hello Timers  ac
ipv4 rt0-eth0    ACTIVE 00:00:14 5/15           1
ipv4 rt0-eth1    ACTIVE 00:00:14 5/15           1
ipv4 rt0-eth2    ACTIVE 00:00:14 5/15           1
ipv4 rt0-eth3    ACTIVE 00:00:14 5/15           1
ipv4 rt0-eth4    ACTIVE 00:00:14 5/15           1
ipv4 rt0-eth5    ACTIVE 00:00:14 5/15           1
ipv4 rt0-eth6    ACTIVE 00:00:14 5/15           1
ipv4 rt0-eth7    ACTIVE 00:00:14 5/15           1
ipv4 rt0-eth8    ACTIVE 00:00:14 5/15           1
ipv4 rt0-eth9    ACTIVE 00:00:14 5/15           1
ipv4 rt0-eth10   ACTIVE 00:00:14 5/15           1
ipv4 rt0-eth11   ACTIVE 00:00:14 5/15           1

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

Merge pull request #1261 from dwalton76/bgpd-4byte-display

bgpd: fix 4-byte AS display in bestpath-from-AS

vtysh: more multi-instance fixing

Preexisting code to detect commands targeted at wrong instance didn't
work because a command applied to a disconnected vtysh_client counted as
a successful run overriding wrong instance error message

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

pimd: Add 'show ip mroute fill' command

When displaying the mroute table, add ability to fill
in all the missing data that for human eyes can be assumed.
This will allow people using grep over the command
can get more information.

sharpd@robot ~> sudo vtysh -c "show ip mroute fill"
Source          Group           Proto  Input      Output     TTL  Uptime
*               229.1.1.1       STATIC dum0       enp3s0     1    00:11:07 Default-IP-Routing-Table
*               229.1.1.1       STATIC dum0       virbr0     1    00:01:47 Default-IP-Routing-Table
sharpd@robot ~>

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

ospfd: fix ospf virtual-link command options

use argv_find to extract ospf area virtual-link command options.

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

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

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

Merge pull request #1180 from dwalton76/ipv6-static-route-null0

Ipv6 static route null0

Merge pull request #1247 from qlyoung/mcast-boundary-igmp

pimd: add support for boundaries

bgpd: fix 4-byte AS display in bestpath-from-AS

Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>
Before
======
cel-redxp-10# show ip bgp 20.1.3.0/24
BGP routing table entry for 20.1.3.0/24
Paths: (1 available, best #1, table Default-IP-Routing-Table)
  Advertised to non peer-group peers:
  top1(10.1.1.2) bottom0(20.1.2.2)
  4294967292
    20.1.2.2 from bottom0(20.1.2.2) (20.1.1.1)
      Origin IGP, metric 0, localpref 100, valid, external,
bestpath-from-AS -4, best
      Community: 99:1
      AddPath ID: RX 0, TX 92
      Last update: Wed Sep 27 16:02:34 2017

cel-redxp-10#

After
=====
cel-redxp-10# show ip bgp 20.1.3.0/24
BGP routing table entry for 20.1.3.0/24
Paths: (1 available, best #1, table Default-IP-Routing-Table)
  Advertised to non peer-group peers:
  bottom0(20.1.2.2)
  4294967292
    20.1.2.2 from bottom0(20.1.2.2) (20.1.1.1)
      Origin IGP, metric 0, localpref 100, valid, external,
bestpath-from-AS 4294967292, best
      Community: 99:1
      AddPath ID: RX 0, TX 2
      Last update: Wed Sep 27 16:07:09 2017

cel-redxp-10#

debian: Fix logrotate when FRR is not running

Fix the logrotate script to complete successfully even
if FRR is not currently running.

Signed-off-by: Dave Olson <olson@cumulusnetworks.com>

vtysh: 'x not running' redux

when warning about daemons that are not running, make sure to handle the
multi instance case

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

zebra: split ip route DEFUNs for blackhole, reject, etc

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

Merge pull request #1201 from dwalton76/bgpd-debug-updates-prefix

Bgpd debug updates prefix

Merge branch 'master' of https://github.com/dwalton76/frr into bgpd-debug-updates-prefix

Merge pull request #1257 from donaldsharp/pim_bp

pimd: Some compilers are complaining about my bp

Merge branch 'master' into bgpd-debug-updates-prefix

bgpd: remove clippy references in bgp_debug.c

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

Merge pull request #1242 from donaldsharp/eigrp_tm

Eigrp tm

Merge pull request #1240 from donaldsharp/allow_self_delete

zebra: Do not allow delete of route from kernel in non-startup case

pimd: Some compilers are complaining about my bp

Fix the bp variable possibly ( yeah not really ) be
initialized upon use.

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

vtysh: remove extra / in config path

before:

frrdev# do wr
Note: this version of vtysh never writes vtysh.conf
Building Configuration...
Integrated configuration saved to /etc/frr//frr.conf
[OK]
frrdev#

after:

frrdev# do wr
Note: this version of vtysh never writes vtysh.conf
Building Configuration...
Integrated configuration saved to /etc/frr/frr.conf
[OK]
frrdev#

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

Merge pull request #1249 from donaldsharp/debugodebug

Debugodebug

Merge pull request #1208 from donaldsharp/multiaccess

bgpd: Reduce multiaccess_check_v4 overhead for subgroups

Merge pull request #1243 from qlyoung/tm-destroy-cond

lib: destroy threadmaster condition variable