topotests: python3, replace iteritems with items

replace iteritems() calls with items()

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

topotests: python3, rename iterkey() with key() for dictionary usage

dictionary method iterkey() is not used in python3 and is replaced with
key() method.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

topotests: python3, fix error ValueError: can't have unbuffered text I/O

This error occurs when passing some 0 values to open() extra argument.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

topotests: python3, replace execfile with exec

python3 does not support execfile implementation.
replace it with open and exec api that are available in both python 2
and 3 implementations.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Merge pull request #7569 from patrasar/pim_coverity_fix

pimd: Fix SA issues found in Coverity Scan

pimd: Fix SA issues found in Coverity Scan

Signed-off-by: Sarita Patra <saritap@vmware.com>

Merge pull request #7562 from donaldsharp/python_fixes

tests: Fix common_config.py issues

Merge pull request #7559 from ckishimo/translator

ospfd: fix NSSA translator

Merge pull request #7546 from mjstapp/limit_fds

lib: add startup-time option to limit fds used

Merge pull request #7568 from idryzhov/fix-naming

lib: fix local variable shadowing global one

Merge pull request #7544 from andylemin/master

bgpd: added bmp read check to detect broken sessions

Merge pull request #7564 from pguibert6WIND/fix_typo_topotest

doc: topotest json typo error

Revert "debian: Merge various debian changelogs in debian/changelog"

This reverts commit da3a39ff6a131039ef67e01637333353f3095adb.

Revert "debian: Adjust tarsource.sh to use native debian/changelog"

This reverts commit 4ffb9a4c9ddd0ef3f770c0bc2674c4818331780d.

Revert "debian: Update tools/build-debian-package.sh to use git-buildpackage"

This reverts commit 63c0c8ed89e1564294ff174b819e520d5977f499.

Revert "debian: Remove the changelog-auto automation in favor of dch"

This reverts commit cace1d9bf178d447599fe7caa5ccad9a89babe03.

Revert "debian: Remove now obsolete tarsource.sh script"

This reverts commit bc304e08ea48523b4c70d1d330eaefba3b6b7a6c.

Revert "debian: Update the tools/build-debian-package.sh to be example build script"

This reverts commit c950a7f214cd713652da1b6b78eca9278c33b448.

lib: fix local variable shadowing global one

start_config and end_config are already used as function names in DEFUN,
so the current naming is a little bit confusing. Let's use different
names for arguments.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

debian: Update the tools/build-debian-package.sh to be example build script

Signed-off-by: Ondřej Surý <ondrej@sury.org>

debian: Remove now obsolete tarsource.sh script

debian: Remove the changelog-auto automation in favor of dch

Signed-off-by: Ondřej Surý <ondrej@sury.org>

debian: Update tools/build-debian-package.sh to use git-buildpackage

Signed-off-by: Ondřej Surý <ondrej@sury.org>

debian: Adjust tarsource.sh to use native debian/changelog

Signed-off-by: Ondřej Surý <ondrej@sury.org>

debian: Merge various debian changelogs in debian/changelog

Signed-off-by: Ondřej Surý <ondrej@sury.org>

Merge pull request #7566 from ton31337/fix/topotests_bgp_ebgp_requires_policy

tests: Add more descriptive error messages and increase timeouts

Merge pull request #7424 from donaldsharp/config_commands

lib, sharpd, vtysh: When reading a file in for config send start/end …

Merge pull request #6145 from patrasar/pim_nb_code_upstream

pimd: northbound backend code

Merge pull request #7563 from donaldsharp/why_not_cut_n_paste

ospf6d: More lists being leaked

Merge pull request #7561 from donaldsharp/resources_are_the_life

lib: Stop socket from leaking.

lib: Add information log of how long configuration took to read in

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

lib, sharpd, vtysh: When reading a file in for config send start/end indicators

When reading a file in for configuration, send start and end indicators
to interested parties.

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

tests: Add more descriptive error messages and increase timeouts

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

doc: topotest json typo error

fix the typo error.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

ospf6d: More lists being leaked

Apparently the person who wrote this code was big into
cut-n-paste.  Commit 710a61d57c8f1b0ea66a37f09bad2161d7e2ddb7
found the first instance, but upon code inspection this morning
it became evident that 2 other functions had the exact same
problem.

Fix.  Note I have not cleaned up the cut-n-paste code for
two reasons: a) I'm chasing something else b) this code
has been fairly un-maintained for a very long time.  No
need to start up now.

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

tests: Fix common_config.py issues

1) in generate_ips if start_ipaddr does not have a `/` in it
there exists a code path where both mask and step are
null values.  Write a bit of code to ensure this pre-req
is found early and often

2) in verify_rib there exists a code path where static_route
is null when we get to the non static route section.  Change
the code to operate on the advertise_network_dict that
we are iterating over.

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

lib: Stop socket from leaking.

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

Merge pull request #7554 from donaldsharp/sockunion2hostprefix_watch_returns

bgpd, lib, nhrpd, zebra: verify return of sockunion2hostprefix

Merge pull request #7528 from donaldsharp/changelog_update

Changelog update

ospfd: fix NSSA translator

Having 2 ABR in NSSA area where R3 is the elected translator

R3# show ip ospf
  We are an ABR and the NSSA Elected Translator.
R2# show ip ospf
  We are an ABR, but not the NSSA Elected Translator.

When R3 loses the Border condition by shutting down the interface
to the backbone, we end up with no translator in the NSSA area. It
is expected R2 to take over the translator role

R3# sh ip ospf
  It is not ABR, therefore not Translator.
R2# show ip ospf
   We are an ABR, but not the NSSA Elected Translator.

This PR forces the ABR to reevaluate the translator condition, so
R2 becomes the elected Translator

Signed-off-by: ckishimo <carles.kishimoto@gmail.com>

pimd: fix indentation issues

Signed-off-by: Sarita Patra <saritap@vmware.com>

Merge pull request #7210 from idryzhov/route-types-disable-daemon

lib: respect --disable-daemon flags in route_types.h

Merge pull request #7454 from ranjanyash54/dev_8

ospf6d: Json support added for command "show ipv6 ospf6 interface traffic [json]"

Merge pull request #7455 from ranjanyash54/dev_3

ospf6d: Json support added for command "show ipv6 ospf6 redistribute [json]"

Merge pull request #7467 from ton31337/fix/bgpd_null_dereference

bgpd: Check if peer is not NULL before calling peer_as_change()

Merge pull request #7558 from opensourcerouting/bfd-v6-same

bfdd: fix multiple links same address

Merge pull request #7296 from YashRanjanVMWare/dev_1

ospf6d: Json support added for command "show ipv6 ospf6 [json]"

Merge pull request #7556 from donaldsharp/memory_shenanigans

Memory shenanigans

lib: add startup option to limit fds

Add a startup-time option to limit the number of fds used
by the thread/event infrastructure. If nothing is configured,
the system ulimit is used.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #7552 from volta-networks/fix_reload_l2pvn

tools: fix frr-reload l2vpn delete

Merge pull request #7555 from idryzhov/cppcheck-fixes

fix a couple of issues found by cppcheck

bfdd: fix multiple links same address

Allows users with multiple links using same IPv6 address (same VRF) to
work.

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

bgpd: Prevent ecommunity_ecom2str memory leak

We were allocating but never freeing memory associated with the
ecommunity_ecom2str allocation.

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

bgpd: Do not leak vname if we find the match

If we are attempting to store the bgp name for route
leaking and we find a match do not leak the memory.

Please note this is probably not really going to happen
ever.

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

Merge pull request #7553 from donaldsharp/mem_leak_ferr

lib: Cleanup mem-leaks in error case

ospf6d: Free up list memory

OSPFv3 snmp code creates a list but never deletes the actual
list.  Do so.

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

pimd: A buffer may be used uninited

If we screw up and don't have the right flags we'll print
out garbage.  At the very least just print out nothing.

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

lib: fix macro style

We prefer semicolon in the actual code instead of macro elsewhere in the
code.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

isisd: fix uninitialized variable

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

ospf6d: fix compilation of debug code

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

zebra: fix writing to pointer instead of value

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

bgpd, lib, nhrpd, zebra: verify return of sockunion2hostprefix

The return from sockunion2hostprefix tells us if the conversion
succeeded or not.  There are places in the code where we
always assume that it just `works`, since it can fail
notice and try to do the right thing.

Please note that failure of this function for most cases
of sockunion2hostprefix is highly highly unlikely as that
the sockunion was already created and tested elsewhere
it's just that this function can fail.

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

lib: Cleanup mem-leaks in error case

When we fail to properly lookup an error code, properly cleanup
on return.

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

Merge pull request #7478 from donaldsharp/buffer

Buffer

tools: fix frr-reload l2vpn delete

when deleting a whole l2vpn context in ldpd which also had pseudowires
in it, we were first deleting the l2vpn with a 'no l2vpn XXX' command,
and then adding it again by running 'l2vpn XXX\n no member pseudowire YYY'
which obviously was not needed. As a result the l2vpn would be reinstated.

Signed-off-by: Emanuele Di Pascale <emanuele@voltanet.io>

Merge pull request #7549 from vincentbernat/fix/oot-build

vtysh: fix out-of-tree build not finding vtysh/daemons.pl

Merge pull request #7548 from vincentbernat/feature/json-passive

bgpd: identify passive connections in JSON summary

Merge pull request #7543 from GalaxyGorilla/fix_refcmp_newline

tests: ignore Windows vs Unix style newlines

Merge pull request #7534 from chiragshah6/yang_nb1

bgpd: convert to yang modeled nbr cmds

vtysh: fix out-of-tree build not finding vtysh/daemons.pl

This is needed to build with:

    mkdir build && cd build && ../configure && make

Signed-off-by: Vincent Bernat <vincent@bernat.ch>

bgpd: identify passive connections in JSON summary

When using non JSON output, passive connections are marked with:

    BGP state = Active (passive)

However, such an information is not available in JSON output. This
commit adds that. It also adds "Active (NSF passive)" flag, like the
regular output.

Signed-off-by: Vincent Bernat <vincent@bernat.ch>

Merge pull request #6851 from kuldeepkash/bgp_graceful_restart

tests: Fix for bgp_gr_functionality_topo1/topo2 failures

Merge pull request #7303 from mobash-rasool/pim-fixes

pimd: checksum must be validated before accepting igmpv2 leave packet

Merge pull request #7511 from donaldsharp/vtysh_warn

Vtysh excessive/too little doc string re-instatement

bgpd: convert to yang modeled nbr cmds

Convert to (DEFUN_YANG) yang modeled neighbor configuration
commands to count them in batched execution as part of backoff timer
introduced as part of PR # 6727

These configuration commands already converted transactional cli
but missed to mark them as (DEFUN_YANG).

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

Merge pull request #7541 from ton31337/fix/hardcoded_function_names

bgpd: Use __func__ instead of hardcoded strings for some functions

pimd: checksum must be validated before accepting igmp packets

Issue: When an IGMPv2 leave packet is received, it did not validate
the checksum and hence the packet is accepted and group specific
query is sent out in response to this.

Due to this IGMP conformance test case 6.1 failed.

https://github.com/FRRouting/frr/issues/6868

Fix: Validate the checksum for all IGMP packets

Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

bgpd: added bmp read check to detect broken sessions

Signed-off-by: Andy Lemin <andy@lemin.io>

tests: ignore Windows vs Unix style newlines

Within unit tests the output of vtysh commands is compared to hand
made reference files. For some reason the output of those vtysh
commands contains Windows Style newlines which results in error
outputs which make it hard to identify this problem.

Since there seems to be no benefit in checking those newlines
anyway this commit just normalizes them.

Signed-off-by: GalaxyGorilla <sascha@netdef.org>

Merge pull request #7540 from ton31337/fix/typo

*: Replace typo (supress => suppress)

Merge pull request #7450 from ranjanyash54/dev_5

ospf6d: Json support added for command "show ipv6 ospf6 zebra [json]"

Merge pull request #7453 from ranjanyash54/dev_7

ospf6d: Json support added for command "show ipv6 ospf6 interface [json]"

doc: Update release procedure

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

build: Update changelog from 7.5 release data

Update the changelog from 7.5 into masters changelog

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

Merge pull request #7476 from opensourcerouting/bfd-fixes

bfdd,lib: integration fixes

Merge pull request #7518 from donaldsharp/asic_offload_more

Asic offload more

lib: Change zlog_debug to zlog_err for doc string issues

Modify the code to change from zlog_debug to zlog_err.
vtysh was not outputting the vtysh doc string issues
after a change a couple of months back.  By changing
to error level we start seeing them on vtysh start up
again.  This will allow us to catch these issues
in the CI runs again.

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

ospfd: Fix excessive doc string for `no summary-address ..` command

The `no summary-address ..` command had excessive doc string.
Fix.

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

Merge pull request #7520 from volta-networks/fix_ospf_mpls_te

ospfd: reset mpls-te prior to ospf router removal

Merge pull request #7523 from donaldsharp/route_map_object_t

*: Remove route_map_object_t from the system

Merge pull request #7536 from ton31337/fix/documentation

doc: Replace _ to `` when bolding keyword for recently added BGP commands

Merge pull request #7530 from ton31337/fix/safeguard_path->attr

bgpd: Make sure path->attr is not NULL before dereferencing

bgpd: Use __func__ instead of hardcoded strings for some functions

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

*: Replace typo (supress => suppress)

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

doc: Replace _ to `` when bolding keyword for recently added BGP commands

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

Merge pull request #7521 from donaldsharp/set_src_future_us

zebra: Allow `set src X` to work on startup

bgpd: Make sure path->attr is not NULL before dereferencing

Null-checking "path->attr" suggests that it may be null, but it has
already been dereferenced on all paths leading to the check.

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

ospf6d: Json support added for command "show ipv6 ospf6 interface [json]"

Modify code to add JSON format output in show command
"show ipv6 ospf6 interface" with proper formating

Signed-off-by: Yash Ranjan <ranjany@vmware.com>

ospf6d: Json support added for command "show ipv6 ospf6 zebra [json]"

Modify code to add JSON format output in show command
"show ipv6 ospf6 zebra" with proper formating

Signed-off-by: Yash Ranjan <ranjany@vmware.com>

ospf6d: Json support added for command "show ipv6 ospf6 [json]"

Modify code to add JSON format output in show command
"show ipv6 ospf6" with proper formating.

Signed-off-by: Yash Ranjan <ranjany@vmware.com>