- Notifications
You must be signed in to change notification settings - Fork5
Commit58c909b
committed
OK -- here's take#5.
It "make"s and "make check"s clean against current cvs tip.There are now both Text and Name variants, and the regression test supportis rolled into the patch. Note that to be complete wrt Name based variants,there are now 12 user visible versions of has_table_privilege:has_table_privilege(Text usename, Text relname, Text priv_type)has_table_privilege(Text usename, Name relname, Text priv_type)has_table_privilege(Name usename, Text relname, Text priv_type)has_table_privilege(Name usename, Name relname, Text priv_type)has_table_privilege(Text relname, Text priv_type) /* assumes current_user */has_table_privilege(Name relname, Text priv_type) /* assumes current_user */has_table_privilege(Text usename, Oid reloid, Text priv_type)has_table_privilege(Name usename, Oid reloid, Text priv_type)has_table_privilege(Oid reloid, Text priv_type) /* assumes current_user */has_table_privilege(Oid usesysid, Text relname, Text priv_type)has_table_privilege(Oid usesysid, Name relname, Text priv_type)has_table_privilege(Oid usesysid, Oid reloid, Text priv_type)For the Text based inputs, a new internal function, get_Name is used(shamelessly copied from get_seq_name in sequence.c) to downcase if notquoted, or remove quotes if quoted, and truncate. I also added a few testcases for the downcasing, quote removal, and Name based variants to theregression test.Joe Conway1 parent13a52c1 commit58c909b
File tree
6 files changed
+825
-5
lines changed- src
- backend/utils/adt
- include
- catalog
- utils
- test/regress
6 files changed
+825
-5
lines changed0 commit comments
Comments
(0)