Changes between Initial Version and Version 1 of Ticket #16835

Ticket #16835 – Description

-              initial
+              v1
 Function rtlogFlush() in vboxguest/common/log/log.c performs a paronoid check:
+{{{
     /*
      * If the ring buffer is active, the other destinations are only written
 …
+    }
     else ...
+}}}
 This implies that pLogger->pInt could be NULL (even if this is an unlikely paranoid case). In the else path we have the following code that dereferences pLogginer->pInt and hence we may have a NULL pointer deference:
+{{{
         if (pLogger->pInt->pfnFlush)
             pLogger->pInt->pfnFlush(pLogger);
+}}}
 I guess this is unlikely, but I think it would be good to correctly sanity check this for this corner case.