RFR: 8297531: sun/security/krb5/MicroTime.java fails with "Exception: What? only 100 musec precision?"
Artur Barashev
abarashev at openjdk.org
Mon Mar 3 18:35:53 UTC 2025
On Mon, 3 Mar 2025 16:26:21 GMT, Weijun Wang <weijun at openjdk.org> wrote:
> Loosen the check; the modified test is sufficient to demonstrate sub-millisecond precision.
test/jdk/sun/security/krb5/MicroTime.java line 49:
> 47: // Before this change, KerberosTime was implemented in milliseconds.
> 48: // Now there should be more.
> 49: if (count < 1001) {
Why 1001 and not 1000?
test/jdk/sun/security/krb5/MicroTime.java line 50:
> 48: // Now there should be more.
> 49: if (count < 1001) {
> 50: throw new Exception("What? only " + (1000000/count) +
Shouldn't we add a zero to "1000000/count" as well to account for the measured precision change?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23867#discussion_r1977988750
PR Review Comment: https://git.openjdk.org/jdk/pull/23867#discussion_r1977988447
More information about the security-dev
mailing list