[LOGCXX-394] Levels are not thread safe

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Component/s: Appender
• Labels: thread-safety
• Environment: Windows 7, Visual Studio 2010 SP1
• Assignee: Thorsten Schöning
• Reporter: Petro Protsyk
• Created: 05/Mar/12 15:54
• Updated: 03/Mar/20 07:45
• Resolved: 16/Feb/14 19:18

Description

Level.cpp. All default level variables are function static. This is not thread safe, especially in Visual C++ compiler.

Here is code to reproduce the issue in VC++ 2010:

Concurrency::parallel_for (int(0), int(10), [&](int i)
{
if (::log4cxx::Level::getWarn() == NULL)

{ throw std::exception(); }

});

Issue Links

• is duplicated by: LOGCXX-411 Crash when logging on multiple threads.
• is related to: LOGCXX-322 Crashes on exit from multithreaded program using log4cxx
• relates to: LOGCXX-430 LogManager::getRootLogger is not thread-safe
• relates to: LOGCXX-485 Levels leak memory