tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ma...@apache.org
Subject svn commit: r1003861 - in /tomcat/trunk: java/org/apache/el/parser/AstFloatingPoint.java java/org/apache/el/parser/AstInteger.java java/org/apache/el/parser/AstString.java webapps/docs/changelog.xml
Date Sat, 02 Oct 2010 18:53:31 GMT
Author: markt
Date: Sat Oct  2 18:53:30 2010
New Revision: 1003861

URL: http://svn.apache.org/viewvc?rev=1003861&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49985
Make thread safe

Modified:
    tomcat/trunk/java/org/apache/el/parser/AstFloatingPoint.java
    tomcat/trunk/java/org/apache/el/parser/AstInteger.java
    tomcat/trunk/java/org/apache/el/parser/AstString.java
    tomcat/trunk/webapps/docs/changelog.xml

Modified: tomcat/trunk/java/org/apache/el/parser/AstFloatingPoint.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/el/parser/AstFloatingPoint.java?rev=1003861&r1=1003860&r2=1003861&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/el/parser/AstFloatingPoint.java (original)
+++ tomcat/trunk/java/org/apache/el/parser/AstFloatingPoint.java Sat Oct  2 18:53:30 2010
@@ -34,7 +34,7 @@ public final class AstFloatingPoint exte
         super(id);
     }
 
-    private Number number;
+    private volatile Number number;
 
     public Number getFloatingPoint() {
         if (this.number == null) {

Modified: tomcat/trunk/java/org/apache/el/parser/AstInteger.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/el/parser/AstInteger.java?rev=1003861&r1=1003860&r2=1003861&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/el/parser/AstInteger.java (original)
+++ tomcat/trunk/java/org/apache/el/parser/AstInteger.java Sat Oct  2 18:53:30 2010
@@ -34,7 +34,7 @@ public final class AstInteger extends Si
         super(id);
     }
 
-    private Number number;
+    private volatile Number number;
 
     protected Number getInteger() {
         if (this.number == null) {

Modified: tomcat/trunk/java/org/apache/el/parser/AstString.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/el/parser/AstString.java?rev=1003861&r1=1003860&r2=1003861&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/el/parser/AstString.java (original)
+++ tomcat/trunk/java/org/apache/el/parser/AstString.java Sat Oct  2 18:53:30 2010
@@ -32,7 +32,7 @@ public final class AstString extends Sim
         super(id);
     }
 
-    private String string;
+    private volatile String string;
 
     public String getString() {
         if (this.string == null) {

Modified: tomcat/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1003861&r1=1003860&r2=1003861&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/trunk/webapps/docs/changelog.xml Sat Oct  2 18:53:30 2010
@@ -269,6 +269,9 @@
         Catalina code as well as making it easier for Geronimo and others to
         integrate Jasper. Patch provided by David Jencks. (markt)
       </fix>
+      <fix>
+        <bug>49985</bug>: Fix thread safety issue in EL parser. (markt)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Cluster">



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


Mime
View raw message