| From: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com> |
|---|---|
| To: | "Gregory Stark (as CFM)" <stark(dot)cfm(at)gmail(dot)com> |
| Cc: | Andres Freund <andres(at)anarazel(dot)de>, pgsql-hackers(at)lists(dot)postgresql(dot)org, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: File descriptors in exec'd subprocesses |
| Date: | 2023-03-02 22:04:21 |
| Message-ID: | CA+hUKGKPNFcfBQduqof4-7C=avjcSfdkKBGvQoRuAvfocnvY0A@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Mar 2, 2023 at 9:57 AM Thomas Munro <thomas(dot)munro(at)gmail(dot)com> wrote:
> On Thu, Mar 2, 2023 at 9:49 AM Gregory Stark (as CFM)
> <stark(dot)cfm(at)gmail(dot)com> wrote:
> > On Mon, 20 Feb 2023 at 23:04, Thomas Munro <thomas(dot)munro(at)gmail(dot)com> wrote:
> > > Done like that in this version. This is the version I'm thinking of
> > > committing, unless someone wants to argue for another level.
> >
> > FWIW the cfbot doesn't understand this patch series. I'm not sure why
> > but it's only trying to apply the first (the MacOS one) and it's
> > failing to apply even that.
>
> Ah, it's because I committed one patch in the series. I'll commit one
> more, and then repost the rest, shortly.
I pushed the main patch, "Don't leak descriptors into subprograms.".
Here's a rebase of the POSIX-next stuff, but I'll sit on these for a
bit longer to see if the build farm agrees with my claim about the
ubiquity of O_CLOEXEC, and if anyone has comments on this stuff.
| Attachment | Content-Type | Size |
|---|---|---|
| v5-0001-Use-accept4-to-accept-connections-where-available.patch | text/x-patch | 5.6 KB |
| v5-0002-Use-newer-client-socket-options-where-available.patch | text/x-patch | 2.9 KB |
| v5-0003-Use-pipe2-for-postmaster-pipe-where-available.patch | text/x-patch | 4.9 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2023-03-02 22:06:04 | Re: buildfarm + meson |
| Previous Message | Andrew Dunstan | 2023-03-02 22:00:47 | Re: buildfarm + meson |