experiments/clixon
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

* Fixed: [Recursive calling xml_apply_ancestor is no need #180](https://github.com/clicon/clixon/issues/180)

Browse source
This commit is contained in:
Olof hagsand 2021-02-23 15:33:53 +01:00
parent 2a4de52e56
commit e1e391ad0d
2 changed files with 2 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

1
CHANGELOG.md
View file

@ -119,6 +119,7 @@ Developers may need to change their code
### Corrected Bugs ### Corrected Bugs
* Fixed: [Recursive calling xml_apply_ancestor is no need #180](https://github.com/clicon/clixon/issues/180)
* Fixed: [Negation operator in 'must' statement makes backend segmentation fault](https://github.com/clicon/clixon/issues/179) * Fixed: [Negation operator in 'must' statement makes backend segmentation fault](https://github.com/clicon/clixon/issues/179)
* Fixed extension/unknown problem shown in latest openconfig where other than a single space was used between the unknown identifier and string * Fixed extension/unknown problem shown in latest openconfig where other than a single space was used between the unknown identifier and string
* Fixed: [Augment that reference a submodule as target node fails #178](https://github.com/clicon/clixon/issues/178) * Fixed: [Augment that reference a submodule as target node fails #178](https://github.com/clicon/clixon/issues/178)

4
lib/src/clixon_xml.c
View file

@ -2192,9 +2192,7 @@ xml_apply_ancestor(cxobj *xn,
int ret; int ret;
while ((xp = xml_parent(xn)) != NULL) { while ((xp = xml_parent(xn)) != NULL) {
if (fn(xp, arg) < 0) if ((ret = fn(xp, arg)) < 0)
goto done;
if ((ret = xml_apply_ancestor(xp, fn, arg)) < 0)
goto done; goto done;
if (ret > 0){ if (ret > 0){
retval = ret; retval = ret;