[jboss-cvs] JBossAS SVN: r100566 - branches/JBPAPP_5_0/tomcat/src/main/org/jboss/web/tomcat/service/session.

[jboss-cvs-commits at lists.jboss.org]

Author: bstansberry at jboss.com
Date: 2010-02-05 15:37:13 -0500 (Fri, 05 Feb 2010)
New Revision: 100566

Modified:
   branches/JBPAPP_5_0/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossCacheManager.java
Log:
[JBPAPP-2929] Don't leak ref to embryonic session

Modified: branches/JBPAPP_5_0/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossCacheManager.java
===================================================================
--- branches/JBPAPP_5_0/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossCacheManager.java	2010-02-05 19:39:59 UTC (rev 100565)
+++ branches/JBPAPP_5_0/tomcat/src/main/org/jboss/web/tomcat/service/session/JBossCacheManager.java	2010-02-05 20:37:13 UTC (rev 100566)
@@ -2162,6 +2162,14 @@
                      log_.trace("loadSession(): session " + realId +
                                 " not found in distributed cache");
                   }
+                  
+                  if (initialLoad)
+                  {                     
+                     // The session is now in the regular map, or the session
+                     // doesn't exist in the distributed cache. either way
+                     // it's now safe to stop tracking this embryonic session
+                     embryonicSessions.remove(realId);
+                  }
                }
             }
             finally