From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Add some missing exit() calls in error paths for various binarie |
Date: | 2021-07-29 02:44:34 |
Message-ID: | E1m8w2E-0001ZL-K1@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Add some missing exit() calls in error paths for various binaries
The following changes are done:
- In pg_archivecleanup, the cleanup of older WAL segments would never
fail immediately.
- In pgbench, the initialization of a thread barrier would not fail
hard.
- In pg_recvlogical, a stat() failure never got the call.
- In pg_basebackup, two chmod() reported a failure without exit()'ing
when unpacking some tar data freshly received. It may be possible to
continue writing some data even after this failure, but that could be
confusing to the user at the end.
These are arguably bugs, but they would happen for code paths where a
failure is unlikely going to happen, so no backpatch is done.
Reviewed-by: Robert Haas, Fabien Coelho
Discussion: https://postgr.es/m/YQDMdB+B68yePFeT@paquier.xyz
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/856de3b39cf6041338b286a99257c324ce647f4e
Modified Files
--------------
src/bin/pg_archivecleanup/pg_archivecleanup.c | 11 ++++++++++-
src/bin/pg_basebackup/pg_basebackup.c | 6 ++++++
src/bin/pg_basebackup/pg_recvlogical.c | 3 +++
src/bin/pgbench/pgbench.c | 3 +++
4 files changed, 22 insertions(+), 1 deletion(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2021-07-29 05:51:12 | pgsql: Fix typo in tab-complete.c |
Previous Message | Michael Paquier | 2021-07-29 02:00:31 | pgsql: Add missing exit() in pg_verifybackup when failing to find pg_wa |