| Message ID | 20220129132932.10564-1-hjl.tools@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | tst-socket-timestamp-compat.c: Check __TIMESIZE [BZ #28837] | expand |
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);
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);