bus: Fix maximum log levels when mixing log/vlog implementing loggers

The maximum would not get set correctly when a logger is removed and the
first remaining logger in the list (the one with the highest log level) does
e.g. only implement vlog() while there are other loggers that implement log().
This would result in only max_vlevel getting set correctly while max_level
would incorrectly get set to -1 so that log() would not get called for any
of the loggers anymore.

References #574.

diff --git a/src/libcharon/bus/bus.c b/src/libcharon/bus/bus.c
index e17d629d225b51cc4f022221e9fca31f34fd8e65..ba44a22f282313c35bde941fde0d6d31e3214685 100644 (file)
--- a/src/libcharon/bus/bus.c
+++ b/src/libcharon/bus/bus.c
@@ -207,6 +207,24 @@ static inline void register_logger(private_bus_t *this, debug_t group,
        }
 }
 
+/**
+ * Find the log level of the first registered logger that implements log or
+ * vlog (or both).
+ */
+static bool find_max_levels(log_entry_t *entry, debug_t *group, level_t *level,
+                                                       level_t *vlevel)
+{
+       if (entry->logger->log && *level == LEVEL_SILENT)
+       {
+               *level = entry->levels[*group];
+       }
+       if (entry->logger->vlog && *vlevel == LEVEL_SILENT)
+       {
+               *vlevel = entry->levels[*group];
+       }
+       return *level > LEVEL_SILENT && *vlevel > LEVEL_SILENT;
+}
+
 /**
  * Unregister a logger from all log groups (destroys the log_entry_t)
  */
@@ -240,18 +258,8 @@ static inline void unregister_logger(private_bus_t *this, logger_t *logger)
                        {
                                loggers = this->loggers[group];
                                loggers->remove(loggers, found, NULL);
-
-                               if (loggers->get_first(loggers, (void**)&entry) == SUCCESS)
-                               {
-                                       if (entry->logger->log)
-                                       {
-                                               level = entry->levels[group];
-                                       }
-                                       if (entry->logger->vlog)
-                                       {
-                                               vlevel = entry->levels[group];
-                                       }
-                               }
+                               loggers->find_first(loggers, (linked_list_match_t)find_max_levels, NULL,
+                                                                       &group, &level, &vlevel);
                                set_level(&this->max_level[group], level);
                                set_level(&this->max_vlevel[group], vlevel);
                        }