diff options
| author | Michael Paquier <michael@paquier.xyz> | 2021-01-20 11:39:14 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2021-01-20 11:39:14 +0900 |
| commit | 31e0f9d76bb0196b92f6870a8e1e3e2a4e5e2b28 (patch) | |
| tree | a466ed64f8abd55bb6d621c53d5781221e304e56 /src/backend | |
| parent | 188cd4f440ed6bb2b3120ade9a2277c91d79215c (diff) | |
| download | postgresql-31e0f9d76bb0196b92f6870a8e1e3e2a4e5e2b28.tar.gz postgresql-31e0f9d76bb0196b92f6870a8e1e3e2a4e5e2b28.zip | |
Fix ALTER DEFAULT PRIVILEGES with duplicated objects
Specifying duplicated objects in this command would lead to unique
constraint violations in pg_default_acl or "tuple already updated by
self" errors. Similarly to GRANT/REVOKE, increment the command ID after
each subcommand processing to allow this case to work transparently.
A regression test is added by tweaking one of the existing queries of
privileges.sql to stress this case.
Reported-by: Andrus
Author: Michael Paquier
Reviewed-by: Álvaro Herrera
Discussion: https://postgr.es/m/ae2a7dc1-9d71-8cba-3bb9-e4cb7eb1f44e@hot.ee
Backpatch-through: 9.5
Diffstat (limited to 'src/backend')
| -rw-r--r-- | src/backend/catalog/aclchk.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/backend/catalog/aclchk.c b/src/backend/catalog/aclchk.c index cb2c4972ade..4f1e8a46e0d 100644 --- a/src/backend/catalog/aclchk.c +++ b/src/backend/catalog/aclchk.c @@ -1365,6 +1365,9 @@ SetDefaultACL(InternalDefaultACL *iacls) ReleaseSysCache(tuple); table_close(rel, RowExclusiveLock); + + /* prevent error when processing duplicate objects */ + CommandCounterIncrement(); } |