diff options
author | Renato Westphal <renato@opensourcerouting.org> | 2020-10-20 04:56:54 +0200 |
---|---|---|
committer | Renato Westphal <renato@opensourcerouting.org> | 2020-10-23 23:18:22 +0200 |
commit | 9bde0b256919ff3987ea60101229fec195324102 (patch) | |
tree | 03d5a635efaf775dca58f389073b58c5fa2833d9 /eigrpd/eigrp_siareply.c | |
parent | Merge pull request #7361 from wesleycoakley/alpine-build-fix (diff) | |
download | frr-9bde0b256919ff3987ea60101229fec195324102.tar.xz frr-9bde0b256919ff3987ea60101229fec195324102.zip |
lib: fix iteration over schema nodes of a single YANG module
The only safe way to iterate over all schema nodes of a given YANG
module is by iterating over all schema nodes of all YANG modules
and filter out the nodes that belong to other modules.
The original yang_snodes_iterate_module() code did the following:
1 - Iterate over all top-level schema nodes of the given module;
2 - Iterate over all augmentations of the given module.
While that iteration strategy is more efficient, it does't handle
well more complex YANG hierarchies containing nested augmentations
or self-augmenting modules. Any iteration that isn't done on the
resolved YANG data hierarchy is fragile and prone to errors.
Fixes regression introduced by commit 8a923b48513316b where the
gen_northbound_callbacks tool was generating duplicate callbacks
for certain modules.
Signed-off-by: Renato Westphal <renato@opensourcerouting.org>
Diffstat (limited to 'eigrpd/eigrp_siareply.c')
0 files changed, 0 insertions, 0 deletions