Re: Some incorrect option sizes for PQconninfoOption in libpq - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: Some incorrect option sizes for PQconninfoOption in libpq
Date
Msg-id 03325A52-FB1F-4BA6-8050-66E31333AC5D@yesql.se
Whole thread Raw
In response to Some incorrect option sizes for PQconninfoOption in libpq  (Michael Paquier <michael@paquier.xyz>)
Responses Re: Some incorrect option sizes for PQconninfoOption in libpq  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
> On 28 Jan 2020, at 06:36, Michael Paquier <michael@paquier.xyz> wrote:

> I was reviewing the libpq code for the recent SSL protocol patch, and
> noticed two mistakes with dispsize for the following parameters:
> - channel_binding should be at 8, the largest value being "require".
> - gssencmode should be at 8.
> 
> In those cases the zero-terminator was forgotten in the count.  A
> similar mistake was done in the past for sslmode that was fixed by
> f4051e36.  It is unlikely that dispsize is being used, but we cannot
> break that on compatibility grounds, and the current numbers are
> incorrect so let's fix it.
> 
> Thoughts?

Nice catch! +1 on the attached patch.

cheers ./daniel



pgsql-hackers by date:

Previous
From: Kyotaro Horiguchi
Date:
Subject: ReadRecord wrongly assumes randAccess after 38a957316d.
Next
From: Heikki Linnakangas
Date:
Subject: Re: ReadRecord wrongly assumes randAccess after 38a957316d.