summaryrefslogtreecommitdiffstats
path: root/isisd
diff options
context:
space:
mode:
authorIgor Ryzhov <iryzhov@nfware.com>2021-03-14 22:43:17 +0100
committerIgor Ryzhov <iryzhov@nfware.com>2021-03-16 15:25:49 +0100
commit4ba756ed9cf95ba278e940e6c959f1987112e87b (patch)
treef75a06c0b8f2eb58ade26dd515ac1b96f57cc7dc /isisd
parentMerge pull request #8260 from volta-networks/fix_isis_snmp_test (diff)
downloadfrr-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.c2
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);