Author: rjung
Date: Tue Jun 30 20:28:30 2009
New Revision: 789935

URL: http://svn.apache.org/viewvc?rev=789935&view=rev
Log:
Fix BZ 46717: Hard to reproduce thread safety issue with
session expiration. Port of r781735 from Tomcat 5.5.

Modified:
    tomcat/sandbox/tomcat-oacc/trunk/docs/changelog.xml
    
tomcat/sandbox/tomcat-oacc/trunk/src/share/org/apache/catalina/cluster/session/DeltaSession.java

Modified: tomcat/sandbox/tomcat-oacc/trunk/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/sandbox/tomcat-oacc/trunk/docs/changelog.xml?rev=789935&r1=789934&r2=789935&view=diff
==============================================================================
--- tomcat/sandbox/tomcat-oacc/trunk/docs/changelog.xml (original)
+++ tomcat/sandbox/tomcat-oacc/trunk/docs/changelog.xml Tue Jun 30 20:28:30 2009
@@ -33,6 +33,11 @@
   <subsection name="Cluster">
       <changelog>
       <fix>
+        <bug>46717</bug>: Hard to reproduce thread safety issue with
+        session expiration.
+        Port from Tomcat 5.5. (rjung)
+      </fix>
+      <fix>
         Fix misprints in comments and javadoc.
         Port from Tomcat 5.5. (rjung)
       </fix>

Modified: 
tomcat/sandbox/tomcat-oacc/trunk/src/share/org/apache/catalina/cluster/session/DeltaSession.java
URL: 
http://svn.apache.org/viewvc/tomcat/sandbox/tomcat-oacc/trunk/src/share/org/apache/catalina/cluster/session/DeltaSession.java?rev=789935&r1=789934&r2=789935&view=diff
==============================================================================
--- 
tomcat/sandbox/tomcat-oacc/trunk/src/share/org/apache/catalina/cluster/session/DeltaSession.java
 (original)
+++ 
tomcat/sandbox/tomcat-oacc/trunk/src/share/org/apache/catalina/cluster/session/DeltaSession.java
 Tue Jun 30 20:28:30 2009
@@ -164,7 +164,7 @@
     /**
      * The last accessed time for this Session.
      */
-    private long lastAccessedTime = creationTime;
+    private volatile long lastAccessedTime = creationTime;
 
     /**
      * The session event listeners for this Session.
@@ -234,7 +234,7 @@
     /**
      * The current accessed time for this session.
      */
-    private long thisAccessedTime = creationTime;
+    private volatile long thisAccessedTime = creationTime;
 
     /**
      * only the primary session will expire, or be able to expire due to



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

Reply via email to