On Tue, 5 Sep 2023 11:56:19 GMT, Per Minborg <pminb...@openjdk.org> wrote:

>> This PR proposes to read the system environment variable "SystemRoot" using 
>> a privileged operation so it will work in the event a default 
>> SecurityManager is in place.
>> 
>> As the `SecurityManager` is deprecated for removal, no support methods were 
>> added for reading environmental variables.
>
> Per Minborg has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Add extra test run with no SecutityManager

test/jdk/java/foreign/security.policy line 1:

> 1: grant {

To avoid inadvertently granting these permissions to codebases other than the 
tests, you should change the grant clause to `grant codebase 
"file:${test.classes}/" {`

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/15564#discussion_r1316053185

Reply via email to