[strongSwan-dev] [PATCH] conftest: Fix log level settings for stdout

[Thomas Klute]

This patch fixes bug #272 ("conftest ignores log settings for stdout").
http://wiki.strongswan.org/issues/272

According to the documentation of add_logger in src/libcharon/bus/bus.h,
the relevant log levels of a logger are registered with the logging
subsystem when adding the logger. If the log levels change later, the
logger must be re-added to propagate the new settings. In conftest.c,
the stdout logger is initialized and added before reading the logging
settings, but wasn't re-added after reading the settings.
---
 src/conftest/conftest.c |    3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/conftest/conftest.c b/src/conftest/conftest.c
index bb4c5ed..e8c257d 100644
--- a/src/conftest/conftest.c
+++ b/src/conftest/conftest.c
@@ -386,6 +386,9 @@ static void load_loggers(file_logger_t *logger)
        char *section;
 
        load_log_levels(logger, "stdout");
+       /* Re-add the logger to propagate configuration changes to the
+        * logging system */
+       charon->bus->add_logger(charon->bus, &logger->logger);
 
        enumerator = conftest->test->create_section_enumerator(conftest->test, 
"log");
        while (enumerator->enumerate(enumerator, &section))
-- 
1.7.10.4


_______________________________________________
Dev mailing list
Dev@lists.strongswan.org
https://lists.strongswan.org/mailman/listinfo/dev