[8u-dev] Request for review & approval: 8036630 & 8085903

[Seán Coffey]

Approved for jdk8u-dev.

Regards,
Sean.

On 04/05/16 17:55, Sean Mullan wrote:
> On 05/04/2016 09:37 AM, Rob McKenna wrote:
>> Hi folks,
>>
>> Looking for a review/approval for:
>>
>> 8085903: New fix for memory leak in ProtectionDomain cache
>> bug: https://bugs.openjdk.java.net/browse/JDK-8085903
>> 9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/41491b4e93d1
>> 9 review: 
>> http://mail.openjdk.java.net/pipermail/security-dev/2016-January/013295.html
>> 8 webrev: http://cr.openjdk.java.net/~robm/8085903/webrev.01/
>
> This looks fine to me.
>
> --Sean
>
>>
>> and approval only for:
>>
>> 8036630: Null ProtectionDomain in JVM can cause NPE because 
>> principals field is not initialized to an empty array
>> bug: https://bugs.openjdk.java.net/browse/JDK-8036630
>> changeset: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/26ddfd00fe60
>> review: 
>> http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2014-March/011143.html
>>
>> I'm putting them together to provide some context for the backport of 
>> 8036630.
>>
>>     -Rob
>>