From: Donatas Abraitis <donatas@opensourcerouting.org>
Date: Thu, 9 Feb 2023 21:14:46 +0000 (+0200)
Subject: lib: Free dnode before returning if YANG translator model is already loaded
X-Git-Tag: frr-8.4.3~15^2
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=bcc9e7a548115da1b5ab5c0ebe41f0d74373f839;p=mirror%2Ffrr.git

lib: Free dnode before returning if YANG translator model is already loaded

Seems just a missed one because at `goto error` we release dnode.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
(cherry picked from commit ecf82aa41e76ecfbaf3f5762af1e90ebc9263721)
---

diff --git a/lib/yang_translator.c b/lib/yang_translator.c
index f3a355f88c..a2f6e9d366 100644
--- a/lib/yang_translator.c
+++ b/lib/yang_translator.c
@@ -165,6 +165,7 @@ struct yang_translator *yang_translator_load(const char *path)
 		flog_warn(EC_LIB_YANG_TRANSLATOR_LOAD,
 			  "%s: module translator \"%s\" is loaded already",
 			  __func__, family);
+		yang_dnode_free(dnode);
 		return NULL;
 	}