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 01:48:32 PDT 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: http://mail.openjdk.java.net/pipermail/security-dev/attachments/20120718/d5a1d08a/attachment.html 


More information about the security-dev mailing list