diff options
| author | Renato Westphal <renato@opensourcerouting.org> | 2019-11-07 13:20:06 -0300 |
|---|---|---|
| committer | Renato Westphal <renato@opensourcerouting.org> | 2019-11-29 21:30:01 -0300 |
| commit | 083be18cb8616deeb594fcc6b94a7232d377a0fb (patch) | |
| tree | 4fb017638422dbd36bcb19cd5dba3b891501195e /doc/manpages/conf.py | |
| parent | 6fc3fed23c95da21887acb57a92c474e4d0ca3a4 (diff) | |
lib: fix display of candidate configurations
Commit 5e6a9350c16 implemented an optimization where candidate
configurations are validated only before being displayed. The
validation is done only to create default child nodes (due to
how libyang works) and any possible error is ignored (candidate
configurations can be invalid/incomplete).
The problem is that we were calling lyd_validate() only when the
CLI "with-defaults" option was used. But some cli_show() callbacks
assume that default nodes exist and can crash when displaying a
candidate configuration that isn't validated. To fix this, call
lyd_validate() before displaying candidate configuration even when
"with-defaults" is not used (that was a micro-optimization that
shouldn't have been done).
Signed-off-by: Renato Westphal <renato@opensourcerouting.org>
Diffstat (limited to 'doc/manpages/conf.py')
0 files changed, 0 insertions, 0 deletions
