svn commit: r1703408 - /tomcat/trunk/java/org/apache/catalina/tribes/io/ObjectReader.java

[markt]

Author: markt
Date: Wed Sep 16 13:37:35 2015
New Revision: 1703408

URL: http://svn.apache.org/r1703408
Log:
Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=58386
On the basis that access() and finish() are synchronized, extend 
synchronization to other methods that access same fields.

Modified:
    tomcat/trunk/java/org/apache/catalina/tribes/io/ObjectReader.java

Modified: tomcat/trunk/java/org/apache/catalina/tribes/io/ObjectReader.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/io/ObjectReader.java?rev=1703408&r1=1703407&r2=1703408&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/tribes/io/ObjectReader.java (original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/io/ObjectReader.java Wed Sep 
16 13:37:35 2015
@@ -82,7 +82,7 @@ public class ObjectReader {
         this.lastAccess = System.currentTimeMillis();
     }
 
-    public boolean isAccessed() {
+    public synchronized boolean isAccessed() {
         return this.accessed;
     }
 
@@ -150,7 +150,7 @@ public class ObjectReader {
         this.buffer = null;
     }
 
-    public long getLastAccess() {
+    public synchronized long getLastAccess() {
         return lastAccess;
     }
 
@@ -158,7 +158,7 @@ public class ObjectReader {
         return cancelled;
     }
 
-    public void setLastAccess(long lastAccess) {
+    public synchronized void setLastAccess(long lastAccess) {
         this.lastAccess = lastAccess;
     }
 



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