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