Author: kkolinko
Date: Mon Nov 9 14:41:35 2009
New Revision: 834084
URL: http://svn.apache.org/viewvc?rev=834084&view=rev
Log:
Revoke patch that has concerns. Vote for the alternative one.
Modified:
tomcat/tc5.5.x/trunk/STATUS.txt
tomcat/tc6.0.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=834084&r1=834083&r2=834084&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/STATUS.txt (original)
+++ tomcat/tc5.5.x/trunk/STATUS.txt Mon Nov 9 14:41:35 2009
@@ -159,8 +159,8 @@
+1: kkolinko
-1:
- 2) Patch for SecurityClassLoad to preload o.a.c.loader.ResourceEntry class
- http://svn.apache.org/viewvc?rev=831830&view=rev
+ 2) Add a new PrivilegedAction. Patch by markt
+ http://svn.apache.org/viewvc?rev=834080&view=rev
+1: kkolinko
-1:
Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL:
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=834084&r1=834083&r2=834084&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Mon Nov 9 14:41:35 2009
@@ -196,18 +196,13 @@
-1:
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48097
- Patch for SecurityClassLoad to preload o.a.c.loader.ResourceEntry class
- http://svn.apache.org/viewvc?rev=831830&view=rev
- +1: kkolinko
- -0: billbarker Exposing ResourceEntry is harmless enough, but
WebappClassLoader should
- really be using it's PrivilegedFindResource class in this case instead.
- After all, that is what it is there for.
- -1:
Alternative patch that adds a new PrivilegedAction. The test case provided
passes with this patch
http://people.apache.org/~markt/patches/2009-11-06-bug48097-alt.patch
- +1: markt, funkman, billbarker
+ +1: markt, funkman, billbarker, kkolinko
-1:
+ kkolinko: Confirming that testcase passes. Applied to trunk as
+ http://svn.apache.org/viewvc?rev=834080&view=rev
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47893
Use StringBuilder instead of StringBuffer
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
