Request for Review: 8035973: NPE in ForwardBuilder

Sean Mullan sean.mullan at oracle.com
Mon Mar 3 13:38:06 UTC 2014


On 02/28/2014 02:56 PM, Jason Uh wrote:
> Could I please get a review of this change?

Looks fine to me, but the priority should be higher if this requires a 
backport to 7u. Also, the bug should be labeled with "8-na" and "9-na" 
since this is not an issue in 8 and 9.

--Sean

>
> Just a simple fix for 7u-dev that adds a check for a null CertStore, in
> case URICertStore.getInstance returns null.
>
> webrev: http://cr.openjdk.java.net/~juh/8035973/webrev.00/
> bug: http://bugs.openjdk.java.net/browse/JDK-8035973
>
> noreg-sqe because an SQE test is already in place.
>
> Thanks,
> Jason




More information about the security-dev mailing list