bgpd: only allow unicast|multicast SAFIs in VRF/View (i.e., non-core) BGP instances

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

rfapi: block vnc commands on vrfs/views

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

rfapi: explicitly test against success of is_configured.

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

rfapi: remove vestigial assert

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

Merge pull request #1312 from donaldsharp/snmp_compile

ospf6d: Fix snmp compile breakage

ospf6d: Fix snmp compile breakage

The recent clang fixes for Static Analysis were run without
compiling `--enable-snmp` these changes broke the snmp build.

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

Merge pull request #1310 from vjardin6WIND/null_references

lib: linklist avoid access NULL->data

Merge pull request #1309 from vjardin6WIND/clean

Fix clang warnings

Merge pull request #1311 from vjardin6WIND/csv_unittest

lib: csv unit tests

lib: fix wrong warning from clang

The compiler cannot guess that rise() will not return here.
One should help.

Warning:
  Access to field 'file' results in a dereference of a null pointer
  (loaded from variable 'error')
aka error->file while error is NULL.

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

lib: csv unit tests

Fix csv unit tests.

To be run using,
  gcc -o csv csv.c -DTEST_CSV
  ./csv

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

lib: fix NULL->field_len access

Currenlty, this function is used only by:
  - unit test of csv.c (see its main() section)
  - ptm_lib.c

In case of ptm, it is safe to return NULL because:
  csv_encode_record() -> return NULL
     _ptm_lib_encode_header() -> return NULL
        the only consumer of the return value is: ptm_lib_init_msg()
        that checks the NULL return.

Warning:
  Access to field 'field_len' results in a dereference of a null
  pointer (loaded from variable 'fld')

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

lib: fix a64448ba, invalid NULL->num_labels

We should assume match OK only when neither nhl1
and neither nhl2 are NULL.
If both are NULL, it means match NOK.

Clang Warning:
  Access to field 'num_labels' results in a dereference of a null
  pointer (loaded from variable 'nhl1')

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

lib: linklist avoid access NULL->data

Let's assert(NULL) if the datastructure is not set.
The code assumes that the pointer is always non NULL. So, let's enforce
this semantic.

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

bgpd: fix aspath parsing

clang provides a notice about it that this p++ is useless,
because ++ would be done after the return.

From code review, I understand that p shall be incremented
for each token that is parsed from the buf. So let's keep
this intent.

Note that this commit is changing the behaviour of the source
code since from now p++ will be returned instead of p.
However, it does not hurt since the only consumer
just free() the aspath if it is parsed as as_token_unknown.
Let's be safe with a proper execution flow from now.

PS:
C reminders:

int f7(void) {
  int j = 7;

  return ++j; // return 8
}

int f8(void) {
  int j = 7;

  return j++; // return 7
}

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

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