-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ping tests: Fix test on i386 platforms #874
Conversation
Technically the problem isn't with 32-bit platforms, just i386. 32-bit arm has a 64-bit |
Ahhh. Let me change the commit message then. As we only care for i386 and armv7. |
Actually use INT_MAX (0x7fffffff), not UINT_MAX (0xffffffff) to avoid overflowing time_t and missing the test. Reported by: Jenkins Reviewed by: markj Differential Revision: https://reviews.freebsd.org/D42268
3dd3a0d
to
f833255
Compare
Merged! |
Actually use INT_MAX (0x7fffffff), not UINT_MAX (0xffffffff) to avoid overflowing time_t and missing the test. Reported by: Jenkins Reviewed by: markj Fixes: 5b8af90 ("ping: Add ATF-Python tests") Pull Request: #874 Differential Revision: https://reviews.freebsd.org/D42268
May I kindly request an MFC to stable/14: |
Fixed, thank you. |
Actually use INT_MAX (0x7fffffff), not UINT_MAX (0xffffffff) to avoid overflowing time_t and missing the test. Reported by: Jenkins Reviewed by: markj Fixes: 5b8af90 ("ping: Add ATF-Python tests") Pull Request: #874 Differential Revision: https://reviews.freebsd.org/D42268 (cherry picked from commit 24fe0cb)
Actually use INT_MAX (0x7fffffff), not UINT_MAX (0xffffffff) to avoid overflowing time_t and missing the test. Reported by: Jenkins Reviewed by: markj Fixes: 5b8af90 ("ping: Add ATF-Python tests") Pull Request: freebsd/freebsd-src#874 Differential Revision: https://reviews.freebsd.org/D42268
Actually use
INT_MAX
(0x7fffffff
), notUINT_MAX
(0xffffffff
) to avoid overflowingtime_t
and missing the test.Reported by: Jenkins
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D42268
cc/ @markjdb