diff options
| author | Michael Paquier <michael@paquier.xyz> | 2021-07-29 11:00:00 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2021-07-29 11:00:00 +0900 |
| commit | efe169c90090684d2a4ffb0810a0cf2c9b72e19d (patch) | |
| tree | c21140483a6fb520c024d5b02bd6dbbf99c0a0f1 /src/bin/pg_verifybackup/pg_verifybackup.c | |
| parent | a66b05b422d74111f2a6407108f4ba7bdddbb6a0 (diff) | |
Add missing exit() in pg_verifybackup when failing to find pg_waldump
pg_verifybackup needs by default pg_waldump to check after a range of
WAL segments required for a backup, except if --no-parse-wal is
specified. The code checked for the presence of the binary pg_waldump
in an installation and reported an error, but it forgot to properly
exit(). This could lead to confusing errors reported.
Reviewed-by: Robert Haas, Fabien Coelho
Discussion: https://postgr.es/m/YQDMdB+B68yePFeT@paquier.xyz
Backpatch-through: 13
Diffstat (limited to 'src/bin/pg_verifybackup/pg_verifybackup.c')
| -rw-r--r-- | src/bin/pg_verifybackup/pg_verifybackup.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/bin/pg_verifybackup/pg_verifybackup.c b/src/bin/pg_verifybackup/pg_verifybackup.c index 20140aa0274..f2ab2c526b9 100644 --- a/src/bin/pg_verifybackup/pg_verifybackup.c +++ b/src/bin/pg_verifybackup/pg_verifybackup.c @@ -304,6 +304,7 @@ main(int argc, char **argv) "but was not the same version as %s.\n" "Check your installation.", "pg_waldump", full_path, "pg_verifybackup"); + exit(1); } } |