pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Nathan Bossart <nathan@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Fix Y2038 issues with MyStartTime.
Date: Mon, 07 Oct 2024 18:52:54 +0000
Message-ID: <E1sxsqk-002aMH-JG@gemulon.postgresql.org> (raw)

Fix Y2038 issues with MyStartTime.

Several places treat MyStartTime as a "long", which is only 32 bits
wide on some platforms.  In reality, MyStartTime is a pg_time_t,
i.e., a signed 64-bit integer.  This will lead to interesting bugs
on the aforementioned systems in 2038 when signed 32-bit integers
are no longer sufficient to store Unix time (e.g., "pg_ctl start"
hanging).  To fix, ensure that MyStartTime is handled as a 64-bit
value everywhere.  (Of course, users will need to ensure that
time_t is 64 bits wide on their system, too.)

Co-authored-by: Max Johnson
Discussion: https://postgr.es/m/CO1PR07MB905262E8AC270FAAACED66008D682%40CO1PR07MB9052.namprd07.prod.outlook.com
Backpatch-through: 12

Branch
------
REL_16_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/8aaf88b63de3d6e665b0fcf1ff123f08b961fe66

Modified Files
--------------
contrib/postgres_fdw/option.c     | 2 +-
src/backend/utils/error/csvlog.c  | 2 +-
src/backend/utils/error/elog.c    | 6 +++---
src/backend/utils/error/jsonlog.c | 4 ++--
src/backend/utils/init/miscinit.c | 4 ++--
src/bin/pg_ctl/pg_ctl.c           | 2 +-
6 files changed, 10 insertions(+), 10 deletions(-)



view thread (7+ messages)  latest in thread

Message-ID: <E1sxsqk-002aMH-JG@gemulon.postgresql.org>
Permalink:  ../E1sxsqk-002aMH-JG@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1sxsqk-002aMH-JG@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: nathan@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Fix Y2038 issues with MyStartTime.
  In-Reply-To: <E1sxsqk-002aMH-JG@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox