Request for Review: 8035973: NPE in ForwardBuilder

Jason Uh jason.uh at oracle.com
Mon Mar 3 11:46:32 PST 2014


On 3/3/14 5:38 AM, Sean Mullan wrote:
> 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.

Thanks. Bumped priority up to P3 and added the labels.

> --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