diff options
| author | Peter Eisentraut <peter_e@gmx.net> | 2012-06-15 22:55:03 +0300 |
|---|---|---|
| committer | Peter Eisentraut <peter_e@gmx.net> | 2012-06-15 22:55:03 +0300 |
| commit | 15b1918e7d3532f0e4ec3455ae6ce45fae09c86f (patch) | |
| tree | 659b8a57116719720c1b59d7d6ad507d72d3c748 /src/backend/commands/tablecmds.c | |
| parent | d933092e0ab55ed95615977e50963f969ac9793d (diff) | |
| download | postgresql-15b1918e7d3532f0e4ec3455ae6ce45fae09c86f.tar.gz postgresql-15b1918e7d3532f0e4ec3455ae6ce45fae09c86f.zip | |
Improve reporting of permission errors for array types
Because permissions are assigned to element types, not array types,
complaining about permission denied on an array type would be
misleading to users. So adjust the reporting to refer to the element
type instead.
In order not to duplicate the required logic in two dozen places,
refactor the permission denied reporting for types a bit.
pointed out by Yeb Havinga during the review of the type privilege
feature
Diffstat (limited to 'src/backend/commands/tablecmds.c')
| -rw-r--r-- | src/backend/commands/tablecmds.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c index 5c69cfb85a2..70753e33e43 100644 --- a/src/backend/commands/tablecmds.c +++ b/src/backend/commands/tablecmds.c @@ -526,8 +526,7 @@ DefineRelation(CreateStmt *stmt, char relkind, Oid ownerId) aclresult = pg_type_aclcheck(ofTypeId, GetUserId(), ACL_USAGE); if (aclresult != ACLCHECK_OK) - aclcheck_error(aclresult, ACL_KIND_TYPE, - format_type_be(ofTypeId)); + aclcheck_error_type(aclresult, ofTypeId); } else ofTypeId = InvalidOid; @@ -4500,8 +4499,7 @@ ATExecAddColumn(List **wqueue, AlteredTableInfo *tab, Relation rel, aclresult = pg_type_aclcheck(typeOid, GetUserId(), ACL_USAGE); if (aclresult != ACLCHECK_OK) - aclcheck_error(aclresult, ACL_KIND_TYPE, - format_type_be(typeOid)); + aclcheck_error_type(aclresult, typeOid); collOid = GetColumnDefCollation(NULL, colDef, typeOid); @@ -7248,8 +7246,7 @@ ATPrepAlterColumnType(List **wqueue, aclresult = pg_type_aclcheck(targettype, GetUserId(), ACL_USAGE); if (aclresult != ACLCHECK_OK) - aclcheck_error(aclresult, ACL_KIND_TYPE, - format_type_be(targettype)); + aclcheck_error_type(aclresult, targettype); /* And the collation */ targetcollid = GetColumnDefCollation(NULL, def, targettype); |