public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
* [PATCH 0/1] y2038: Fix support for 64-bit time on legacy ABIs
@ 2023-11-11 17:32 Gaël PORTAY
  2023-11-11 17:32 ` [PATCH 1/1] " Gaël PORTAY
  0 siblings, 1 reply; 3+ messages in thread
From: Gaël PORTAY @ 2023-11-11 17:32 UTC (permalink / raw)
  To: libc-alpha; +Cc: Adhemerval Zanella, Gaël PORTAY

Dear Maintainers,

I was looking at the header sys/stat.h, and I inadvertently found out
the suffix 64 appears to be missing to the function __fstat64_time() at
the definition of the macro fstat64 (if __REDIRECT_NTH is undefined).

I am afraid (and sorry at the same time) but I have not tested this
change as I do not know how to test it to be honest (probably using a
specific architecture like ARM or x86, but how to get in the else part
of __REDIRECT_NTH?).

Kind Regards,
Gaël PORTAY (1):
  y2038: Fix support for 64-bit time on legacy ABIs

 io/sys/stat.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.42.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 1/1] y2038: Fix support for 64-bit time on legacy ABIs
  2023-11-11 17:32 [PATCH 0/1] y2038: Fix support for 64-bit time on legacy ABIs Gaël PORTAY
@ 2023-11-11 17:32 ` Gaël PORTAY
  2023-11-13 14:41   ` Adhemerval Zanella Netto
  0 siblings, 1 reply; 3+ messages in thread
From: Gaël PORTAY @ 2023-11-11 17:32 UTC (permalink / raw)
  To: libc-alpha; +Cc: Adhemerval Zanella, Gaël PORTAY

This fixes a typo.
---
 io/sys/stat.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/io/sys/stat.h b/io/sys/stat.h
index 7b9368f9ec..eb280c1f06 100644
--- a/io/sys/stat.h
+++ b/io/sys/stat.h
@@ -251,7 +251,7 @@ extern int __REDIRECT_NTH (fstat64, (int __fd, struct stat64 *__buf),
      __nonnull ((2));
 #  else
 #   define stat64 __stat64_time64
-#   define fstat64 __fstat64_time
+#   define fstat64 __fstat64_time64
 #  endif
 # endif
 #endif
-- 
2.42.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] y2038: Fix support for 64-bit time on legacy ABIs
  2023-11-11 17:32 ` [PATCH 1/1] " Gaël PORTAY
@ 2023-11-13 14:41   ` Adhemerval Zanella Netto
  0 siblings, 0 replies; 3+ messages in thread
From: Adhemerval Zanella Netto @ 2023-11-13 14:41 UTC (permalink / raw)
  To: Gaël PORTAY, libc-alpha



On 11/11/23 14:32, Gaël PORTAY wrote:
> This fixes a typo.

LGTM, thanks.

Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org>

> ---
>  io/sys/stat.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/io/sys/stat.h b/io/sys/stat.h
> index 7b9368f9ec..eb280c1f06 100644
> --- a/io/sys/stat.h
> +++ b/io/sys/stat.h
> @@ -251,7 +251,7 @@ extern int __REDIRECT_NTH (fstat64, (int __fd, struct stat64 *__buf),
>       __nonnull ((2));
>  #  else
>  #   define stat64 __stat64_time64
> -#   define fstat64 __fstat64_time
> +#   define fstat64 __fstat64_time64
>  #  endif
>  # endif
>  #endif

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-11-13 14:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-11-11 17:32 [PATCH 0/1] y2038: Fix support for 64-bit time on legacy ABIs Gaël PORTAY
2023-11-11 17:32 ` [PATCH 1/1] " Gaël PORTAY
2023-11-13 14:41   ` Adhemerval Zanella Netto

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).