<p>dexter has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/16833">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">application.c: check default loglevels on startup<br><br>Default log levels should not be set lower than LOGL_NOTICE. If a lower<br>loglevel is desired, this loglevel should be set via a configuration<br>file. Lets print a warning if a default loglevel lower than LOGL_NOTICE<br>is set to remind the programmer to set proper default loglevels.<br><br>Related: OS#2577<br>Change-Id: I2d7e345c14a188430a0e991bfd9fb0343d05ea92<br>---<br>M src/application.c<br>1 file changed, 17 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/33/16833/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/application.c b/src/application.c</span><br><span>index 7fd6280..a7ed01e 100644</span><br><span>--- a/src/application.c</span><br><span>+++ b/src/application.c</span><br><span>@@ -117,6 +117,21 @@</span><br><span>  return osmo_init_logging2(NULL, log_info);</span><br><span> }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/* Check preset loglevels. This check prints a warning if the default loglevel</span><br><span style="color: hsl(120, 100%, 40%);">+ * is lower than LOGL_NOTICE, which should never be the default */</span><br><span style="color: hsl(120, 100%, 40%);">+static void check_loglevels(const struct log_info *log_info)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+      unsigned int i;</span><br><span style="color: hsl(120, 100%, 40%);">+       for (i = 0; i < log_info->num_cat; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+               if (log_info->cat[i].loglevel < LOGL_NOTICE) {</span><br><span style="color: hsl(120, 100%, 40%);">+                  printf</span><br><span style="color: hsl(120, 100%, 40%);">+                            ("FIXME: The default loglevel of %s is set to %s. Default log levels should not be lower than LOGL_NOTICE -- please fix!\n",</span><br><span style="color: hsl(120, 100%, 40%);">+                         log_info->cat[i].name,</span><br><span style="color: hsl(120, 100%, 40%);">+                             log_level_name(log_info->cat[i].loglevel));</span><br><span style="color: hsl(120, 100%, 40%);">+           }</span><br><span style="color: hsl(120, 100%, 40%);">+     }</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> int osmo_init_logging2(void *ctx, const struct log_info *log_info)</span><br><span> {</span><br><span>    static int logging_initialized = 0;</span><br><span>@@ -132,6 +147,8 @@</span><br><span> </span><br><span>        log_add_target(osmo_stderr_target);</span><br><span>  log_set_all_filter(osmo_stderr_target, 1);</span><br><span style="color: hsl(120, 100%, 40%);">+    check_loglevels(log_info);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>         return 0;</span><br><span> }</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/16833">change 16833</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/libosmocore/+/16833"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I2d7e345c14a188430a0e991bfd9fb0343d05ea92 </div>
<div style="display:none"> Gerrit-Change-Number: 16833 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>