[jdk17] RFR: 8268349: Provide clear run-time warnings about Security Manager deprecation [v3]

Weijun Wang weijun at openjdk.java.net
Thu Jun 17 14:30:28 UTC 2021


On Thu, 17 Jun 2021 14:05:40 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> Weijun Wang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   new warning text again (6/14)
>
> test/jdk/java/security/ProtectionDomain/RecursionDebug.java line 91:
> 
>> 89:         }
>> 90: 
>> 91:         System.setSecurityManager(null);
> 
> Why did this line need to be removed?

This is where the `setSecurityManager` method is called again when a security manager has already been installed, and the `getProtectionDomain` permission is needed. While I can add the new permission into the policy file, my understanding is that this line was just a clean-up and the test was about implementation of `ProtectionDomain::toString` (esp the `seeAllp` method called by it) when debug is on.

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

PR: https://git.openjdk.java.net/jdk17/pull/13



More information about the security-dev mailing list