Horton

Thread

Commits

  1. Fix dispsize for libpq connection parameters channel_binding and gssencmode

  1. Some incorrect option sizes for PQconninfoOption in libpq

    Michael Paquier <michael@paquier.xyz> — 2020-01-28T05:36:33Z

    Hi all,

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?
--
Michael

  2. Re: Some incorrect option sizes for PQconninfoOption in libpq

    Daniel Gustafsson <daniel@yesql.se> — 2020-01-28T10:57:19Z

    > 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

  3. Re: Some incorrect option sizes for PQconninfoOption in libpq

    Michael Paquier <michael@paquier.xyz> — 2020-01-29T06:10:16Z

    On Tue, Jan 28, 2020 at 11:57:19AM +0100, Daniel Gustafsson wrote:
> Nice catch! +1 on the attached patch.

Thanks, fixed and backpatched down to 12, where gssencmode has been
added.
--
Michael