From 0778211116cc302afa9db32e4986650b728e6124 Mon Sep 17 00:00:00 2001
From: tt4g <mainlinett@gmail.com>
Date: Wed, 12 Feb 2020 09:58:16 +0900
Subject: [PATCH] Add critical section to filename function of  each file sink

---
 include/spdlog/sinks/daily_file_sink.h        | 3 ++-
 include/spdlog/sinks/rotating_file_sink-inl.h | 3 ++-
 include/spdlog/sinks/rotating_file_sink.h     | 2 +-
 3 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/include/spdlog/sinks/daily_file_sink.h b/include/spdlog/sinks/daily_file_sink.h
index e122f10f..65ab24f8 100644
--- a/include/spdlog/sinks/daily_file_sink.h
+++ b/include/spdlog/sinks/daily_file_sink.h
@@ -70,8 +70,9 @@ public:
         }
     }
 
-    const filename_t &filename() const
+    const filename_t &filename()
     {
+        std::lock_guard<Mutex> lock(base_sink<Mutex>::mutex_);
         return file_helper_.filename();
     }
 
diff --git a/include/spdlog/sinks/rotating_file_sink-inl.h b/include/spdlog/sinks/rotating_file_sink-inl.h
index bd2d175d..0d27f057 100644
--- a/include/spdlog/sinks/rotating_file_sink-inl.h
+++ b/include/spdlog/sinks/rotating_file_sink-inl.h
@@ -54,8 +54,9 @@ SPDLOG_INLINE filename_t rotating_file_sink<Mutex>::calc_filename(const filename
 }
 
 template<typename Mutex>
-SPDLOG_INLINE const filename_t &rotating_file_sink<Mutex>::filename() const
+SPDLOG_INLINE const filename_t &rotating_file_sink<Mutex>::filename()
 {
+    std::lock_guard<Mutex> lock(base_sink<Mutex>::mutex_);
     return file_helper_.filename();
 }
 
diff --git a/include/spdlog/sinks/rotating_file_sink.h b/include/spdlog/sinks/rotating_file_sink.h
index 5be8583a..e306dd9b 100644
--- a/include/spdlog/sinks/rotating_file_sink.h
+++ b/include/spdlog/sinks/rotating_file_sink.h
@@ -24,7 +24,7 @@ class rotating_file_sink final : public base_sink<Mutex>
 public:
     rotating_file_sink(filename_t base_filename, std::size_t max_size, std::size_t max_files, bool rotate_on_open = false);
     static filename_t calc_filename(const filename_t &filename, std::size_t index);
-    const filename_t &filename() const;
+    const filename_t &filename();
 
 protected:
     void sink_it_(const details::log_msg &msg) override;