Re: Missing include in be-secure-openssl.c? - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Missing include in be-secure-openssl.c?
Date
Msg-id 284247.1635996826@sss.pgh.pa.us
Whole thread Raw
In response to Re: Missing include in be-secure-openssl.c?  (Thomas Munro <thomas.munro@gmail.com>)
Responses Re: Missing include in be-secure-openssl.c?
List pgsql-hackers
Thomas Munro <thomas.munro@gmail.com> writes:
> Ahh, I think this contains some relevant bits, and we have indeed
> messed around with mentioned Windows headers in master.
> https://stackoverflow.com/questions/49504648/x509-name-macro-in-c-wont-compile/49504794

Oooh .... note the comment there about

    This helped me too, but I found the simplest thing to do was just make sure the OpenSSL headers were #included last
inmy source file. –  

So the fact that be-secure-openssl.c and fe-secure-openssl.c
are including things in different orders *is* relevant.  I'd
previously discovered this bit in OpenSSL's headers (ossl_typ.h):

# ifdef _WIN32
#  undef X509_NAME
...
#endif
...
typedef struct X509_name_st X509_NAME;

So that will work around the problem as long as it's #include'd
after the relevant Windows headers.

But I don't get the point about where HEAD is different from v14?
be-secure-openssl.c isn't.

            regards, tom lane



pgsql-hackers by date:

Previous
From: "houzj.fnst@fujitsu.com"
Date:
Subject: RE: row filtering for logical replication
Next
From: Thomas Munro
Date:
Subject: Re: Missing include in be-secure-openssl.c?