Try again to work around Windows' ERROR_SHARING_VIOLATION in pg_ctl.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 16 Feb 2020 17:20:18 +0000 (12:20 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 16 Feb 2020 17:20:18 +0000 (12:20 -0500)
commite02ea141ee66e5b2372e1eb2d2000b7ed3a8e13a
tree9a5b322e0366f5af0cfeb5292c5c68880f181f28
parentfaade5d4c6d8b4f7d46f91702a57529c96aee86e
Try again to work around Windows' ERROR_SHARING_VIOLATION in pg_ctl.

Commit 0da33c762 introduced an unfortunate regression in pg_ctl on
Windows: if the log file specified with -l doesn't exist yet, and
pg_ctl is running with Administrator privileges, then the log file
might get created with permissions that prevent the postmaster from
writing on it.  (It seems that whether this happens depends on whether
the log file is inside the user's home directory or not, and perhaps
on other phase-of-the-moon conditions, which may explain why we failed
to notice it sooner.)

To fix, just don't create the log file if it doesn't exist yet.  The
case where we need to wait obviously only occurs with a pre-existing
log file.

In passing, switch from using fopen() to plain open(), saving a few
cycles.

Per bug #16259 from Jonathan Katz and Heath Lord.  Back-patch to v12,
as the faulty commit was.

Alexander Lakhin

Discussion: https://postgr.es/m/16259-c5ebed32a262a8b1@postgresql.org
src/bin/pg_ctl/pg_ctl.c