RFR: 8272608: java_lang_System::allow_security_manager() doesn't set its initialization flag
Matias Saavedra Silva
github.com+38446717+matias9927 at openjdk.java.net
Fri Sep 10 00:00:02 UTC 2021
On Thu, 26 Aug 2021 16:00:01 GMT, Matias Saavedra Silva <github.com+38446717+matias9927 at openjdk.org> wrote:
> Originally the initialized flag was always false causing the System fields to be read each time the method is called. The flag is now set the first time it runs.
> Tested with tier1, specifically with runtime/logging/ProtectionDomainVerificationTest.java.
> ⚠️ @matias9927 the full name on your profile does not match the author name in this pull requests' [HEAD](https://git.openjdk.java.net/jdk/pull/5268/commits/d9127a6545e076b757f425d39826a217d487d6dd) commit. If this pull request gets integrated then the author name from this pull requests' [HEAD](https://git.openjdk.java.net/jdk/pull/5268/commits/d9127a6545e076b757f425d39826a217d487d6dd) commit will be used for the resulting commit. If you wish to push a new commit with a different author name, then please run the following commands in a local repository of your personal fork:
>
> ```
> $ git checkout JDK-8272608
> $ git commit -c user.name='Preferred Full Name' --allow-empty -m 'Update full name'
> $ git push
> ```
Should I still do this?
-------------
PR: https://git.openjdk.java.net/jdk/pull/5268
More information about the hotspot-runtime-dev
mailing list