Re: Review Request: 8073361: Missing doPrivileged in com.sun.xml.internal.bind.v2.ClassFactory

2015-02-24 Thread Sean Mullan
This fix looks fine. --Sean On 02/20/2015 01:54 PM, Mandy Chung wrote: This fixes a regression due to JDK-8057645 moving JAXB to ext loader that was tested before the fix for JDK-8054367 went in jdk9. This was uncovered after JDK-8057645and JDK-8054367 met. The fix is simple.

Review Request: 8073361: Missing doPrivileged in com.sun.xml.internal.bind.v2.ClassFactory

2015-02-20 Thread Mandy Chung
This fixes a regression due to JDK-8057645 moving JAXB to ext loader that was tested before the fix for JDK-8054367 went in jdk9. This was uncovered after JDK-8057645and JDK-8054367 met. The fix is simple. Class.getDeclaredConstructor on a class defined by the null loader will require

Re: Review Request: 8073361: Missing doPrivileged in com.sun.xml.internal.bind.v2.ClassFactory

2015-02-20 Thread Daniel Fuchs
Hi Mandy, On 20/02/15 19:54, Mandy Chung wrote: This fixes a regression due to JDK-8057645 moving JAXB to ext loader that was tested before the fix for JDK-8054367 went in jdk9. This was uncovered after JDK-8057645and JDK-8054367 met. The fix is simple. Class.getDeclaredConstructor on a