On 13 October 2010 15:41,  <ma...@apache.org> wrote:
> Author: markt
> Date: Wed Oct 13 14:41:57 2010
> New Revision: 1022117
>
> URL: http://svn.apache.org/viewvc?rev=1022117&view=rev
> Log:
> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49987
> Thread safety issue with population of servlet context initialisation 
> parameters.
> Back-port wasn't completely clean - please review.
>
> Modified:
>    tomcat/tc6.0.x/trunk/   (props changed)
>    tomcat/tc6.0.x/trunk/STATUS.txt
>    tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/ApplicationContext.java
>    tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/StandardContext.java
>    tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
>
> Modified: 
> tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/ApplicationContext.java
> URL: 
> http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/ApplicationContext.java?rev=1022117&r1=1022116&r2=1022117&view=diff
> ==============================================================================
> --- 
> tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/ApplicationContext.java 
> (original)
> +++ 
> tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/ApplicationContext.java 
> Wed Oct 13 14:41:57 2010
> @@ -122,7 +122,8 @@ public class ApplicationContext
>     /**
>      * The merged context initialization parameters for this Context.
>      */
> -    private Map parameters = null;
> +    private Map<String,String> parameters =
> +        new ConcurrentHashMap<String,String>();

This should be final, surely?

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

Reply via email to