- Notifications
You must be signed in to change notification settings - Fork5
Commit9029df1
committed
Fix updateAclDependencies() to not assume that ACL role dependencies can only
be added during GRANT and can only be removed during REVOKE; and fix itscallers to not lie to it about the existing set of dependencies wheninstantiating a formerly-default ACL. The previous coding accidentally failedto malfunction so long as default ACLs contain only references to the object'sowning role, because that role is ignored by updateAclDependencies. Howeverthis is obviously pretty fragile, as well as being an undocumented assumption.The new coding is a few lines longer but IMO much clearer.1 parent80390f4 commit9029df1
File tree
5 files changed
+248
-153
lines changed- src
- backend/catalog
- include/catalog
5 files changed
+248
-153
lines changed0 commit comments
Comments
(0)