tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Konstantin Kolinko <knst.koli...@gmail.com>
Subject Re: svn commit: r905710 - /tomcat/tc5.5.x/trunk/STATUS.txt
Date Tue, 02 Feb 2010 17:53:06 GMT
2010/2/2 Mark Thomas <markt@apache.org>:
> On 02/02/2010 17:31, Konstantin Kolinko wrote:
>> 2010/2/2  <markt@apache.org>:
>>> Author: markt
>>> Date: Tue Feb  2 17:10:05 2010
>>> New Revision: 905710
>>>
>>> URL: http://svn.apache.org/viewvc?rev=905710&view=rev
>>> Log:
>>> Proposal
>>>
>>> Modified:
>>>    tomcat/tc5.5.x/trunk/STATUS.txt
>>>
>>
>>> +* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48581
>>> +  Avoid security exception on first access
>>> +  http://people.apache.org/~markt/patches/2010-02-02-bug48581.patch
>>> +  +1: markt
>>> +  -1:
>>
>> A very elaborate patch.  So, it was "org.apache.coyote.Constants".
>> I thought that we would just preload the class.
>
> Yeah, it did get a bit out of hand. Pre-loading should work too.
>>
>> Also, it would be better to do the same for trunk and 6.0 as well.
>
> Is the error reproducible on those versions?
>

Fair question. It is not needed there.

In 6.0 and trunk there are a lot of classes (such as Http11Processor)
that reference o.a.coyote.Constants.   Because of that it will be
loaded when connectors are instantiated.

There are far less references to it in TC5.5.


Best regards,
Konstantin Kolinko

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


Mime
View raw message