Update CHANGELOG for v1.0.3; Fix crash on empty string_view logging; Add unit test for empty string_view

Author: kzhdev

CHANGELOG

@@ -1,3 +1,9 @@
+# v1.0.3 - 01-08-2026
+- Fix crash when logging empty string_view values
+  - Add length check before memcpy in store_string_in_queue() to handle empty strings safely
+  - Set null terminator directly for zero-length strings instead of copying
+- Add comprehensive unit test for empty string_view logging (EmptyStringView test)
+
 # v1.0.2 - 01-03-2026
 - Update slick_queue to v1.2.1
 - Added GitHub release workflow

CMakeLists.txt

@@ -1,7 +1,7 @@
 cmake_minimum_required(VERSION 3.20)
 
 project(slick_logger 
-    VERSION 1.0.2
+    VERSION 1.0.3
     LANGUAGES CXX)
 
 set(CMAKE_CXX_STANDARD 20)

include/slick/logger.hpp

@@ -1576,10 +1576,15 @@ inline StringRef Logger::store_string_in_queue(std::string_view str) {
 
     // Reserve space in string queue
     uint64_t start_index = string_queue_->reserve(len);
-    // Copy string data
-    char* dest = (*string_queue_)[start_index];
-    std::memcpy(dest, str.data(), len);
 
+    char* dest = (*string_queue_)[start_index];
+    if (length) {
+        // Copy string data
+        std::memcpy(dest, str.data(), len);
+    } 
+    else {
+        *dest = '\0';
+    } 
     // Publish the string data
     string_queue_->publish(start_index, len);
     return StringRef{dest, length};

src/logger.hpp

@@ -1574,10 +1574,15 @@ inline StringRef Logger::store_string_in_queue(std::string_view str) {
 
     // Reserve space in string queue
     uint64_t start_index = string_queue_->reserve(len);
-    // Copy string data
-    char* dest = (*string_queue_)[start_index];
-    std::memcpy(dest, str.data(), len);
 
+    char* dest = (*string_queue_)[start_index];
+    if (length) {
+        // Copy string data
+        std::memcpy(dest, str.data(), len);
+    } 
+    else {
+        *dest = '\0';
+    } 
     // Publish the string data
     string_queue_->publish(start_index, len);
     return StringRef{dest, length};

tests/test_logger.cpp

@@ -17,6 +17,7 @@ class SlickLoggerTest : public ::testing::Test {
         std::filesystem::remove("test_mixed.log");
         std::filesystem::remove("test_char_array.log");
         std::filesystem::remove("test_single_string.log");
+        std::filesystem::remove("test_empty_string.log");
     }
 };
 
@@ -347,6 +348,28 @@ TEST_F(SlickLoggerTest, CharArrayLogging) {
     EXPECT_TRUE(file_contents.find("Log char array: test char array") != std::string::npos);
 }
 
+TEST_F(SlickLoggerTest, EmptyStringView) {
+    std::filesystem::remove("test_empty_string.log");
+    
+    slick::logger::Logger::instance().init("test_empty_string.log", 8);
+
+    std::string_view s;
+    
+    LOG_INFO("Log empty string: {}", s);
+    
+    slick::logger::Logger::instance().shutdown();
+    
+    ASSERT_TRUE(std::filesystem::exists("test_empty_string.log"));
+    
+    std::ifstream log_file("test_empty_string.log");
+    std::string line;
+    std::getline(log_file, line);   // first line is the logger's version
+    EXPECT_TRUE(std::getline(log_file, line));
+    // Check valid formats work
+    EXPECT_EQ(line.size(), 52);
+    EXPECT_EQ(line.find(" [INFO] Log empty string: "), 26);
+}
+
 int main(int argc, char **argv) {
     testing::InitGoogleTest(&argc, argv);
     return RUN_ALL_TESTS();