Re: SSL tests fail on OpenSSL v3.2.0
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: "Tristan Partin" <tristan@neon.tech>
Cc: "Michael Paquier" <michael@paquier.xyz>,
"Nazir Bilal Yavuz" <byavuz81@gmail.com>,
"Andres Freund" <andres@anarazel.de>,
"Bo Anderson" <mail@boanderson.me>,
"pgsql-hackers" <pgsql-hackers@postgresql.org>
Date: 2023-11-28T01:14:56Z
Lists: pgsql-hackers
"Tristan Partin" <tristan@neon.tech> writes: > On Mon Nov 27, 2023 at 6:21 PM CST, Tom Lane wrote: >> What about LibreSSL? In general, I'm not too pleased with just assuming >> that BIO_get_app_data exists. > Falling back to what existed before is invalid. Well, sure it only worked by accident, but it did work with older OpenSSL versions. If we assume that BIO_get_app_data exists, and somebody tries to use it with a version that hasn't got that, it won't work. Having said that, my concern was mainly driven by the comments in configure.ac claiming that this was an OpenSSL 1.1.0 addition. Looking at the relevant commits, 593d4e47d and 5c6df67e0, it seems that that was less about "the function doesn't exist before 1.1.0" and more about "in 1.1.0 we have to use the function because we can no longer directly access the ptr field". If the function does exist in 0.9.8 then I concur that we don't need to test. regards, tom lane
Commits
-
Use BIO_{get,set}_app_data instead of BIO_{get,set}_data.
- efb804682719 13.14 landed
- efa8f606402c 16.2 landed
- c82207a548db 17.0 landed
- b2b1f12882fb 14.11 landed
- 5dd30bb54bfe 15.6 landed
- 0bd682246a61 12.18 landed