You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2010/03/05 11:51:01 UTC
svn commit: r919367 - /tomcat/tc5.5.x/trunk/STATUS.txt
Author: markt
Date: Fri Mar 5 10:51:01 2010
New Revision: 919367
URL: http://svn.apache.org/viewvc?rev=919367&view=rev
Log:
Update patch with deadlock fix
Modified:
tomcat/tc5.5.x/trunk/STATUS.txt
Modified: tomcat/tc5.5.x/trunk/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/STATUS.txt?rev=919367&r1=919366&r2=919367&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/STATUS.txt (original)
+++ tomcat/tc5.5.x/trunk/STATUS.txt Fri Mar 5 10:51:01 2010
@@ -48,14 +48,13 @@
-1:
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=44041
- Threading issue in class loading (using Filip's suggestion)
- http://people.apache.org/~markt/patches/2009-11-18-bug44041.patch
- +1: markt, kkolinko
+ Threading issue in class loading (using Filip's suggestion) plus fix for
+ https://issues.apache.org/bugzilla/show_bug.cgi?id=48694 a potential deadlock
+ http://people.apache.org/~markt/patches/2010-03-05-bug44041.patch
+ Note: The bulk of the patch just adds synchronized (name.intern()) {"
+ around the loadClass(String, boolean) method
+ +1: markt
-1:
- kkolinko: To reviewers: the patch looks scary, but the only thing it
- actually does is to add "synchronized (name.intern()) {" around the
- whole contents of the method.
- rjung: Isn't there a possibility for deadlocks?
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47878
Return 404's rather than a permanent 500 if a JSP is deleted
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org