Fix possible Null reference in colored logging (Coverity #188445)
authorHenning Rogge <hrogge@gmail.com>
Fri, 11 May 2018 04:39:40 +0000 (06:39 +0200)
committerHenning Rogge <hrogge@gmail.com>
Fri, 11 May 2018 04:39:40 +0000 (06:39 +0200)
src/libcore/oonf_logging.c

index e8ab489..67a3805 100644 (file)
@@ -438,7 +438,7 @@ void
 oonf_log_stderr(struct oonf_log_handler_entry *entry, struct oonf_log_parameters *param) {
   bool *color = entry->custom;
 
-  if (*color) {
+  if (color && *color) {
     switch (param->severity) {
       case LOG_SEVERITY_INFO:
         fputs("\033[0;33m", stderr);
@@ -452,7 +452,7 @@ oonf_log_stderr(struct oonf_log_handler_entry *entry, struct oonf_log_parameters
   }
   fputs(param->buffer, stderr);
   fputc('\n', stderr);
-  if (color && param->severity != LOG_SEVERITY_DEBUG) {
+  if (color && *color && param->severity != LOG_SEVERITY_DEBUG) {
     fputs("\033[0m", stderr);
   }
 }