Review request (Resend) - bug 7147060 com/sun/org/apache/xml/internal/security/transforms/ClassLoaderTest.java doesn't run in agentvm mode
Eric Wang
yiming.wang at oracle.com
Wed Jul 18 08:48:32 UTC 2012
Hi Sean,
I'd like to withdraw my fix as there are some concerns about bug 6461674
<http://monaco.us.oracle.com/detail.jsf?cr=6461674> which is verified by
ClassLoaderTest.java (Thanks Max for good suggestions):
The bug 6461674 is caused by classes cannot be found in current
classloader by calling Class.forName(), so the fix of 6461674 is to
replace Class.forName() to
Thread.currentThread().getContextClassLoader(), however, it looks the
bug 7147060 <http://monaco.us.oracle.com/detail.jsf?cr=7147060> is
negative case of 6461674 that classes can be found in current
ClassLoader but cannot be found in thread context ClassLoader because in
jtreg agentvm mode, the thread context ClassLoader is parent of current
ClassLoader.
so the concern is whether the fix of 6461674 is completed as it looks
that it produced the side effect.
Regards,
Eric
<http://monaco.us.oracle.com/detail.jsf?cr=6461674>
On 2012/7/18 15:05, Eric Wang wrote:
> Hi Sean,
>
> As i haven't get the author role, I share webrev to group by using
> dropbox, Can you please help to review it? Thanks!
> http://dl.dropbox.com/u/90659131/fixes/7147060/webrev/index.html
>
> Regards,
> Eric
>
> On 2012/7/9 16:13, Eric Wang wrote:
>> Hi Sean,
>>
>> Looks that security-dev alias rejected my previous mail as i was not
>> the member of it. so I resent this fix again.
>> Can you please help to review? Thanks!
>>
>> Eric
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/security-dev/attachments/20120718/d5a1d08a/attachment.htm>
More information about the security-dev
mailing list