Re: RFR:JDK-8168906-Tighten permissions granted to the jdk.localedata module

2016-11-16 Thread Masayoshi Okutsu
test/sun/util/locale/provider/Bug8152817.java is a test with a SecurityManager. I18n SQE should have some. Masayoshi On 11/14/2016 11:59 PM, Sean Mullan wrote: Looks good. Are there any regression tests for this component that run with a SecurityManager? If not, it would be useful to add some

Re: RFR:JDK-8168906-Tighten permissions granted to the jdk.localedata module

2016-11-16 Thread Masayoshi Okutsu
+1 Masayoshi On 11/15/2016 1:07 AM, Naoto Sato wrote: +1 Naoto On 11/13/16 11:12 PM, Rachna Goel wrote: Hi, Kindly review fix for JDK-8168906. https://bugs.openjdk.java.net/browse/JDK-8168906 Patch :http://cr.openjdk.java.net/~rgoel/JDK-8168906/webrev/ fix: As jdk.localedata module does

Re: RFR:JDK-8168906-Tighten permissions granted to the jdk.localedata module

2016-11-14 Thread Naoto Sato
+1 Naoto On 11/13/16 11:12 PM, Rachna Goel wrote: Hi, Kindly review fix for JDK-8168906. https://bugs.openjdk.java.net/browse/JDK-8168906 Patch :http://cr.openjdk.java.net/~rgoel/JDK-8168906/webrev/ fix: As jdk.localedata module does read any system properties, tightened permissions for same

Re: RFR:JDK-8168906-Tighten permissions granted to the jdk.localedata module

2016-11-14 Thread Sean Mullan
Looks good. Are there any regression tests for this component that run with a SecurityManager? If not, it would be useful to add some to ensure that the proper permissions are being granted to this module. --Sean On 11/14/16 2:27 AM, Rachna Goel wrote: sorry, correction to typo As jdk.locale

Re: RFR:JDK-8168906-Tighten permissions granted to the jdk.localedata module

2016-11-13 Thread Rachna Goel
sorry, correction to typo As jdk.localedata module does *not* read any system properties, tightened permissions for same. On 14/11/16 12:42 PM, Rachna Goel wrote: Hi, Kindly review fix for JDK-8168906. https://bugs.openjdk.java.net/browse/JDK-8168906 Patch :http://cr.openjdk.java.net/~r

Re: RFR:JDK-8168906-Tighten permissions granted to the jdk.localedata module

2016-11-13 Thread Alan Bateman
On 14/11/2016 07:12, Rachna Goel wrote: Hi, Kindly review fix for JDK-8168906. https://bugs.openjdk.java.net/browse/JDK-8168906 Patch :http://cr.openjdk.java.net/~rgoel/JDK-8168906/webrev/ fix: As jdk.localedata module does read any system properties, tightened permissions for same. If you

RFR:JDK-8168906-Tighten permissions granted to the jdk.localedata module

2016-11-13 Thread Rachna Goel
Hi, Kindly review fix for JDK-8168906. https://bugs.openjdk.java.net/browse/JDK-8168906 Patch :http://cr.openjdk.java.net/~rgoel/JDK-8168906/webrev/ fix: As jdk.localedata module does read any system properties, tightened permissions for same. Thanks, Rachna