Re: RFR (13) [testbug]: 8227422: sun/net/www/protocol/file/DirPermissionDenied.java failed on Windows 2016 because DirPermissionDenied directory has no read permission

2019-07-11 Thread Daniel Fuchs
On 11/07/2019 06:03, Frank Yuan wrote: Hi Daniel I have filed another bughttps://bugs.openjdk.java.net/browse/JDK-8227566 to refine the bug. Thanks Frank! And, I tested on Windows 7, the expected exception is raised... I will test on more platform. Good to know - in our test system th

RFR: 8227587: Add internal privileged System.loadLibrary

2019-07-11 Thread Claes Redestad
Hi, by adding a method to load libraries with privileges to JavaLangAccess, we can simplify a slew of places where we are currently implementing adhoc privileged actions. This is a tiny but measurable gain on a range of startup tests. Webrev: http://cr.openjdk.java.net/~redestad/8227587/open.00

Re: RFR: 8227587: Add internal privileged System.loadLibrary

2019-07-11 Thread Roger Riggs
Hi Claes, JavaLangAccess.java: 316: Add @param tag System.java:  2282, 2287 Runtime.loadLibrary0 makes a second check for a security manager. Is there any potential gain by calling ClassLoader.loadLibrary directly? None of the internal uses would have a separatorChar. I expect most of the files

Re: RFR: 8227587: Add internal privileged System.loadLibrary

2019-07-11 Thread Claes Redestad
Hi Roger, On 2019-07-11 16:10, Roger Riggs wrote: Hi Claes, JavaLangAccess.java: 316: Add @param tag done. System.java:  2282, 2287 Runtime.loadLibrary0 makes a second check for a security manager. Is there any potential gain by calling ClassLoader.loadLibrary directly? None of the interna

RE: RFR (13) [testbug]: 8227422: sun/net/www/protocol/file/DirPermissionDenied.java failed on Windows 2016 because DirPermissionDenied directory has no read permission

2019-07-11 Thread Frank Yuan
> > > On 11/07/2019 06:03, Frank Yuan wrote: > > Hi Daniel > > > > I have filed another bughttps://bugs.openjdk.java.net/browse/JDK-8227566 > > to refine the bug. > > Thanks Frank! > > > And, I tested on Windows 7, the expected exception is raised... I will test > > on more platform. > > Go