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-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
