Request for Review: 8035973: NPE in ForwardBuilder

Xuelei Fan xuelei.fan at oracle.com
Fri Feb 28 23:50:41 UTC 2014


On 3/1/2014 7:38 AM, Jason Uh wrote:
> Do I need more than one noreg tag? I already have noreg-sqe.
> 
noreg-sqe is OK.

Thanks,
Xuelei

> On 2/28/14 3:31 PM, Xuelei Fan wrote:
>> Looks fine to me.  noreg-trivial tag?
>>
>> Xuelei
>>
>> On 3/1/2014 3:56 AM, Jason Uh wrote:
>>> Could I please get a review of this change?
>>>
>>> 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