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

[Sean Mullan]

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
>