[Quickfix-users] FileLogFactory bug in 1.12.4 and 1.13.x

[Adam S. <Ada...@Wi...>]

Hi,

There seems to be a bug in FileLogFactory in both 1.12.4 and current trunk.

When FileLogFactory is initialized with a string parameter instead of session settings then second and all subsequent calls of create() method will return null pointer. Following patch fixes that:


--- FileLog.cpp~	2010-04-14 16:47:07.917954800 +0300
+++ FileLog.cpp	2010-04-14 16:51:12.609022800 +0300
@@ -36,7 +36,7 @@
 
   try
   {
-    if ( m_path.size() ) return new FileLog( m_path );
+    if ( m_path.size() ) return m_globalLog = new FileLog( m_path );
     std::string path;
     Dictionary settings = m_settings.get();
     path = settings.getString( FILE_LOG_PATH );



Regards,
Adam Szojda