]> granicus.if.org Git - postgresql/commit
Fix parsing of integer values for connection parameters in libpq
authorMichael Paquier <michael@paquier.xyz>
Mon, 21 Oct 2019 02:17:43 +0000 (11:17 +0900)
committerMichael Paquier <michael@paquier.xyz>
Mon, 21 Oct 2019 02:17:43 +0000 (11:17 +0900)
commit2b0f959b5119cb2bb1d135ac04a8c5272bbcab03
tree2dd65bd1b592f51f01dcbe9f19271aa7434e62e7
parentef13f914e6c2a66f186f73ec1bba682e56dc621b
Fix parsing of integer values for connection parameters in libpq

Commit e7a2217 has introduced stricter checks for integer values in
connection parameters for libpq.  However this failed to correctly check
after trailing whitespaces, while leading whitespaces were discarded per
the use of strtol(3).  This fixes and refactors the parsing logic to
handle both cases consistently.  Note that trying to restrict the use of
trailing whitespaces can easily break connection strings like in ECPG
regression tests (these have allowed me to catch the parsing bug with
connect_timeout).

Author: Michael Paquier
Reviewed-by: Lars Kanis
Discussion: https://postgr.es/m/a9b4cbd7-4ecb-06b2-ebd7-1739bbff3217@greiz-reinsdorf.de
Backpatch-through: 12
src/interfaces/libpq/fe-connect.c