RFR: 8267485: Remove the dependency on SecurityManager in JceSecurityManager.java [v3]

Daniel Fuchs dfuchs at openjdk.java.net
Thu Jun 3 18:01:38 UTC 2021


On Thu, 3 Jun 2021 17:44:42 GMT, Bradford Wetmore <wetmore at openjdk.org> wrote:

>> src/java.base/share/classes/javax/crypto/JceSecurityManager.java line 50:
>> 
>>> 48:  * @since 1.4
>>> 49:  */
>>> 50: @SuppressWarnings("removal")
>> 
>> You should remove this annotation now that the dependency on SecurityManager has been removed.
>
> Unfortunately, we are still calling AccessController, thus the annotation needs to remain.

But if you follow my suggestion you can simply apply it to this line:


@SuppressWarnings("removal")
final List<StackFrame> stack = AccessController.doPrivileged(pa).walk(Stream::toList);

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

PR: https://git.openjdk.java.net/jdk/pull/4150



More information about the security-dev mailing list