Re: Backend specific ifdefs in sha2.h - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Backend specific ifdefs in sha2.h
Date
Msg-id 20190613082927.GA1799@paquier.xyz
Whole thread Raw
In response to Backend specific ifdefs in sha2.h  (Daniel Gustafsson <daniel@yesql.se>)
Responses Re: Backend specific ifdefs in sha2.h
List pgsql-hackers
On Thu, Jun 13, 2019 at 09:32:28AM +0200, Daniel Gustafsson wrote:
> The #ifdef guards in sha2.h are using USE_SSL when they in fact are guarding
> the inclusion of OpenSSL specific code.  This has never caused any issues as
> there only is a single supported TLS backend in core so far, but since we’ve
> spent a significant amount of energy on making the TLS backend non-hardcoded
> it seems we should fix this too.  The Makefile around sha2.c/sha2_openssl.c is
> already testing for openssl rather than ssl (which given src/Makefile.global
> variables makes perfect sense of course).

Right, good catch.  I would not back-patch that though as currently
USE_SSL <=> USE_OPENSSL.  Any suggestions or thoughts from others?
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Alexander Lakhin
Date:
Subject: Re: Fix inconsistencies for v12 (pass 2)
Next
From: Daniel Gustafsson
Date:
Subject: Re: Backend specific ifdefs in sha2.h