Some incorrect option sizes for PQconninfoOption in libpq
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
Attachments:
libpq-connect-dispsize.patchtext/x-diff; charset=us-asciiDownload+3-3
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