[PATCH] tst-socket-timestamp-compat.c: Check __TIMESIZE [BZ #28837]
Adhemerval Zanella
adhemerval.zanella@linaro.org
Sat Jan 29 14:46:07 GMT 2022
On 29/01/2022 10:29, H.J. Lu wrote:
> time_t size is defined by __TIMESIZE, not __WORDSIZE. Check __TIMESIZE,
> instead of __WORDSIZE, for time_t size. This fixes BZ #28837.
LGTM, thanks for catching it.
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
> ---
> sysdeps/unix/sysv/linux/tst-socket-timestamp-compat.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/sysdeps/unix/sysv/linux/tst-socket-timestamp-compat.c b/sysdeps/unix/sysv/linux/tst-socket-timestamp-compat.c
> index de261dae5a..0ff1a214e6 100644
> --- a/sysdeps/unix/sysv/linux/tst-socket-timestamp-compat.c
> +++ b/sysdeps/unix/sysv/linux/tst-socket-timestamp-compat.c
> @@ -237,7 +237,7 @@ do_test (void)
> {
> /* This test only make sense for ABIs that support 32 bit time_t socket
> timestampss. */
> - if (sizeof (time_t) > 4 && __WORDSIZE == 64)
> + if (sizeof (time_t) > 4 && __TIMESIZE == 64)
> return 0;
>
> srv = xsocket (AF_INET, SOCK_DGRAM, 0);
More information about the Libc-alpha
mailing list