You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2010/02/23 08:27:33 UTC

svn commit: r915220 - /tomcat/tc6.0.x/trunk/STATUS.txt

Author: kkolinko
Date: Tue Feb 23 07:27:33 2010
New Revision: 915220

URL: http://svn.apache.org/viewvc?rev=915220&view=rev
Log:
votes

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=915220&r1=915219&r2=915220&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Tue Feb 23 07:27:33 2010
@@ -209,14 +209,20 @@
   Avoid potential deadlock in WebappClassLoader
   OP confirms patch fixes the deadlock
   https://issues.apache.org/bugzilla/show_bug.cgi?id=48694#c1
-  +1: markt, jfclere
+  +1: markt, jfclere, kkolinko
   -1: 
+   kkolinko: Looking at how URLClassLoader.findClass() is implemented, I
+   think that concurrent access to WebappClassLoader.findClass() should not
+   be allowed.  URLClassLoader.findClass() is a protected method, but it is
+   public in URLClassLoader.  I think we should change its access qualifier
+   to be 'protected' in TC7.
+
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48716
   Provide an undocumented 'shoot yourself in the foot' option to disable the
   JULI logging reset on context stop
   http://svn.apache.org/viewvc?rev=908754&view=rev
-  +1: markt, fhanik
+  +1: markt, fhanik, kkolinko
   -1: 
 
 * More memory leak prevent - ResourceBundle
@@ -309,5 +315,5 @@
 
 * Various manifest improvements
   http://people.apache.org/~markt/patches/2010-02-16-manifest-improvements.patch
-  +1: markt, fhanik
+  +1: markt, fhanik, kkolinko
   -1: 



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