dexter has uploaded this change for review. (
https://gerrit.osmocom.org/c/libosmocore/+/18222 )
Change subject: logging: do not allow multiple calls of log_init()
......................................................................
logging: do not allow multiple calls of log_init()
calling log_init() multiple times would lead into memory leaks. The
function should only be called once on startup of the process. Lets make
sure that it does not get called multiple times by accident.
Change-Id: Ibb300e4c9b04767581116ab530b2e6a9a195db08
---
M src/logging.c
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/22/18222/1
diff --git a/src/logging.c b/src/logging.c
index 4aaf515..c14e696 100644
--- a/src/logging.c
+++ b/src/logging.c
@@ -1028,6 +1028,9 @@
int i;
struct log_info_cat *cat_ptr;
+ /* Ensure that log_init is not called multiple times */
+ OSMO_ASSERT(tall_log_ctx == NULL)
+
tall_log_ctx = talloc_named_const(ctx, 1, "logging");
if (!tall_log_ctx)
return -ENOMEM;
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/18222
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ibb300e4c9b04767581116ab530b2e6a9a195db08
Gerrit-Change-Number: 18222
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pma...@sysmocom.de>
Gerrit-MessageType: newchange
