From: Tom Lane Date: Mon, 17 Mar 2014 01:43:57 +0000 (-0400) Subject: Fix advertised dispsize for libpq's sslmode connection parameter. X-Git-Tag: REL8_4_21~3 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=962c14894768f95fabc82d4fb7db38c76d88fb97;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 e0636167f4..728e0fd106 100644 --- a/src/interfaces/libpq/fe-connect.c +++ b/src/interfaces/libpq/fe-connect.c @@ -181,7 +181,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},