]> git.proxmox.com Git - mirror_frr.git/commit
lib,ospfd,ospf6d: remove duplicated function
authorRafael Zalamena <rzalamena@opensourcerouting.org>
Thu, 8 Jul 2021 17:09:20 +0000 (14:09 -0300)
committerRafael Zalamena <rzalamena@opensourcerouting.org>
Fri, 9 Jul 2021 10:55:25 +0000 (07:55 -0300)
commit1fe59b44fc94337f0b508bc762a401a9dbe9c1a3
tree8a525db11e5c2b61c3b8cb29b9eb55c4ed9bd6f5
parentf45f6a22b052988a6ca3399fde66249e8b6e00cc
lib,ospfd,ospf6d: remove duplicated function

Move `is_default_prefix` variations to `lib/prefix.h` and make the code
use the library version instead of implementing it again.

NOTE
----

The function was split into per family versions to cover all types.
Using `union prefixconstptr` is not possible due to static analyzer
warnings which cause CI to fail.

The specific cases that would cause this failure were:
 - Caller used `struct prefix_ipv4` and called the generic function.
 - `is_default_prefix` with signature using `const struct prefix *` or
   `union prefixconstptr`.

The compiler would complain about reading bytes outside of the memory
bounds even though it did not take into account the `prefix->family`
part.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
lib/prefix.h
ospf6d/ospf6_zebra.c
ospfd/ospf_asbr.c
ospfd/ospf_flood.c
ospfd/ospf_lsa.c
ospfd/ospf_lsa.h
ospfd/ospf_vty.c
ospfd/ospf_zebra.c