jdk10 RFR of JDK-8180732: add test to check temp file permission

Brent Christian brent.christian at oracle.com
Tue May 23 16:53:26 UTC 2017


Hi, Hamlin

That looks pretty good.  A couple comments:

* If I'm not mistaken, main/othervm/policy=... is sufficient to enable 
the default security manager, and specifying "-Djava.security.manager" 
is unnecessary.

* Please add 8180732 to the @bug tag

You will also need approval from a JDK 10 Reviewer.

Thanks,
-Brent

On 5/21/17 9:26 PM, Hamlin Li wrote:
> Would you please review the below patch?
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8180732
> webrev: http://cr.openjdk.java.net/~mli/8180732/webrev.00/
>
> Thank you
> -Hamlin


More information about the core-libs-dev mailing list