RFR 8165753: Custom security manager, crashing wth 'java/security/AccessControlException'

[David Holmes]

Looks good to me Vladimir! Thanks for fixing.

David H.

On 13/02/2017 9:06 PM, Vladimir Kempik wrote:
> Hello
>
> Please review the fix for 8165753: Custom security manager, crashing wth
> 'java/security/AccessControlException'
>
> Initially this bug came for jdk8, and it has reproducer for jdk8.
>
> However this reproducer doesn't crash jdk9/10, but still needed there,
> see explanation of this from David Holmes:
>
> https://bugs.openjdk.java.net/browse/JDK-8165753?focusedCommentId=14001767&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14001767
>
>
> The fix changes behaviour from a crash to a nice exception on jdk8, but
> I need to get the fix into jdk10/9 first (jdk9/10 shows no visible
> difference with testcase before/after patch)
>
> Error occurred during initialization of VM
> java.security.AccessControlException: access denied
> ("java.lang.RuntimePermission" "modifyThreadGroup")
>     at
> java.security.AccessControlContext.checkPermission(AccessControlContext.java:472)
>
>     at
> java.security.AccessController.checkPermission(AccessController.java:884)
>     at java.lang.SecurityManager.checkPermission(SecurityManager.java:549)
>     at foo.TestSecurityManager.checkPermission(TestSecurityManager.java:48)
>     at java.lang.SecurityManager.checkAccess(SecurityManager.java:729)
>     at java.lang.ThreadGroup.checkAccess(ThreadGroup.java:315)
>     at java.lang.Thread.init(Thread.java:391)
>     at java.lang.Thread.init(Thread.java:349)
>     at java.lang.Thread.<init>(Thread.java:532)
>
>
> The bug: https://bugs.openjdk.java.net/browse/JDK-8165753
>
> The Webrev: http://cr.openjdk.java.net/~vkempik/8165753/webrev.00/
>
> Testing: jprt testset hotspot
>
> Thanks, Vladimir
>