pgsql: Fix version number calculation for data folder flush in pg_combi - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Fix version number calculation for data folder flush in pg_combi
Date
Msg-id E1v8S18-001Z2d-0x@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix version number calculation for data folder flush in pg_combinebackup

The version number calculated by read_pg_version_file() is multiplied
once by 10000, to be able to do comparisons based on PG_VERSION_NUM or
equivalents with a minor version included.  However, the version number
given sync_pgdata() was multiplied by 10000 a second time, leading to an
overestimated number.

This issue was harmless (still incorrect) as pg_combinebackup does not
support versions of Postgres older than v10, and sync_pgdata() only
includes a version check due to the rename of pg_xlog/ to pg_wal/.  This
folder rename happened in the development cycle of v10.  This would
become a problem if in the future  sync_pgdata() is changed to have more
version-specific checks.

Oversight in dc212340058b, so backpatch down to v17.

Reviewed-by: Chao Li <li.evan.chao@gmail.com>
Discussion: https://postgr.es/m/aOil5d0y87ZM_wsZ@paquier.xyz
Backpatch-through: 17

Branch
------
REL_17_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/a7346770813e31c3fd38616ebcc1beb61d551eea

Modified Files
--------------
src/bin/pg_combinebackup/pg_combinebackup.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Melanie Plageman
Date:
Subject: pgsql: Eliminate XLOG_HEAP2_VISIBLE from vacuum phase III
Next
From: Michael Paquier
Date:
Subject: pgsql: Introduce frontend API able to retrieve the contents of PG_VERSI