Re: Improve errors when setting incorrect bounds for SSL protocols - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: Improve errors when setting incorrect bounds for SSL protocols
Date
Msg-id 14BFD060-8C9D-43B4-897D-D5D9AA6FC92B@yesql.se
Whole thread Raw
In response to Re: Improve errors when setting incorrect bounds for SSL protocols  (Michael Paquier <michael@paquier.xyz>)
Responses Re: Improve errors when setting incorrect bounds for SSL protocols  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
Working in the TLS corners of the backend, I found while re-reviewing and
re-testing for the release that this patch actually was a small, but vital,
brick shy of a load.  The error handling is always invoked due to a set of
missing braces.  Going into the check will cause the context to be freed and
be_tls_open_server error out.  The tests added narrowly escapes it by not
setting the max version in the final test, but I'm not sure it's worth changing
that now as not setting a value is an interesting testcase too.  Sorry for
missing that at the time of reviewing.

cheers ./daniel


Attachment

pgsql-hackers by date:

Previous
From: tushar
Date:
Subject: Re: [Proposal] Global temporary tables
Next
From: Amit Kapila
Date:
Subject: Re: PG compilation error with Visual Studio 2015/2017/2019