<AWT Dev> [9] Review Request: 8136858 Examine the usage of ThreadGroup.stop() in sun.awt.AppContext

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Mon Oct 12 12:57:38 UTC 2015


   The fix looks good to me.

   Thanks,
   Alexandr.

On 10/6/2015 3:50 PM, Chris Hegarty wrote:
> Thanks Sergey, this looks good to me.
>
> -Chris.
>
> On 6 Oct 2015, at 13:42, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
>
>> Hello.
>> Please review the fix for jdk9.
>>
>> In the fix the call to ThreadGroup.stop() was wrapped in privileged block. The new test is provided, I have checked that it will fail if default permission "stopThread" will be removed from java.policy or if the call to ThreadGroup.stop() will be removed from AppContext.java
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8136858
>> Webrev can be found at: http://cr.openjdk.java.net/~serb/8136858/webrev.02
>>
>> -- 
>> Best regards, Sergey.



More information about the awt-dev mailing list