RFR: 8342992: Security manager check should not use deprecated methods [v2]
Kevin Rushforth
kcr at openjdk.org
Thu Nov 7 13:17:33 UTC 2024
> As specified in the security manager removal CSR, [JDK-8341858](https://bugs.openjdk.org/browse/JDK-8341858), JavaFX now checks at startup whether the security manager is enabled and fails fast with an `UnsupportedOperationException` if it is. The check is currently done in the `<clinit>` methods of `PlatformImpl` and `LauncherImpl` by calling the deprecated `System::getSecurityManager` method.
>
> This PR creates a new `SecurityUtility::checkSecurityManager` utility method in `javafx.base` that uses reflection to avoid calling API that is deprecated for removal. I also added a call to `checkSecurityManager` in `ReflectUtil` and `MethodUtil` in `javafx.base` for non-graphical applications that only use `javafx.base`.
Kevin Rushforth has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:
- Merge branch 'master' into 8342992-check-sm
- 8342992: Security manager check should not use deprecated methods
-------------
Changes:
- all: https://git.openjdk.org/jfx/pull/1627/files
- new: https://git.openjdk.org/jfx/pull/1627/files/6e905c17..7ea58b33
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jfx&pr=1627&range=01
- incr: https://webrevs.openjdk.org/?repo=jfx&pr=1627&range=00-01
Stats: 905 lines in 40 files changed: 45 ins; 525 del; 335 mod
Patch: https://git.openjdk.org/jfx/pull/1627.diff
Fetch: git fetch https://git.openjdk.org/jfx.git pull/1627/head:pull/1627
PR: https://git.openjdk.org/jfx/pull/1627
More information about the openjfx-dev
mailing list