RFR: 8354053: Remove unused JavaIOFilePermissionAccess
Chen Liang
liach at openjdk.org
Sun Apr 13 10:10:32 UTC 2025
On Fri, 11 Apr 2025 21:26:08 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
> The JavaIOFilePermissionAccess interface is removed from SharedSecrets and its implementation (FilePermCompat.java) used by the test is moved to java.io FilePermission where cross package access is not needed.
> The test FilePermissionCollectionMerge is updated to access the internal implementation in FilePermission.
> Modernized the initialization from the system property `jdk.io.permissionsUseCanonicalPath`.
> The remaining support will be removed when FilePermission is removed.
src/java.base/share/classes/java/io/FilePermission.java line 254:
> 252:
> 253: // Construct a new Permission with altPath
> 254: // Package private for use by test FilePermissionCollectionMerge
That test is already calling with reflection and +open, we can just make this private and non-static.
The last use of the original methods were removed when AccessControlContext was functionally removed. If security developers can check, maybe we can just remove these methods completely?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24603#discussion_r2041089794
More information about the security-dev
mailing list