Merge pull request #1943 from donaldsharp/update_pr_1942

ripd: considering a interface with 2 or more IP

Merge pull request #1941 from donaldsharp/cs1

Cs1

ripd: considering a interface with 2 or more IP

This commit fixes these three issues:

1) rinfo is used for rip packet sending not tmp_rinfo
2) With RIP_SPLIT_HORIZON and an interface with more than 1 ip addresses
we will not send the routes out an interface that they originate on
3) With RIP_SPLIT_HORIZON_POISONED_REVERSE and an interface with
more than 1 ip address we will not send out ipA with a metric of 16
and ipb with a metric of 1.  Both will be 16 now.

Signed-off-by: lyq140 <34637052+lyq140@users.noreply.github.com>

bgpd: No need to check return value of str2prefix_rd

The str2prefix_rd function can fail, but for auto-derived
values this should be impossible to happen.  So ignore it.

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

bgpd: Clean up irt a tiny bit

This commit does these 2 things:

1) irt->vrfs is never NULL so no need to test for it

2) No need to check for a good irt value returned from
vrf_import_rt_new as that the alloc operation will
dump if memory allocation fails.

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

zebra: Fix comparison to not look at itself

The nexthop vrf comparison should not be against itself

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

zebra: close fd when not needed

Upon decision that the vrf is not going to be supported
by namespaces close the fd.

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

ospf6d: Delete malloced on error path

We create route_to_del and then on the error path
we are not properly freeing it up.  Let's clean it
up for the goodness of mankind.

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

bgpd: peer->bgp must be non NULL

We lock and set peer->bgp at peer creation and only
remove it at deletion.  Therefore these tests are
not needed.

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

bgpd: More ecommunity string leak cleanup

Do not leak the ecommunity string created.

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

bgpd: Check for peer before dereferencing it

Check for the peer structure passed in to be valid
before we deref it.

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

zebra: Ensure that we properly decode the zapi_route sent to us

Ensure that we have properly decoded the zapi_route sent to us
and if we cannot decode, log and move on.

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

zebra: Fix memory leak on re-enter case

When we have a case where the user re-enters the same
ip route line, we need to delete the memory we just
malloc'ed.

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

Merge pull request #1825 from chiragshah6/ospfv3_dev

ospf6d: Intra-prefix LSA update after frr restart

Merge pull request #1933 from qlyoung/move-community-md

frr, doc: Move COMMUNITY.md to doc/developer/workflow.rst

Merge pull request #1913 from LabNConsulting/working/master/bgp-vpn-leak-cli

bgpd: new vpn-policy CLI

Merge pull request #1937 from LabNConsulting/working/master/bgp-vpn-leak-doc

doc: bgp vpn-vrf leaking CLI

Merge pull request #1932 from ajones-rvbd/ajones-issue-1930

redhat: build dist tarball in a chroot

Merge pull request #1909 from donaldsharp/zebra_rule

Zebra rule

Merge pull request #1936 from qlyoung/fix-static-route-config-write-missing-space

zebra: fix static route config write

bgp_vty.c: fix docstrings for bgp_imexport_vpn

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgpd: vpn-vrf-leaking new CLI: address Vivek's review comments + crash fix

    - vpn_leak_to_vpn_active(): check instance type
    - vpn_leak_prechange(): qualify with test for active
    - vpn_leak_postchange(): remove duplicated call to
vpn_leak_from_vrf_update_all()

    - bgp_vty.c: Avoid null-pointer dereference for command "no rt vpn import"

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

bgpd: new vpn-policy CLI

PR #1739 added code to leak routes between (default VRF) VPN safi and unicast RIBs in any VRF. That set of changes included temporary CLI including vpn-policy blocks to specify RD/RT/label/&c. After considerable discussion, we arrived at a consensus CLI shown below.

The code of this PR implements the vpn-specific parts of this syntax:

router bgp <as> [vrf <FOO>]
    address-family <afi> unicast
        rd (vpn|evpn) export (AS:NN | IP:nn)
        label (vpn|evpn) export (0..1048575)
        rt (vpn|evpn) (import|export|both) RTLIST...
        nexthop vpn (import|export) (A.B.C.D | X:X::X:X)
        route-map (vpn|evpn|vrf NAME) (import|export) MAP

        [no] import|export [vpn|evpn|evpn8]
        [no] import|export vrf NAME

User documentation of the vpn-specific parts of the above syntax is in PR #1937

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

doc: bgp vpn-vrf leaking CLI

Signed-off-by: G. Paul Ziemba <paulz@labn.net>

Merge pull request #1875 from ajones-rvbd/ajones-for-upstream

docker build: build Alpine Linux dev packages in docker

zebra: fix static route config write

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

docker build: build Alpine Linux dev packages in docker

Building alpine packages in a "standard" distro can be
complicated due to the limited scope of the distro (embedded
and small docker images).  Building in a VM is one possibility,
but docker support for alpine is very good (default docker images
come in alpine due to the very small size).

Here, we want to package up the current git repo into apk packages
that can be easily installed in alpine linux using the apk tool.
This support is not intended to package released versions of
apk packages, that, if it comes to be, should be done here:

git://git.alpinelinux.org/aports

We're content here to build packages that can be used by developers
to try out frr in docker and other alpine environments.

This is a very minimal environment, we don't support importing
keys (so, installing the packages with apk requires the
--allow-untrusted option).  In addition, we can't use the
git commit id in hex as version tag, as alpine doesn't support hex
digits in the version string.  So, we need to convert the git hash
to decimal before tagging the package with the extra version.
This is yucky, but I can't think of another way to get a
unique version per package.  The alpine way (using a numeric date),
only works for released packages, not for dev packages.

Issue: https://github.com/FRRouting/frr/issues/1859
Signed-off-by: Arthur Jones <arthur.jones@riverbed.com>

redhat: enable dist tarball to build in a chroot

When building the rpms, we can use a chroot (in my case docker) to
ensure that the BuildRequires are complete.  This test failed with
errors like:

    checking for CARES... no
    configure: error: trying to build nhrpd, but libcares not found. install c-ares and its -dev headers.
    error: Bad exit status from /var/tmp/rpm-tmp.FewvLf (%build)

This is due to a couple missing BuildRequires in the spec file.  Here, we
add those in for all RPM builds.

Testing done:

Ran a docker build on CentOS7 which succeeded.  Loaded the modules onto
CentOS6 to make sure they were at least valid there, that succeeded.

Issue: https://github.com/FRRouting/frr/issues/1930
Signed-off-by: Arthur Jones <arthur.jones@riverbed.com>

frr, doc: Move COMMUNITY.md to doc/developer/workflow.rst

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

Merge pull request #1908 from donaldsharp/peer_established

bgpd: Clean up peer status checking for a received nlri

Merge pull request #1924 from donaldsharp/cs

Cs

Merge pull request #1925 from opensourcerouting/bugfix-isis_bpf_read

isisd: Fixed the way isis reads from bpf

bgpd: Don't leak the ecommunity_ecom2str string in debug

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

zebra: Free memory leak

Free the memory leaked stream in failure cases.

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

pimd: Fix leaked fd and prevent null pointer deref

When the pim_nexthop_lookup fails, close the opened fd
as part of the failure condition.

Additionally pim_nexthop_lookup assumes that we've
actually already looked up a nexthop in the past.

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

zebra: Compare to the number of elements not size of array

When figuring out whom to call and if we actually can legally
call into the handler array actually use the number of elements
in the array instead of the size of the array.

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

zebra: Fix leaked fd.

When we detect an error condition, close down the opened
fd.

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

isisd: Fixed the way isis reads from bpf

With this fix, we parse the bpf to process every packet read

Signed-off-by: Ali Rezaee nlndipi@hotmail.com

Merge pull request #1911 from donaldsharp/mpls_love

Mpls love

Merge pull request #1920 from donaldsharp/rpik_more

bgpd: Fix warning on 32 bit systems

Merge pull request #1889 from donaldsharp/static_stuff1

zebra: Auto create the nexthop-vrf if needed.

Merge pull request #1914 from donaldsharp/fix_it

bgpd: Move back nh_vrf setting to where it was

Merge pull request #1918 from opensourcerouting/rpki-pkg-master

Rpki pkg master

bgpd: Fix warning on 32 bit systems

We have a signed/unsigned comparison warning that this should
fix.

This should be ok because the range of input is a very limited
value and should never be of concern

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

debianpkg: Fix - need to always run dh_gencontrol

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

debianpkg: Add option to build with RPKI

No RPKI support for Ubuntu 12.04

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

redhat: Add RPKI comment on build instructions

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

redhat: Add option to build with RPKI

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

Merge pull request #1898 from opensourcerouting/reload-helper2

tools: Add helper frr-reload shell script for clean error

Merge pull request #1890 from LabNConsulting/working/master/checkpatch1

checkpatch minor tweaks

zebra: Extend unknown vrf to vrf part of ip route

When specifying a ip route:

ip route 4.3.2.0/24 192.168.201.1 vrf DONNA

Accept DONNA even if it has not been created yet.

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

zebra: Allow duplicate entry of a route

If a user enters a route inside a non kernel existant vrf:

vrf BLOOP
  ip route 4.3.2.0/24 192.168.201.1
!

They should be able to enter it over and over and over and
over and over no matter how futile it is.

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

zebra: Auto create the nexthop-vrf if needed.

Currently if I try to use a nexthop-vrf that has
not been specified yet we get a failure from the cli.

Add code to zebra so that if we fail to find the nexthop-vrf
we auto create it, instead of failing the install.

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

bgpd: Move back nh_vrf setting to where it was

Place the nexthop vrf id setting outside the afi if split
so that each side does not need to keep track of this.

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

Merge pull request #1902 from donaldsharp/vtysh_doc

doc: Clean up vtysh man page

zebra: Cleanup function prototypes for rib.h

Add to the function prototypes the names of variables
to hopefully make it easier for people to program against
this header.

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

sharpd: Add ability to register for nht

Add code to sharpd to allow it to register for nexthop tracking
and then to display when it receives information about the nexthop
it is watching.

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

lib, zebra: Signal the existence of labels on a nexthop for nht

When we are signaling to a client from zebra that a nexthop
has changed, include the labels on the nexthop as well.
Upper level protocols need to know if the labels exist
in order to make intelligent decisions about what to do.

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

zebra: Apply label to all nexthops when needed

The application of a label to a route entry needs to
look at all non-recursive nexthops to be attached to
instead of just the first one.

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

zebra: Cleanup dead function rib_weed_table

the rib_wib_table function was uncalled by anyone remove
and additionally remove it's static function it called.

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

zebra: Read in on startup arbitrary tables

When we receive an arbitrary table over the netlink bus
save it for later perusal and sweep any routes that
we may have created from an earlier run.

The current redistribute code is limited to
ZEBRA_KERNEL_TABLE_MAX.  I left this alone for the
moment because I believe it needs to be converted
to a RB tree instead of a flat array.  Which is more
work for the future.  Additionally this proposed
change might necessitate some cli changes or rethinks.

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

zebra: Upon client disconnect remove routes from all tables

It is possible for clients to install routes into tables
that they desire.  Modify the code to delete these routes
from these tables as well.

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

zebra: Allow for deletion of rules when the originator goes away

When zebra detects that the originator has dissapeared
delete all rules associated with that client.

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

bgpd: Clean up peer status checking for a received nlri

In bgp_update_receive the first thing we do is establish
that the peer->status is Established.  We then do a bunch
of work and call bgp_nlri_parse where we break out for
each address family.  Each AFI is then checking for
being peer->status is Established again.  There is no
point in checking this again.

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

Merge pull request #1885 from msablic/pim_mtrace_client

pimd: improvments to mtrace

doc: Clean up vtysh man page

The vtysh man page was referencing eigrp.  Looked like a
cut-n-paste error?

Added some missing cli for the vtysh program, that users
might want to see.

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

Merge pull request #1739 from LabNConsulting/working/master/bgp-vpn-vrf-leaking

bgpd: vpn - vrf route leaking

Merge pull request #1897 from donaldsharp/vrf_knowledge

zebra: Add some additional vrf info to debugs

zserv.c: delete unused zsend_write_nexthop

zserv.c: zebra_server_send_message() second parameter

tools: Add helper frr-reload shell script for clean error

Adding the shell script allows a clean error if frr-pythontools
is not installed.

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

zebra: Add some additional vrf info to debugs

There were several places where when I am attempting
to debug zebra functionality that I would really
like to have the ability to know what vrf I think
I am operating on.

Add the vrf_id to a bunch of zlog_debug messages
to help figure out issues when they happen.

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

Merge branch 'master' into working/master/bgp-vpn-vrf-leaking

Merge pull request #1821 from vivek-cumulus/pmsi-parse-display

bgpd: Parse PMSI Tunnel Attribute and display

Merge pull request #1893 from opensourcerouting/doc-isis-threeway

doc: document IS-IS three-way handshake config

doc: document IS-IS three-way handshake config

Merge pull request #1863 from donaldsharp/more_nh_groups

More nh groups

Merge pull request #1886 from donaldsharp/strlcpy

lib: Fixup strlcat and strlcpy to be a bit more descriptive

Merge pull request #1869 from opensourcerouting/frr-reload-vnc-fix

tools/frr-reload.py: process vnc settings as sub-context

tools: checkpatch.sh - Add warning that // style comments don't match
       recommended (kernel) comment style

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

Merge pull request #1819 from donaldsharp/static_stuff

Static stuff

tools: checkpatch.sh - use run specific /tmp directories, remove them when done

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

Merge pull request #1884 from donaldsharp/spammy_messages

ospf6d: Stop debug messages happening in rare case

lib: Fixup strlcat and strlcpy to be a bit more descriptive

When I use these functions and am programming on linux I
always have to pull up a man page for these two functions
since they exist in *BSD land only.

Modify the name of the size variable to destsize on
pass in to give me the small hint I need to know
what to do.

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

lib: Add the ability for other people to call a nexthop write line

Add code to allow nexthops to be written by people who are
interested in writing their own nexthop line.

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

lib: Expose nhgc_find command

Expose to the world the nhgc_find command so that
interested parties can find a stored nexthop group.

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

lib: Add nexthop-group cli

Add a nexthop-group cli:

nexthop-group NAME
  nexthop A
  nexthop B
  nexthop C
!

This will allow interested parties to hook into the cli for
nexthops.  Users can add callback functions for add/delete
of a nexthop group as well as add/delete of each individual
nexthop.

Future work( PBR and static routes ) will take advantage
of this.

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

Merge pull request #1830 from opensourcerouting/isis-three-way-adj

IS-IS: implement three-way handshake as per RFC5303

Merge pull request #1880 from pguibert6WIND/enforce_vrf_netns_enable

lib: enforce vrf netns if setns() returns ok

pimd: improvments to mtrace

mtracebis has been updated, so it behaves more like the original
mtrace command.

Signed-off-by: Mladen Sablic <mladen.sablic@gmail.com>

Merge pull request #1867 from LabNConsulting/working/master/indent2

more indent.py related changes

ospf6d: Stop debug messages happening in rare case

When we have a interface disabled in ospfv3 and
we are receiving messages on it.  We will spam
the log file repeatedly.  Debug guard the message.

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

Merge pull request #1704 from qlyoung/docuser

Documentation update to ReST

Merge pull request #1831 from qlyoung/frr-pthread-fixups

lib: some frr_pthread fixes

doc: remove misc trailing whitespace

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

doc: Update CentOS6 doc to use puias repo for updated sphinx

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

bgpd: Fix warning

Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>

bgpd: Fix enum, use API for log string

Suggested-by: Philippe Guibert <philippe.guibert@6wind.com>
Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>

Merge branch 'master' of https://github.com/frrouting/frr into pmsi-parse-display

doc: don't check for sphinx-build on `make clean`

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