From: | Robert Haas <rhaas(at)postgresql(dot)org> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: pg_combinebackup: Error if incremental file exists in full backu |
Date: | 2024-11-04 15:26:37 |
Message-ID: | E1t7yyT-0009cy-EE@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | 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
------
REL_17_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/e367114429e09696fce01dd6f256e58a2c858734
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(+)
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2024-11-04 15:53:32 | Re: pgsql: Implement pg_wal_replay_wait() stored procedure |
Previous Message | Robert Haas | 2024-11-04 15:09:36 | pgsql: pg_combinebackup: When reconstructing, avoid double slash in fil |