diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-06-14 19:59:02 -0400 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-07-24 13:51:40 -0400 |
| commit | 449a3f5b10eca91786482d2a5d063cd4a69de84d (patch) | |
| tree | 46196d3f7eeaaa2b2eb433495e39600c6789b99d /pimd/pim_msdp_socket.c | |
| parent | 05d8470ff58c62bc67658aa66435ac69f404b51b (diff) | |
pimd: Wisely delay startup of msdp listen/connect when vrf is not fully up
When we do not really have the vrf up, wisely do not attempt
to dereference the ifp looked up.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'pimd/pim_msdp_socket.c')
| -rw-r--r-- | pimd/pim_msdp_socket.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/pimd/pim_msdp_socket.c b/pimd/pim_msdp_socket.c index b8c103fe27..9473462e63 100644 --- a/pimd/pim_msdp_socket.c +++ b/pimd/pim_msdp_socket.c @@ -153,6 +153,11 @@ int pim_msdp_sock_listen(struct pim_instance *pim) if (pim->vrf_id != VRF_DEFAULT) { struct interface *ifp = if_lookup_by_name(pim->vrf->name, pim->vrf_id); + if (!ifp) { + zlog_err("%s: Unable to lookup vrf interface: %s", + __PRETTY_FUNCTION__, pim->vrf->name); + return -1; + } pim_socket_bind(sock, ifp); } @@ -226,6 +231,11 @@ int pim_msdp_sock_connect(struct pim_msdp_peer *mp) if (mp->pim->vrf_id != VRF_DEFAULT) { struct interface *ifp = if_lookup_by_name(mp->pim->vrf->name, mp->pim->vrf_id); + if (!ifp) { + zlog_err("%s: Unable to lookup vrf interface: %s", + __PRETTY_FUNCTION__, mp->pim->vrf->name); + return -1; + } pim_socket_bind(mp->fd, ifp); } |