From 3cd6ddaec6fd41ac84ef23595ffc7e3d18334839 Mon Sep 17 00:00:00 2001 From: Igor Ryzhov Date: Mon, 26 Feb 2024 11:30:57 +0200 Subject: [PATCH] lib: fix setting temporary log options for libyang By calling `ly_log_options` with `LY_LOSTORE`, the current code effectively disables libyang logging and never enables it back. The call is done to get the current logging options, but we don't really need that. When looking for a schema node, we don't want neither to log nor to store the error, so simply set the temporary options to 0. Signed-off-by: Igor Ryzhov --- lib/northbound.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/lib/northbound.c b/lib/northbound.c index f9f9449ea66d..25ea658bc43c 100644 --- a/lib/northbound.c +++ b/lib/northbound.c @@ -157,7 +157,7 @@ void nb_nodes_delete(void) struct nb_node *nb_node_find(const char *path) { const struct lysc_node *snode; - uint32_t llopts; + uint32_t llopts = 0; /* * Use libyang to find the schema node associated to the path and get @@ -165,8 +165,6 @@ struct nb_node *nb_node_find(const char *path) * disable logging temporarily to avoid libyang from logging an error * message when the node is not found. */ - llopts = ly_log_options(LY_LOSTORE); - llopts &= ~LY_LOLOG; ly_temp_log_options(&llopts); snode = yang_find_snode(ly_native_ctx, path, 0);