]> git.puffer.fish Git - mirror/frr.git/commit
isisd: fix update link params after circuit is up 16731/head
authorLouis Scalbert <louis.scalbert@6wind.com>
Tue, 27 Aug 2024 16:22:27 +0000 (18:22 +0200)
committerMergify <37929162+mergify[bot]@users.noreply.github.com>
Tue, 3 Sep 2024 14:57:53 +0000 (14:57 +0000)
commit7147f956722cb9a3505c699e96f877332dd41886
tree9bbb879b4120e7a65adc9b2fb74a3c15020fb0d6
parentb2be78e58f7269d6bdfdda349488e50e4bef4236
isisd: fix update link params after circuit is up

If the link-params are set when the circuit not yet up, the link-params
are never updated.

isis_link_params_update() is called from isis_circuit_up() but returns
immediately because circuit->state != C_STATE_UP. circuit->state is
updated in isis_csm_state_change after isis_circuit_up().

> struct isis_circuit *isis_csm_state_change(enum isis_circuit_event event,
>     struct isis_circuit *circuit,
>     void *arg)
> {
> [...]
>  if (isis_circuit_up(circuit) != ISIS_OK) {
>  isis_circuit_deconfigure(circuit, area);
>  break;
>  }
>  circuit->state = C_STATE_UP;
>  isis_event_circuit_state_change(circuit, circuit->area,
>  1);

Do not return isis_link_params_update() if circuit->state != C_STATE_UP.

Fixes: 0fdd8b2b11 ("isisd: update link params after circuit is up")
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
(cherry picked from commit 6ce6b7a8564f661495fec17f3ea33eeaf9e2f48c)
isisd/isis_te.c