From: Tom Lane Date: Mon, 17 Mar 2014 01:43:49 +0000 (-0400) Subject: Fix advertised dispsize for libpq's sslmode connection parameter. X-Git-Tag: REL9_1_13~5 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6b736102e40dd970037a21e756696f8e22bd8e61;p=postgresql Fix advertised dispsize for libpq's sslmode connection parameter. "8" was correct back when "disable" was the longest allowed value, but since "verify-full" was added, it should be "12". Given the lack of complaints, I wouldn't be surprised if nobody is actually using these values ... but still, if they're in the API, they should be right. Noticed while pursuing a different problem. It's been wrong for quite a long time, so back-patch to all supported branches. --- diff --git a/src/interfaces/libpq/fe-connect.c b/src/interfaces/libpq/fe-connect.c index bd23e9d143..448d975200 100644 --- a/src/interfaces/libpq/fe-connect.c +++ b/src/interfaces/libpq/fe-connect.c @@ -220,7 +220,7 @@ static const PQconninfoOption PQconninfoOptions[] = { * to exclude them since none of them are mandatory. */ {"sslmode", "PGSSLMODE", DefaultSSLMode, NULL, - "SSL-Mode", "", 8}, /* sizeof("disable") == 8 */ + "SSL-Mode", "", 12}, /* sizeof("verify-full") == 12 */ {"sslcert", "PGSSLCERT", NULL, NULL, "SSL-Client-Cert", "", 64},