<i18n dev> RFR: 8152817: Locale data loading fails silently when running with a security manager

Alan Bateman Alan.Bateman at oracle.com
Wed Mar 30 15:54:49 UTC 2016

On 30/03/2016 16:48, Mandy Chung wrote:
>> On Mar 30, 2016, at 8:40 AM, Masayoshi Okutsu <masayoshi.okutsu at oracle.com> wrote:
>> Hello,
>> Please review the fix for JDK-8152817. The fix is to load locale data from its own module without calling ResourceBundleProviderSupport.loadResourceBundle.
>> I changed the synopsis of the JBS issue because it's a general issue of loading locale data under a security manager, and the regression test uses DateFormatSymbols.
>> Issue:
>> https://bugs.openjdk.java.net/browse/JDK-8152817
>> Webrev:
>> http://cr.openjdk.java.net/~okutsu/9/8152817/webrev.00/
> The patch looks fine.  It makes sense to have its own copy of the utility method to load a module-private class.
I agree. There's another usage in SupplementaryLocaleDataProvider and I 
wonder if we have an issue there too.

> You can call Class::newInstance instead of Constructor::newInstance to make the code much simpler. Class::newInstance doesn’t throw InvocationTargetException and no need to have uncheckedThrow.
I was wondering about that too.


More information about the i18n-dev mailing list