Don't leak descriptors into subprograms.
authorThomas Munro <tmunro@postgresql.org>
Thu, 2 Mar 2023 21:28:47 +0000 (10:28 +1300)
committerThomas Munro <tmunro@postgresql.org>
Thu, 2 Mar 2023 21:43:33 +0000 (10:43 +1300)
commit1da569ca1f1fd08ae728ccde0952b688feff7d9c
tree28212659a1fd9f9dc34e02012ed661534e5ce30c
parent6b661b01f48bb0d3129ad0e3909210a6ba0534b3
Don't leak descriptors into subprograms.

Open long-lived data and WAL file descriptors with O_CLOEXEC.  This flag
was introduced by SUSv4 (POSIX.1-2008), and by now all of our target
Unix systems have it.  Our open() implementation for Windows already had
that behavior, so provide a dummy O_CLOEXEC flag on that platform.

For now, callers of open() and the "thin" wrappers in fd.c that deal in
raw descriptors need to pass in O_CLOEXEC explicitly if desired.  This
commit does that for WAL files, and automatically for everything
accessed via VFDs including SMgrRelation and BufFile.  (With more
discussion we might decide to turn it on automatically for the thin
open()-wrappers too to avoid risk of missing places that need it, but
these are typically used for short-lived descriptors where we don't
expect to fork/exec, and it's remotely possible that extensions could be
using these APIs and passing descriptors to subprograms deliberately, so
that hasn't been done here.)

Do the same for sockets and the postmaster pipe with FD_CLOEXEC.  (Later
commits might use modern interfaces to remove these extra fcntl() calls
and more where possible, but we'll need them as a fallback for a couple
of systems, so do it that way in this initial commit.)

With this change, subprograms executed for archiving, copying etc will
no longer have access to the server's descriptors, other than the ones
that we decide to pass down.

Reviewed-by: Andres Freund <andres@anarazel.de> (earlier version)
Discussion: https://postgr.es/m/CA%2BhUKGKb6FsAdQWcRL35KJsftv%2B9zXqQbzwkfRf1i0J2e57%2BhQ%40mail.gmail.com
src/backend/access/transam/xlog.c
src/backend/libpq/pqcomm.c
src/backend/storage/file/fd.c
src/backend/utils/init/miscinit.c
src/include/port/win32_port.h