]> 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:09 +0000 (23:05 -0500)
committerStephen Frost <sfrost@snowman.net>
Mon, 30 Jan 2017 04:05:09 +0000 (23:05 -0500)
commit20064c0ec201fd2302757c1fdb2279e9dc9a4030
tree53fbc33f1cc5a6e8ab8f3cc1e175f58449fbdfab
parent73cd4896f41052bdaf779fa1b63aca5f10c56a62
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