tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ma...@apache.org
Subject svn commit: r452287 - in /tomcat/container/tc5.5.x: catalina/src/share/org/apache/catalina/core/StandardPipeline.java webapps/docs/changelog.xml
Date Tue, 03 Oct 2006 01:06:13 GMT
Author: markt
Date: Mon Oct  2 18:06:12 2006
New Revision: 452287

URL: http://svn.apache.org/viewvc?view=rev&rev=452287
Log:
Fix bug 39724. Removing the only valve from a pipeline did not return the pipeline to its
original state. Patched provided by David Gagnon.

Modified:
    tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/core/StandardPipeline.java
    tomcat/container/tc5.5.x/webapps/docs/changelog.xml

Modified: tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/core/StandardPipeline.java
URL: http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/core/StandardPipeline.java?view=diff&rev=452287&r1=452286&r2=452287
==============================================================================
--- tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/core/StandardPipeline.java
(original)
+++ tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/core/StandardPipeline.java
Mon Oct  2 18:06:12 2006
@@ -530,6 +530,8 @@
             current = current.getNext();
         }
 
+        if (first == basic) first = null;
+        
         if (valve instanceof Contained)
             ((Contained) valve).setContainer(null);
 

Modified: tomcat/container/tc5.5.x/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/webapps/docs/changelog.xml?view=diff&rev=452287&r1=452286&r2=452287
==============================================================================
--- tomcat/container/tc5.5.x/webapps/docs/changelog.xml (original)
+++ tomcat/container/tc5.5.x/webapps/docs/changelog.xml Mon Oct  2 18:06:12 2006
@@ -29,6 +29,11 @@
         is required to enable this test. (markt)
       </fix>
       <fix>
+        <bug>39724</bug>: Removing the last valve from a pipeline did not
+        return the pipeline to the original state. Patch provided by
+        David Gagon. (markt)
+      </fix>
+      <fix>
         <bug>40528</bug>: Add missing message localisations as provided by
         Ben Clifford. (markt)
       </fix>



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message