Use FD_CLOEXEC on ListenSockets (was Re: Refactoring backend fork+exec code)

From: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>
To: Andres Freund <andres(at)anarazel(dot)de>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Tristan Partin <tristan(at)neon(dot)tech>
Subject: Use FD_CLOEXEC on ListenSockets (was Re: Refactoring backend fork+exec code)
Date: 2023-08-24 11:41:44
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

Focusing on this one patch in this series:

On 11/07/2023 01:50, Andres Freund wrote:
>> From: Heikki Linnakangas <heikki(dot)linnakangas(at)iki(dot)fi>
>> Date: Mon, 12 Jun 2023 16:33:20 +0300
>> Subject: [PATCH 4/9] Use FD_CLOEXEC on ListenSockets
>> We went through some effort to close them in the child process. Better to
>> not hand them down to the child process in the first place.
> I think Thomas has a larger version of this patch:

Hmm, no, that's a little different. Thomas added the FD_CLOEXEC option
to the *accepted* socket in commit 1da569ca1f. That was part of that
thread. This patch adds the option to the *listen* sockets. That was not
discussed in that thread, but it's certainly in the same vein.

Thomas: What do you think of the attached?

On 11/07/2023 00:07, Tristan Partin wrote:
>> @@ -831,7 +834,8 @@ StreamConnection(pgsocket server_fd, Port *port)
>> void
>> StreamClose(pgsocket sock)
>> {
>> - closesocket(sock);
>> + if (closesocket(sock) != 0)
>> + elog(LOG, "closesocket failed: %m");
>> }
>> /*
> Do you think WARNING would be a more appropriate log level?

No, WARNING is for messages that you expect the client to receive. This
failure is unexpected at the system level, the message is for the
administrator. The distinction isn't always very clear, but LOG seems
more appropriate in this case.

Heikki Linnakangas
Neon (

Attachment Content-Type Size
v2-0001-Use-FD_CLOEXEC-on-ListenSockets.patch text/x-patch 3.5 KB

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Dagfinn Ilmari Mannsåker 2023-08-24 12:38:59 Re: Adding argument names to aggregate functions
Previous Message Quan Zongliang 2023-08-24 10:55:28 Improving the heapgetpage function improves performance in common scenarios