RFR: 8210469: Missing doPriviledged block and permission for jdk.zipfs module

Mandy Chung mandy.chung at oracle.com
Thu Jan 24 21:50:57 UTC 2019



On 1/24/19 12:48 PM, Lance Andersen wrote:
> Hi all,
>
> Please review the following patch which addresses https://bugs.openjdk.java.net/browse/JDK-8210469 where jdk.zipfs was missing a doPriviledged block and permission  needed when closing a zip file system when a security manager has been enabled.
>
> The webrev can be found at: http://cr.openjdk.java.net/~lancea/8210469/webrev.00/index.html

Alternatively you can simply do:
             PrivilegedExceptionAction<Path> pea = () -> 
tempPath.toRealPath();
             return AccessController.doPrivileged(pea);

In this case, limited doPrivileged does not make whole lot of difference 
jdk.zip explicitly grants for limited permissions and this doPrivileged 
block is one single operation, toRealPath.

The new test has a couple of blank lines (line 64 an 83). Formatting 
nit: indentation in line 74.

Mandy



More information about the core-libs-dev mailing list