pgsql: pg_combinebackup: Error if incremental file exists in full backu - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: pg_combinebackup: Error if incremental file exists in full backu
Date
Msg-id E1t7yyT-0009cx-Dr@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
pg_combinebackup: Error if incremental file exists in full backup.

Suppose that you run a command like "pg_combinebackup b1 b2 -o output",
but both b1 and b2 contain an INCREMENTAL.$something file in a directory
that is expected to contain relation files. This is an error, but the
previous code would not detect the problem and instead write a garbage
full file named $something to the output directory. This commit adds
code to detect the error and a test case to verify the behavior.

It's difficult to imagine that this will ever happen unless someone
is intentionally trying to break incremental backup, but per discussion,
let's consider that the lack of adequate sanity checking in this area is
a bug and back-patch to v17, where incremental backup was introduced.

Patch by me, reviewed by Bertrand Drouvot and Amul Sul.

Discussion: http://postgr.es/m/CA+TgmoaD7dBYPqe7kMtO0dyto7rd0rUh7joh=JPUSaFszKY6Pg@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/0b1765d959508050dba023099e36325cfeec3ff9

Modified Files
--------------
src/bin/pg_combinebackup/meson.build           |  1 +
src/bin/pg_combinebackup/reconstruct.c         |  8 ++++
src/bin/pg_combinebackup/t/009_no_full_file.pl | 66 ++++++++++++++++++++++++++
3 files changed, 75 insertions(+)


pgsql-committers by date:

Previous
From: Robert Haas
Date:
Subject: pgsql: pg_combinebackup: When reconstructing, avoid double slash in fil
Next
From: Heikki Linnakangas
Date:
Subject: pgsql: Split ProcSleep function into JoinWaitQueue and ProcSleep