RCORE-2234 Crash in dart due to debug output when app is being torn d…

…own (#7985)

* Updated stderrlogger to use a mutex shared_ptr instead of a static
* Removed additional Logger static defines
* Updates from review - removed StderrLogger mutex - reverted some changes
* Reverted unnecessary changes, using global pointer mutex reference
* Removed StderrLogger local mutex reference

CHANGELOG.md

@@ -8,6 +8,7 @@
 * <How do the end-user experience this issue? what was the impact?> ([#????](https://github.com/realm/realm-core/issues/????), since v?.?.?)
 * Using an empty KeyPath in C API would result in no filtering being done ([#7805](https://github.com/realm/realm-core/issues/7805), since 13.24.0)
 * Filtering notifications with backlink columns as last element could sometimes give wrong results ([#7530](https://github.com/realm/realm-core/issues/7530), since 11.1.0)
+* Fix crash during client app shutdown when Logger log level is set higher than Info. ([#7969](https://github.com/realm/realm-core/issues/7969), since v13.23.3)
 
 ### Breaking changes
 * None.

src/realm/util/logger.cpp

@@ -26,6 +26,7 @@ namespace realm::util {
 
 namespace {
 auto& s_logger_mutex = *new std::mutex;
+auto& s_stderr_logger_mutex = *new std::mutex;
 std::shared_ptr<util::Logger> s_default_logger;
 } // anonymous namespace
 
@@ -177,8 +178,7 @@ const std::string_view Logger::level_to_string(Level level) noexcept
 
 void StderrLogger::do_log(const LogCategory& cat, Level level, const std::string& message)
 {
-    static Mutex mutex;
-    LockGuard l(mutex);
+    std::lock_guard l(s_stderr_logger_mutex);
     // std::cerr is unbuffered, so no need to flush
     std::cerr << cat.get_name() << " - " << get_level_prefix(level) << message << '\n'; // Throws
 }