diff options
author | Igor Ryzhov <iryzhov@nfware.com> | 2021-03-14 22:43:17 +0100 |
---|---|---|
committer | Igor Ryzhov <iryzhov@nfware.com> | 2021-03-16 15:25:49 +0100 |
commit | 4ba756ed9cf95ba278e940e6c959f1987112e87b (patch) | |
tree | f75a06c0b8f2eb58ade26dd515ac1b96f57cc7dc /isisd | |
parent | Merge pull request #8260 from volta-networks/fix_isis_snmp_test (diff) | |
download | frr-4ba756ed9cf95ba278e940e6c959f1987112e87b.tar.xz frr-4ba756ed9cf95ba278e940e6c959f1987112e87b.zip |
*: fix aborts when validating configuration
There are places in the code where function nb_running_get_entry is used
with abort_if_not_found set to true during the config validation stage.
This is incorrect because when used in transactional CLI, the running
entry won't be set until the apply stage, and such usage leads to crash.
Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
Diffstat (limited to 'isisd')
-rw-r--r-- | isisd/isis_nb_config.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/isisd/isis_nb_config.c b/isisd/isis_nb_config.c index f0663c691..259e10bbd 100644 --- a/isisd/isis_nb_config.c +++ b/isisd/isis_nb_config.c @@ -131,7 +131,7 @@ int isis_instance_area_address_create(struct nb_cb_create_args *args) switch (args->event) { case NB_EV_VALIDATE: - area = nb_running_get_entry(args->dnode, NULL, true); + area = nb_running_get_entry(args->dnode, NULL, false); if (area == NULL) return NB_ERR_VALIDATION; addr.addr_len = dotformat2buff(buff, net_title); |