JDK-8342997 : Remove use of System::getSecurityManager and SecurityManager from JavaFX
  • Type: Bug
  • Component: javafx
  • Sub-Component: other
  • Affected Version: jfx24
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2024-10-24
  • Updated: 2024-11-16
  • Resolved: 2024-11-14
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
jfx24 masterFixed
Related Reports
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Description
Remove all other calls to System::getSecurityManager (45 calls in 27 files) and all other uses of SecurityManager (2 instances in 2 of those files) other than the ones in the com.sun.javafx.reflect package (those are part of a larger change tracked by JDK-8342994).

As part of doing this, remove all code paths associated with a non-null security manager.

Comments
Changeset: dd600658 Branch: master Author: Kevin Rushforth <kcr@openjdk.org> Date: 2024-11-14 20:15:36 +0000 URL: https://git.openjdk.org/jfx/commit/dd60065808bc847d8efbb818b8f83c4e61290d29
14-11-2024

OK, I'll change it before I integrate.
14-11-2024

Thanks, understood. However, my concern was mostly about the JBS, where this issue lives in the JDK- namespace.
14-11-2024

This one is for JavaFX, which is in a different repo from the JDK. You won't see any PRs for this in the jdk repo (or any JDK mailing list).
14-11-2024

I think the title of this issue should be updated to reflect the scope of this change. There are a lot of similar changes coming in different areas.
14-11-2024

A pull request was submitted for review. Branch: master URL: https://git.openjdk.org/jfx/pull/1631 Date: 2024-11-08 21:46:38 +0000
08-11-2024

As noted in the main SM removal issue, JDK-8341090, one of the follow-up items is: * Remove calls to deprecated SM methods from PlatformUtil and MethodUtil in javafx.base (these are lightly modified copies of those classes in java.base so I will check how they are being handled) I will either do that as part of this fix or file a separate bug for those two classes.
05-11-2024