Page MenuHomeFreeBSD

ping tests: Fix test on 32-bit platforms
ClosedPublic

Authored by jlduran_gmail.com on Oct 17 2023, 4:26 PM.
Tags
None
Referenced Files
F96283715: D42268.id128939.diff
Tue, Sep 24, 8:37 AM
Unknown Object (File)
Mon, Sep 16, 9:28 PM
Unknown Object (File)
Mon, Sep 9, 7:37 PM
Unknown Object (File)
Mon, Sep 9, 5:36 PM
Unknown Object (File)
Aug 6 2024, 8:26 AM
Unknown Object (File)
Jul 8 2024, 11:53 PM
Unknown Object (File)
Jul 4 2024, 7:00 AM
Unknown Object (File)
Jun 23 2024, 8:58 AM

Details

Summary
Actually use INT_MAX (0x7fffffff), not UINT_MAX (0xffffffff) to avoid
overflowing and missing the test.

Reported by:    Jenkins
Test Plan

The idea is to hit the test, not to overflow 32-bit architecture's time_t.
I do not have a 32-bit VM at the moment, would it be possible to run the test on ci.freebsd.org, and verify that test:
https://ci.freebsd.org/view/Test/job/FreeBSD-main-i386-test/17993/testReport/junit/sbin.ping.test_ping/py/TestPing__test_pinger__0_0_special_warp_/
Passes on i386?:

/usr/tests# kyua debug sbin/ping/test_ping.py:TestPing::test_pinger[_0_0_special_warp]

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I do not have a 32-bit VM at the moment, would it be possible to run the test on ci.freebsd.org, and verify that test:

Unfortunately there's no easy way to manually test patches using ci.freebsd.org yet. In the meantime I downloaded an i386 VM image and verified that the test passes.

This revision is now accepted and ready to land.Oct 18 2023, 1:22 PM

I do not have a 32-bit VM at the moment, would it be possible to run the test on ci.freebsd.org, and verify that test:

Unfortunately there's no easy way to manually test patches using ci.freebsd.org yet. In the meantime I downloaded an i386 VM image and verified that the test passes.

Thank you!

Please submit a PR with the patch, or mail git format-patch output to me directly.

This revision was automatically updated to reflect the committed changes.