]> granicus.if.org Git - postgresql/commitdiff
Fix multiple minor infelicities in aclchk.c error reports.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 13 Jun 2016 17:53:10 +0000 (13:53 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 13 Jun 2016 17:53:10 +0000 (13:53 -0400)
pg_type_aclmask reported the wrong type's OID when complaining that
it could not find a type's typelem.  It also failed to provide a
suitable errcode when the initially given OID doesn't exist (which
is a user-facing error, since that OID can be user-specified).
pg_foreign_data_wrapper_aclmask and pg_foreign_server_aclmask likewise
lacked errcode specifications.  Trivial cosmetic adjustments too.

The wrong-type-OID problem was reported by Petru-Florin Mihancea in
bug #14186; the other issues noted by me while reading the code.
These errors all seem to be aboriginal in the respective routines, so
back-patch as necessary.

Report: <20160613163159.5798.52928@wrigleys.postgresql.org>

src/backend/catalog/aclchk.c

index acfb8fa4c93448ca44a4ac850f288251acd6d36e..98b6a298a99afcc1eced8c9ec16f7170be3edbd5 100644 (file)
@@ -3870,7 +3870,8 @@ pg_foreign_data_wrapper_aclmask(Oid fdw_oid, Oid roleid,
        tuple = SearchSysCache1(FOREIGNDATAWRAPPEROID, ObjectIdGetDatum(fdw_oid));
        if (!HeapTupleIsValid(tuple))
                ereport(ERROR,
-                               (errmsg("foreign-data wrapper with OID %u does not exist",
+                               (errcode(ERRCODE_UNDEFINED_OBJECT),
+                                errmsg("foreign-data wrapper with OID %u does not exist",
                                                fdw_oid)));
        fdwForm = (Form_pg_foreign_data_wrapper) GETSTRUCT(tuple);
 
@@ -3931,7 +3932,8 @@ pg_foreign_server_aclmask(Oid srv_oid, Oid roleid,
        tuple = SearchSysCache1(FOREIGNSERVEROID, ObjectIdGetDatum(srv_oid));
        if (!HeapTupleIsValid(tuple))
                ereport(ERROR,
-                               (errmsg("foreign server with OID %u does not exist",
+                               (errcode(ERRCODE_UNDEFINED_OBJECT),
+                                errmsg("foreign server with OID %u does not exist",
                                                srv_oid)));
        srvForm = (Form_pg_foreign_server) GETSTRUCT(tuple);