forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitc727f51
committed
Refactor aclcheck functions
Instead of dozens of mostly-duplicate pg_foo_aclcheck() functions,write one common function object_aclcheck() that can handle almost allof them. We already have all the information we need, such as whichsystem catalog corresponds to which catalog table and which column isthe ACL column.There are a few pg_foo_aclcheck() that don't work via the genericfunction and have special APIs, so those stay as is.I also changed most pg_foo_aclmask() functions to static functions,since they are not used outside of aclchk.c.Reviewed-by: Corey Huinker <corey.huinker@gmail.com>Reviewed-by: Antonin Houska <ah@cybertec.at>Discussion:https://www.postgresql.org/message-id/flat/95c30f96-4060-2f48-98b5-a4392d3b6066@enterprisedb.com1 parentafbfc02 commitc727f51
File tree
37 files changed
+319
-682
lines changed- contrib/dblink
- src
- backend
- access/common
- catalog
- commands
- executor
- optimizer/util
- parser
- tcop
- utils
- adt
- fmgr
- init
- include/utils
- pl/tcl
37 files changed
+319
-682
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2838 | 2838 |
| |
2839 | 2839 |
| |
2840 | 2840 |
| |
2841 |
| - | |
| 2841 | + | |
2842 | 2842 |
| |
2843 | 2843 |
| |
2844 | 2844 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
821 | 821 |
| |
822 | 822 |
| |
823 | 823 |
| |
824 |
| - | |
| 824 | + | |
825 | 825 |
| |
826 | 826 |
| |
827 | 827 |
| |
|
0 commit comments
Comments
(0)