RFR 8222275: sun/security/tools/keytool/Serial64.java: assertTrue: expected true, was false
Weijun Wang
weijun.wang at oracle.com
Sun Apr 14 02:17:04 UTC 2019
> On Apr 14, 2019, at 1:54 AM, Xuelei Fan <Xuelei.Fan at Oracle.Com> wrote:
>
>
>
>> On Apr 13, 2019, at 7:35 AM, Weijun Wang <weijun.wang at oracle.com> wrote:
>>
>>
>>
>>> On Apr 12, 2019, at 10:59 PM, Xuelei Fan <xuelei.fan at oracle.com> wrote:
>>>
>>> To test a 3% or 10% chance of 64-bits serial number,
>>
>> Can you explain what 3% or 10% you mean here?
>>
> I think the code is checking one 64 bits SN out of 10 or 30 SNs?
Yes, it's similar to before. I just check while generating so I can succeed early.
I think what you mean if no matter if the probability is high or low, it's still random and could fail. But this time the difference is not 3 folds it's 1000000. So I think it's worth a rewrite.
I'll push it.
--Max
>
> Xuelei
>
>>> I don't think the test really make sense now. I would prefer to drop this test, or just keep the positive value checking (one cert only).
>>
>> ...if it fails again. :-)
>>
>> Thanks,
>> Max
>>
>>>
>>> If you want to keep it, the code looks good to me.
>>>
>>> Xuelei
>>>
>>>> On 4/12/2019 12:59 AM, Weijun Wang wrote:
>>>> Please take a review at
>>>> https://cr.openjdk.java.net/~weijun/8222275/webrev.00/
>>>> Now the test would generate 10 to 30 certs. It could still fail, but the probably is extremely small now (2^-30 is 10^-9).
>>>> And the test would succeed early if a 64 bit SN is found.
>>>> Thanks,
>>>> Max
>>
>
More information about the security-dev
mailing list