mirror/postgresql
2
0
Fork 0
mirror of https://git.postgresql.org/git/postgresql.git synced 2024-12-27 08:39:28 +08:00
Code Issues Packages Projects Releases Wiki Activity

Teach libpq to decode server version correctly from future servers.

Browse Source 
Beginning with the next development cycle, PG servers will report two-part
not three-part version numbers.  Fix libpq so that it will compute the
correct numeric representation of such server versions for reporting by
PQserverVersion().  It's desirable to get this into the field and
back-patched ASAP, so that older clients are more likely to understand the
new server version numbering by the time any such servers are in the wild.

(The results with an old client would probably not be catastrophic anyway
for a released server; for example "10.1" would be interpreted as 100100
which would be wrong in detail but would not likely cause an old client to
misbehave badly.  But "10devel" or "10beta1" would result in sversion==0
which at best would result in disabling all use of modern features.)

Extracted from a patch by Peter Eisentraut; comments added by me

Patch: <802ec140-635d-ad86-5fdf-d3af0e260c22@2ndquadrant.com>
This commit is contained in:
Tom Lane 2016-08-05 18:58:12 -04:00
parent fc509cd824
commit 69dc5ae408
1 changed files with 22 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
src/interfaces/libpq/fe-exec.c
View File

@ -983,14 +983,31 @@ pqSaveParameterStatus(PGconn *conn, const char *name, const char *value)
cnt = sscanf(value, "%d.%d.%d", &vmaj, &vmin, &vrev);
if (cnt < 2)
conn->sversion = 0; /* unknown */
else
if (cnt == 3)
{
if (cnt == 2)
vrev = 0;
/* old style, e.g. 9.6.1 */
conn->sversion = (100 * vmaj + vmin) * 100 + vrev;
}
else if (cnt == 2)
{
if (vmaj >= 10)
{
/* new style, e.g. 10.1 */
conn->sversion = 100 * 100 * vmaj + vmin;
}
else
{
/* old style without minor version, e.g. 9.6devel */
conn->sversion = (100 * vmaj + vmin) * 100;
}
}
else if (cnt == 1)
{
/* new style without minor version, e.g. 10devel */
conn->sversion = 100 * 100 * vmaj;
}
else
conn->sversion = 0; /* unknown */
}
}