]> granicus.if.org Git - postgresql/commit
Handle ALTER EXTENSION ADD/DROP with pg_init_privs
authorStephen Frost <sfrost@snowman.net>
Mon, 30 Jan 2017 04:05:07 +0000 (23:05 -0500)
committerStephen Frost <sfrost@snowman.net>
Mon, 30 Jan 2017 04:05:07 +0000 (23:05 -0500)
commite54f75722c720b596ec5e72154cc899da199de5b
tree4ea64893fb063beb96d9014ec53c08ec9bcb7c18
parentfb94ca77f1400e236b00d432dccfbe4f1124971c
Handle ALTER EXTENSION ADD/DROP with pg_init_privs

In commit 6c268df, pg_init_privs was added to track the initial
privileges of catalog objects and extensions.  Unfortunately, that
commit didn't include understanding of ALTER EXTENSION ADD/DROP, which
allows the objects associated with an extension to be changed after the
initial CREATE EXTENSION script has been run.

The result of this meant that ACLs for objects added through
ALTER EXTENSION ADD were not recorded into pg_init_privs and we would
end up including those ACLs in pg_dump when we shouldn't have.

This commit corrects that by making sure to have pg_init_privs updated
when ALTER EXTENSION ADD/DROP is run, recording the permissions as they
are at ALTER EXTENSION ADD time, and removing any if/when ALTER
EXTENSION DROP is called.

This issue was pointed out by Moshe Jacobson as commentary on bug #14456
(which was actually a bug about versions prior to 9.6 not handling
custom ACLs on extensions correctly, an issue now addressed with
pg_init_privs in 9.6).

Back-patch to 9.6 where pg_init_privs was introduced.
src/backend/catalog/aclchk.c
src/backend/commands/extension.c
src/include/utils/acl.h
src/test/modules/test_pg_dump/expected/test_pg_dump.out
src/test/modules/test_pg_dump/sql/test_pg_dump.sql
src/test/modules/test_pg_dump/t/001_base.pl