From fe80f8f03adb5b4c30becd12b9c793ce527121f8 Mon Sep 17 00:00:00 2001 From: Claudio Jeker Date: Thu, 22 Feb 2018 07:42:39 +0000 Subject: The IF_EVT_NBR_CHNG event needs to be fired when a neighbor transitions to a state of 2-Way or higher. There is no need to trigger the event for new neighbors. With this situations with multiple DRs after a netsplit should be solved. OK sthen@ and remi@ --- usr.sbin/ospfd/hello.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'usr.sbin') diff --git a/usr.sbin/ospfd/hello.c b/usr.sbin/ospfd/hello.c index afb64a3f267..59dc14b746b 100644 --- a/usr.sbin/ospfd/hello.c +++ b/usr.sbin/ospfd/hello.c @@ -1,4 +1,4 @@ -/* $OpenBSD: hello.c,v 1.21 2014/11/18 20:54:29 krw Exp $ */ +/* $OpenBSD: hello.c,v 1.22 2018/02/22 07:42:38 claudio Exp $ */ /* * Copyright (c) 2005 Claudio Jeker @@ -188,7 +188,6 @@ recv_hello(struct iface *iface, struct in_addr src, u_int32_t rtr_id, char *buf, nbr->dr.s_addr = hello.d_rtr; nbr->bdr.s_addr = hello.bd_rtr; nbr->priority = hello.rtr_priority; - nbr_change = 1; } /* actually the neighbor address shouldn't be stored on virtual links */ @@ -201,8 +200,10 @@ recv_hello(struct iface *iface, struct in_addr src, u_int32_t rtr_id, char *buf, memcpy(&nbr_id, buf, sizeof(nbr_id)); if (nbr_id == ospfe_router_id()) { /* seen myself */ - if (nbr->state & NBR_STA_PRELIM) + if (nbr->state & NBR_STA_PRELIM) { nbr_fsm(nbr, NBR_EVT_2_WAY_RCVD); + nbr_change = 1; + } break; } buf += sizeof(nbr_id); -- cgit v1.2.3