Re: RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-16 Thread Weijun Wang
On 02/14/2017 11:55 PM, Sean Mullan wrote: Hi Max, I agree this change is necessary so that we can resolve this tck-red issue before ZBB. However, since the TCK Policy provider implementation is not a "typical" implementation in the sense that it is denying permissions instead of granting perm

Re: RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-15 Thread Weijun Wang
The last existing paragraph is already A system property named jdk.io.permissionsUseCanonicalPath has also been introduced. When it is set to "true", FilePermission will canonicalize its pathname as it did before JDK 9. The default value of this property is "false". and I'd like the first

Re: RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-15 Thread Sean Mullan
On 2/15/17 3:35 AM, Weijun Wang wrote: I's also like to append the following paragraph to the current release note at https://bugs.openjdk.java.net/browse/JDK-8165836: Another new system property named jdk.security.filePermCompat, when set to "true", allows the compatibility layer above to wo

Re: RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-15 Thread Sean Mullan
Looks good. --Sean On 2/14/17 8:04 PM, Weijun Wang wrote: Webrev updated at http://cr.openjdk.java.net/~weijun/8168410/webrev.01/ Change since webrev.00: http://cr.openjdk.java.net/~weijun/8168410/webrev.01/interdiff.patch.html Thanks Max On 02/14/2017 11:55 PM, Sean Mullan wrote: Hi

Re: RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-15 Thread Weijun Wang
I's also like to append the following paragraph to the current release note at https://bugs.openjdk.java.net/browse/JDK-8165836: Another new system property named jdk.security.filePermCompat, when set to "true", allows the compatibility layer above to work on third-party Policy implementati

Re: RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-14 Thread Weijun Wang
Webrev updated at http://cr.openjdk.java.net/~weijun/8168410/webrev.01/ Change since webrev.00: http://cr.openjdk.java.net/~weijun/8168410/webrev.01/interdiff.patch.html Thanks Max On 02/14/2017 11:55 PM, Sean Mullan wrote: Hi Max, I agree this change is necessary so that we can resolv

Re: RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-14 Thread Sean Mullan
Hi Max, I agree this change is necessary so that we can resolve this tck-red issue before ZBB. However, since the TCK Policy provider implementation is not a "typical" implementation in the sense that it is denying permissions instead of granting permissions, I think we should continue to exp

RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-13 Thread Weijun Wang
Please take a review at cr.openjdk.java.net/~weijun/8168410/webrev.00 JDK-8164705 introduced a compatibility layer to make sure that when FilePermission on "/pwd/x" is granted you can still read "x". The compatibility layer has 2 parts: 1. Inside our own default policy provider. 2. Insi