bfd: fix bfd_key_id update 16/34116/2
authorAlexander Skorichenko <askorichenko@netgate.com>
Fri, 15 Oct 2021 16:04:44 +0000 (16:04 +0000)
committerMatthew Smith <mgsmith@netgate.com>
Tue, 19 Oct 2021 14:50:35 +0000 (14:50 +0000)
commit4c3c60daf7bccffc0f1fe59d8d09557bc62d00b0
tree2ff3dd0503e1b2923e28c452527957c5489f1243
parent0dfad21e88a5af3d429af5b429f7a7b5a9af7c04
bfd: fix bfd_key_id update

Type: fix

Currently, auth activation CLI command
"bfd upd session auth activate ... conf-key-id <cki> bfd-key-id <bki>"
allows to change both key-ids to new values at once.
But if only bfd-key-id should be corrected, e.g. as a result of mistyping,
we can't do that in a single operation, and have to deactivate auth first
and then reactivate it with a correctly entered pair of ids.

Currently, backend's bfd_auth_activate() function returns immediately,
with no action, if it finds that submitted conf-key-id matches the
current record. No check on bfd-key-id value is made.

With this fix, bfd_auth_activate() checks if session's bfd-key-id has to
be changed to a new value, and if so, it updates and logs appropriately.

Change-Id: I3b915a936cb1721707860bb503f70e7dd29e0ddd
Signed-off-by: Alexander Skorichenko <askorichenko@netgate.com>
src/vnet/bfd/bfd_main.c