protext all access to _logstream with the existing mutex

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>

diff --git a/src/libsync/logger.cpp b/src/libsync/logger.cpp
index 42830d32f312582e0823e4a3964adeda10f0da2d..d38be4c6bf69358bbeac76045ca6441a76876588 100644 (file)
--- a/src/libsync/logger.cpp
+++ b/src/libsync/logger.cpp
@@ -118,7 +118,6 @@ void Logger::doLog(QtMsgType type, const QMessageLogContext &ctx, const QString
     cout << msg << endl;
 #endif
     {
-        QMutexLocker lock(&_mutex);
         if (_logFile.size() >= MaxLogSizeBytes) {
             close();
             enterNextLogFile();
@@ -127,6 +126,8 @@ void Logger::doLog(QtMsgType type, const QMessageLogContext &ctx, const QString
         _crashLogIndex = (_crashLogIndex + 1) % CrashLogSize;
         _crashLog[_crashLogIndex] = msg;
 
+        QMutexLocker lock(&_mutex);
+
         if (_logstream) {
             (*_logstream) << msg << Qt::endl;
             if (_doFileFlush)
@@ -145,6 +146,7 @@ void Logger::doLog(QtMsgType type, const QMessageLogContext &ctx, const QString
 
 void Logger::close()
 {
+    QMutexLocker lock(&_mutex);
     dumpCrashLog();
     if (_logstream)
     {