Author: kfujino
Date: Wed Apr 14 08:48:43 2010
New Revision: 933881

URL: http://svn.apache.org/viewvc?rev=933881&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49116
If session is already invalid, expire session to prevent memory leak.

Modified:
    tomcat/trunk/java/org/apache/catalina/session/StandardManager.java

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=933881&r1=933880&r2=933881&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/session/StandardManager.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/session/StandardManager.java Wed Apr 
14 08:48:43 2010
@@ -397,6 +397,12 @@ public class StandardManager extends Man
                     session.setManager(this);
                     sessions.put(session.getIdInternal(), session);
                     session.activate();
+                    if (!session.isValidInternal()) {
+                        // If session is already invalid,
+                        // expire session to prevent memory leak.
+                        session.setValid(true);
+                        session.expire();
+                    }
                     sessionCounter++;
                 }
             } catch (ClassNotFoundException e) {



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

Reply via email to