Re: Add ssl_(supported|shared)_groups to sslinfo

From: Zsolt Parragi <zsolt(dot)parragi(at)percona(dot)com>
To: Dmitry Dolgov <9erthalion6(at)gmail(dot)com>
Cc: Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com>, Daniel Gustafsson <daniel(at)yesql(dot)se>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Add ssl_(supported|shared)_groups to sslinfo
Date: 2026-05-12 21:48:20
Message-ID: CAN4CZFMvnEPa2xpyvd7SYQ2U0os6gDucu_T_fsvM0q+xiqCD=w@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hello!

+#define HAVE_SSL_GROUPS \
+ defined(HAVE_DECL_SSL_GET1_GROUPS) && \
+ defined(HAVE_DECL_SSL_GET_NEGOTIATED_GROUP) && \
+ defined(HAVE_SSL_GROUP_TO_NAME)

I don't think this check works properly
1. autoconf/meson always defines HAVE_DECL to 0/1, so defined() always
returns true
2. in practice it should work, but using defined() in a macro
expansion is undefined behavior

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Afrah Razzak 2026-05-12 21:59:31 [PATCH] psql: Add missing IO option to EXPLAIN tab completion
Previous Message Marcos Pegoraro 2026-05-12 21:45:05 Re: Missing jsonb_ ... functions on DOCs