From fe27ce1c136d191b46717fda55b267a2d26f806c Mon Sep 17 00:00:00 2001 From: Amit Kapila Date: Wed, 18 Mar 2020 10:08:56 +0530 Subject: [PATCH] Add missing errcode() in a few ereport calls. This will allow to specifying SQLSTATE error code for the errors in the missing places. Reported-by: Sawada Masahiko Author: Sawada Masahiko Backpatch-through: 9.5 Discussion: https://postgr.es/m/CA+fd4k6N8EjNvZpM8nme+y+05mz-SM8Z_BgkixzkA34R+ej0Kw@mail.gmail.com --- contrib/adminpack/adminpack.c | 6 +++--- src/backend/storage/page/bufpage.c | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/contrib/adminpack/adminpack.c b/contrib/adminpack/adminpack.c index 0dae0b98ce2..30753088e0b 100644 --- a/contrib/adminpack/adminpack.c +++ b/contrib/adminpack/adminpack.c @@ -128,7 +128,7 @@ pg_file_write(PG_FUNCTION_ARGS) if (stat(filename, &fst) >= 0) ereport(ERROR, - (ERRCODE_DUPLICATE_FILE, + (errcode(ERRCODE_DUPLICATE_FILE), errmsg("file \"%s\" exists", filename))); f = AllocateFile(filename, "wb"); @@ -194,7 +194,7 @@ pg_file_rename(PG_FUNCTION_ARGS) if (rc >= 0 || errno != ENOENT) { ereport(ERROR, - (ERRCODE_DUPLICATE_FILE, + (errcode(ERRCODE_DUPLICATE_FILE), errmsg("cannot rename to target file \"%s\"", fn3 ? fn3 : fn2))); } @@ -225,7 +225,7 @@ pg_file_rename(PG_FUNCTION_ARGS) else { ereport(ERROR, - (ERRCODE_UNDEFINED_FILE, + (errcode(ERRCODE_UNDEFINED_FILE), errmsg("renaming \"%s\" to \"%s\" was reverted", fn2, fn3))); } diff --git a/src/backend/storage/page/bufpage.c b/src/backend/storage/page/bufpage.c index 11607827d8d..bf0d3494bd3 100644 --- a/src/backend/storage/page/bufpage.c +++ b/src/backend/storage/page/bufpage.c @@ -140,7 +140,7 @@ PageIsVerified(Page page, BlockNumber blkno) if (checksum_failure) { ereport(WARNING, - (ERRCODE_DATA_CORRUPTED, + (errcode(ERRCODE_DATA_CORRUPTED), errmsg("page verification failed, calculated checksum %u but expected %u", checksum, p->pd_checksum)));