ospf6: get instance from lsdb data

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

ospf6: get instance from route table scope

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

ospf6: don't create instance at startup

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

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.

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>

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>

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

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

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>

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>

Merge pull request #7533 from donaldsharp/zlog_missing_on_failure

lib: When aborting log data

Merge pull request #7531 from qlyoung/fix-doc-block-quote

doc: fix missing space before list

lib: When aborting log data

When a FRR process dies due to SIGILL/SIGABORT/etc attempt
to drain the log buffer.  This code change is capturing
some missing logs that were not part of the log file on
a crash.

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

doc: fix missing space before list

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

Merge pull request #7526 from donaldsharp/i_am_a_robot_wumpalump

Cleanup on Aisle 5: Memory leaks, Uninited data and code snarfles

ospfd: reset mpls-te prior to ospf router removal

This commits attempts to fix a problem that occurs when mpls-te gets
removed from ospfd config. Mpls-te has an inter-as option, which can be
set to Off/Area/AS. Whenever the inter-as takes "Area" or "AS" as a
value, this value will not be cleaned after removing mpls-te or after
removing the ospf router. Therefore, if mpls-te is configured with
inter-as AS or Area and we remove mpls-te or the ospf router, the
inter-as will still preserve its value; therefore, next time mpls-te is
enabled, it will automatically inherits the previous inter-as value
(either Area or AS). This leads to wrong configuration, which can be a
problem for frr_reload.py.
The commits forces mpls-te to reset inter-as to Off before it mpls-te
gets removed from the configuration and before the ospf router gets
removed.

Signed-off-by: Carlo Galiotto <carlo@voltanet.io>

Merge pull request #7461 from donaldsharp/attribute_setget

Attribute setget

Merge pull request #7510 from donaldsharp/xpath_stay

lib: Fix crash walking up command chain in bgp commands

Merge pull request #7514 from donaldsharp/respect_e

bgpd: Respect `-e X` value for multipath

Merge pull request #7525 from donaldsharp/dest_lock

bgpd: Fix missed unlocks

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

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

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

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

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

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

*: Convert all usage of zclient_send_message to new enum

The `enum zclient_send_status` enum needs to be extended
throughout the code base to use the new states and
to fix up places where we tested against the return
value being non zero.

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

bgpd, lib, sharpd: Add enum for zclient_send_message return

Add a `enum zclient_send_status` for appropriate handling
of return codes from zclient_send_message.  Touch all the places
where we handle this.

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

sharpd: Add buffering support

Have sharpd notice that when sending routes to zebra that the
underlying system has buffered data and to pause sending
more data to zebra until such time we get a callback that
the write was successful to zebra.

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