diff options
| author | Amit Kapila <akapila@postgresql.org> | 2020-03-18 09:37:08 +0530 |
|---|---|---|
| committer | Amit Kapila <akapila@postgresql.org> | 2020-03-18 09:37:08 +0530 |
| commit | 18c41cb6b7140a0d360900c10ac23072579194a1 (patch) | |
| tree | 9d8bb379027762289110b33641d8e29de1444199 | |
| parent | 974c8ea0afe35a55c1b9ffb9f2090058b9f356dd (diff) | |
| download | postgresql-18c41cb6b7140a0d360900c10ac23072579194a1.tar.gz postgresql-18c41cb6b7140a0d360900c10ac23072579194a1.zip | |
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
| -rw-r--r-- | contrib/adminpack/adminpack.c | 6 | ||||
| -rw-r--r-- | 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 4f613e240bb..d4b87b38bd6 100644 --- a/contrib/adminpack/adminpack.c +++ b/contrib/adminpack/adminpack.c @@ -187,7 +187,7 @@ pg_file_write_internal(text *file, text *data, bool replace) if (stat(filename, &fst) >= 0) ereport(ERROR, - (ERRCODE_DUPLICATE_FILE, + (errcode(ERRCODE_DUPLICATE_FILE), errmsg("file \"%s\" exists", filename))); f = AllocateFile(filename, "wb"); @@ -319,7 +319,7 @@ pg_file_rename_internal(text *file1, text *file2, text *file3) if (rc >= 0 || errno != ENOENT) { ereport(ERROR, - (ERRCODE_DUPLICATE_FILE, + (errcode(ERRCODE_DUPLICATE_FILE), errmsg("cannot rename to target file \"%s\"", fn3 ? fn3 : fn2))); } @@ -350,7 +350,7 @@ pg_file_rename_internal(text *file1, text *file2, text *file3) 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 dfbda5458fd..d2ea38bdabc 100644 --- a/src/backend/storage/page/bufpage.c +++ b/src/backend/storage/page/bufpage.c @@ -147,7 +147,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))); |