Re: [PATCH] Avoid mixing custom and OpenSSL BIO functions - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: [PATCH] Avoid mixing custom and OpenSSL BIO functions
Date
Msg-id 37DB2E05-503D-49EA-8B17-7B31C92F09AD@yesql.se
Whole thread Raw
In response to Re: [PATCH] Avoid mixing custom and OpenSSL BIO functions  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
> On 29 Nov 2024, at 19:36, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> David Benjamin <davidben@google.com> writes:
>> Thanks! I got asked about release branches, so I thought I'd pass it along:
>> how do you all handle merges to release branches and would it make sense to
>> merge this change? On the one hand, nothing is actively on fire yet, but
>> the current setup does risk breakage if OpenSSL ever migrates BIO_s_socket
>> to their new size_t-clean internals.
>
> We theoretically could back-patch 6f782a2a1, as it doesn't appear to
> introduce any ABI-breaking changes.  (The new field in struct Port
> could be an issue, but it looks like it fits into what was padding
> space, so probably fine.)  However, I'm not sure that it's attractive
> to do so from a risk/benefit standpoint.  That code's received only
> minimal testing so far, and the problem it's fixing is as yet
> hypothetical.
>
> On balance I think I'd vote against a back-patch now.  We could
> reconsider next year once PG v18 has gotten a reasonable amount of
> beta testing.

I agree with all of the above.

--
Daniel Gustafsson




pgsql-hackers by date:

Previous
From: Marcos Pegoraro
Date:
Subject: Re: UUID v7
Next
From: Masahiko Sawada
Date:
Subject: Re: UUID v7