[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