pgsql: Fix our Windows stat() emulation to handle file sizes > 4GB.

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Fix our Windows stat() emulation to handle file sizes > 4GB.
Date: 2024-11-07 23:19:39
Message-ID: E1t9Bms-000hQQ-Np@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix our Windows stat() emulation to handle file sizes > 4GB.

Hack things so that our idea of "struct stat" is equivalent to Windows'
struct __stat64, allowing it to have a wide enough st_size field.

Instead of relying on native stat(), use GetFileInformationByHandle().
This avoids a number of issues with Microsoft's multiple and rather
slipshod emulations of stat(). We still need to jump through hoops
to deal with ERROR_DELETE_PENDING, though :-(

Pull the relevant support code out of dirmod.c and put it into
its own file, win32stat.c.

Still TODO: do we need to do something different with lstat(),
rather than treating it identically to stat()?

Juan José Santamaría Flecha, reviewed by Emil Iggland;
based on prior work by Michael Paquier, Sergey Zubkovsky, and others

Discussion: https://postgr.es/m/1803D792815FC24D871C00D17AE95905CF5099@g01jpexmbkw24
Discussion: https://postgr.es/m/15858-9572469fd3b73263@postgresql.org
(cherry picked from commit bed90759fcbcd72d4d06969eebab81e47326f9a2)

Author: Alexandra Wang <alexandra(dot)wang(dot)oss(at)gmail(dot)com>

Branch
------
REL_13_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/a9beed67670e680edeadd2a3cf7557a3c9808adf
Author: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>

Modified Files
--------------
configure | 6 +
configure.in | 1 +
src/include/port/win32_port.h | 44 +++++--
src/port/dirmod.c | 52 --------
src/port/win32stat.c | 299 ++++++++++++++++++++++++++++++++++++++++++
src/tools/msvc/Mkvcbuild.pm | 2 +-
6 files changed, 339 insertions(+), 65 deletions(-)

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Richard Guo 2024-11-08 02:25:45 pgsql: Fix inconsistent RestrictInfo serial numbers
Previous Message Andrew Dunstan 2024-11-07 23:19:38 pgsql: Provide lstat() for Windows.