diff options
| author | Michael Paquier <michael@paquier.xyz> | 2021-07-29 10:59:49 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2021-07-29 10:59:49 +0900 |
| commit | 2ad98fdf53edf1a90123e3b189cc6f0a31986891 (patch) | |
| tree | c39410bdcddbe939a42e9a93f15449ab105490a9 /src | |
| parent | 245de48455daaf51f2f3e8b198d87b5dd9fd4516 (diff) | |
| download | postgresql-2ad98fdf53edf1a90123e3b189cc6f0a31986891.tar.gz postgresql-2ad98fdf53edf1a90123e3b189cc6f0a31986891.zip | |
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')
| -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 f5ebd57a47f..bb93b43093d 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); } } |