You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2014/07/24 17:59:46 UTC

svn commit: r1613171 - in /logging/log4j/log4j2/trunk: log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java src/changes/changes.xml

Author: ggregory
Date: Thu Jul 24 15:59:45 2014
New Revision: 1613171

URL: http://svn.apache.org/r1613171
Log:
[LOG4J2-738] RollingFileManager deadlock if async action thread fails to start.

Modified:
    logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java
    logging/log4j/log4j2/trunk/src/changes/changes.xml

Modified: logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java?rev=1613171&r1=1613170&r2=1613171&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java (original)
+++ logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java Thu Jul 24 15:59:45 2014
@@ -187,7 +187,7 @@ public class RollingFileManager extends 
             }
             return false;
         } finally {
-            if (thread == null) {
+            if (thread == null || !thread.isAlive()) {
                 semaphore.release();
             }
         }

Modified: logging/log4j/log4j2/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/src/changes/changes.xml?rev=1613171&r1=1613170&r2=1613171&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/src/changes/changes.xml (original)
+++ logging/log4j/log4j2/trunk/src/changes/changes.xml Thu Jul 24 15:59:45 2014
@@ -22,6 +22,9 @@
   </properties>
   <body>
     <release version="?" date="2014-mm-dd" description="?">
+      <action issue="LOG4J2-738" dev="ggregory" type="fix" due-to="Timothy Stack">
+        RollingFileManager deadlock if async action thread fails to start.
+      </action>
       <action issue="LOG4J2-736" dev="mattsicker" type="fix">
         Fixed log4j-bom so that it won't specify a default scope on any third party dependencies.
       </action>