RFR: 8345286: Remove use of SecurityManager API from misc areas [v8]

Alan Bateman alanb at openjdk.org
Mon Dec 2 18:57:43 UTC 2024


On Mon, 2 Dec 2024 18:48:27 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> Jaikiran Pai has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 13 commits:
>> 
>>  - merge latest from master branch
>>  - remove changes to src/java.base/unix/classes/sun/security/provider/NativePRNG.java
>>  - remove unused import
>>  - replace remaining Paths.get() with Path.of() in the updated files
>>  - Update src/java.base/linux/classes/jdk/internal/platform/CgroupSubsystemController.java
>>    
>>    Co-authored-by: Severin Gehwolf <jerboaa at gmail.com>
>>  - Update src/java.base/linux/classes/jdk/internal/platform/cgroupv2/CgroupV2Subsystem.java
>>    
>>    Co-authored-by: Severin Gehwolf <jerboaa at gmail.com>
>>  - Path.of() instead of Paths.get() in CgroupV2Subsystem.java
>>  - remove unnecessary space
>>  - Path.of() instead of Paths.get()
>>  - fix formatting of try-with-resources in CgroupSubsystemController.java
>>  - ... and 3 more: https://git.openjdk.org/jdk/compare/30b8bbe2...9baa279b
>
> src/java.base/share/classes/jdk/internal/misc/InnocuousThread.java line 31:
> 
>> 29: 
>> 30: /**
>> 31:  * A thread that has no permissions, is not a member of any user-defined
> 
> I think you can also remove the words "has no permissions" as it no longer applies. @AlanBateman ?

Yes, that can be removed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22478#discussion_r1866425533


More information about the serviceability-dev mailing list