7142172: Custom TrustManagers that return null for getAcceptedIssuers will NPE

Xuelei Fan xuelei.fan at oracle.com
Fri Mar 30 19:15:53 PDT 2012


Looks fine if you only changed the testing comment.

Xuelei

On 3/31/2012 6:37 AM, Brad Wetmore wrote:
> Reposted to same location under ".01".
> 
> Brad
> 
> 
> On 3/29/2012 7:42 PM, Brad Wetmore wrote:
>> Thanks, I won't regenerate the webrev unless someone really wants to see
>> it.
>>
>> Brad
>>
>>
>> On 3/29/2012 7:30 PM, Xuelei Fan wrote:
>>> Forgot one thing about the regression test. JSSE test should run in
>>> othervm. Would you please add the following lines to the test:
>>>
>>> * SunJSSE does not support dynamic system properties, no way to
>>> * re-use system properties in samevm/agentvm mode.
>>> * @run main/othervm NullGetAcceptedIssuers
>>> *
>>>
>>> Thanks,
>>> Xuelei
>>>
>>> On 3/30/2012 10:26 AM, Xuelei Fan wrote:
>>>> Looks fine to me.
>>>>
>>>> Thanks,
>>>> Xuelei
>>>>
>>>> On 3/30/2012 10:20 AM, Brad Wetmore wrote:
>>>>> Hi Xuelei,
>>>>>
>>>>> Can you please rereview this bug, the 7u4 and 8 changes are here:
>>>>>
>>>>> http://cr.openjdk.java.net/~wetmore/7142172/
>>>>>
>>>>> Thanks,
>>>>>
>>>>> Brad
>>>>>
>>>>
>>>




More information about the security-dev mailing list