pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Heikki Linnakangas <heikki.linnakangas@iki.fi>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Make the memory layout of Port struct independent of USE_OPENSSL
Date: Fri, 22 Nov 2024 16:05:34 +0000
Message-ID: <E1tEWA2-0032yz-EA@gemulon.postgresql.org> (raw)

Make the memory layout of Port struct independent of USE_OPENSSL

Commit d39a49c1e4 added new fields to the struct, but missed the "keep
these last" comment on the previous fields. Add placeholder variables
so that the offsets of the fields are the same whether you build with
USE_OPENSSL or not. This is a courtesy to extensions that might peek
at the fields, to make the ABI the same regardless of the options used
to build PostgreSQL.

In reality, I don't expect any extensions to look at the 'raw_buf'
fields. Firstly, they are new in v17, so no one's written such
extensions yet. Secondly, extensions should have no business poking at
those fields anyway. Nevertheless, fix this properly on 'master'. On
v17, we mustn't change the memory layout, so just fix the comments.

Author: Jacob Champion
Discussion: https://www.postgresql.org/message-id/raw/CAOYmi%2BmKVJNzn5_TD_MK%3DhqO64r_w8Gb0FHCLk0oAkW-PJv8jQ@ma...

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/8fb5936703e84945766b8425bfe54e0060dea231

Modified Files
--------------
src/include/libpq/libpq-be.h | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)



view thread (2+ messages)

Message-ID: <E1tEWA2-0032yz-EA@gemulon.postgresql.org>
Permalink:  ../E1tEWA2-0032yz-EA@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1tEWA2-0032yz-EA@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: heikki.linnakangas@iki.fi, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Make the memory layout of Port struct independent of USE_OPENSSL
  In-Reply-To: <E1tEWA2-0032yz-EA@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox