Author: slaurent
Date: Fri Dec  3 22:00:58 2010
New Revision: 1042022

URL: http://svn.apache.org/viewvc?rev=1042022&view=rev
Log:
bug 48973 (enhancement): Don't create SESSIONS.ser when no session exists 
https://issues.apache.org/bugzilla/show_bug.cgi?id=48973

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt
    tomcat/trunk/java/org/apache/catalina/session/StandardManager.java
    tomcat/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1042022&r1=1042021&r2=1042022&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri Dec  3 22:00:58 2010
@@ -207,3 +207,10 @@ PATCHES PROPOSED TO BACKPORT:
   http://people.apache.org/~markt/patches/2010-12-02-bug50201-tc6.patch
   +1: markt
   -1:
+
+* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48973
+  Avoid creating file SESSIONS.ser if there's no session.
+  The patch provided by Marc Guillemot works for tc 6 & 7
+  +1: slaurent
+  -1:
+  
\ No newline at end of file

Modified: tomcat/trunk/java/org/apache/catalina/session/StandardManager.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/session/StandardManager.java?rev=1042022&r1=1042021&r2=1042022&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/session/StandardManager.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/session/StandardManager.java Fri Dec  
3 22:00:58 2010
@@ -366,6 +366,11 @@ public class StandardManager extends Man
 
         if (log.isDebugEnabled())
             log.debug("Unloading persisted sessions");
+        
+        if (sessions.isEmpty()) {
+            log.debug("No persisted sessions to unload");
+            return; // nothing to do
+        }
 
         // Open an output stream to the specified pathname, if any
         File file = file();

Modified: tomcat/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1042022&r1=1042021&r2=1042022&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/trunk/webapps/docs/changelog.xml Fri Dec  3 22:00:58 2010
@@ -33,6 +33,7 @@
     <author email="t...@apache.org">Tim Whittington</author>    
     <author email="mt...@apache.org">Mladen Turk</author>
     <author email="schu...@apache.org">Christopher Schultz</author>
+    <author email="slaur...@apache.org">Sylvain Laurent</author>
     <title>Changelog</title>
   </properties>
 
@@ -41,6 +42,11 @@
 <section name="Tomcat 7.0.6 (markt)">
   <subsection name="Catalina">
     <changelog>
+      <add>
+        <bug>48973</bug>: Avoid creating a SESSIONS.ser file when stopping an 
+        application if there's no session. Patch provided by Marc Guillemot.
+        (slaurent)
+      </add>
       <update>
         <bug>48692</bug>: Provide option to parse
         <code>application/x-www-form-urlencoded</code> PUT requests. (schultz)



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

Reply via email to