bfdd: close bfd echo sockets, upon vrf disable

upon vrf disable, an event informs bfd daemon that the vrf contexts
should be removed. in the case a vrf backend is netns based, all sockets
opened under that netns have to be closed. otherwise it is impossible
for the system to completely close the network namespace. that implies
that some interfaces may not be deleted, and may not be given back to
default vrf.

PR=65291
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Acked-by: Julien Floret <julien.floret@6wind.com>

diff --git a/bfdd/bfd.c b/bfdd/bfd.c
index 5d143d4e5f60379fd27d8e86a9574e9c206bf772..1f1568f5114399dc439c830781591f4fccfa525e 100644 (file)
--- a/bfdd/bfd.c
+++ b/bfdd/bfd.c
@@ -1706,6 +1706,8 @@ static int bfd_vrf_disable(struct vrf *vrf)
        socket_close(&bvrf->bg_mhop);
        socket_close(&bvrf->bg_shop6);
        socket_close(&bvrf->bg_mhop6);
+       socket_close(&bvrf->bg_echo);
+       socket_close(&bvrf->bg_echov6);
 
        /* free context */
        XFREE(MTYPE_BFDD_VRF, bvrf);