forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit58538a0
committed
Cause '*' and 'foo.*' notations to mark the referenced RTE(s) as
requiring read permissions. Up till now there was no possible casein which the RTEs wouldn't already have ACL_SELECT set ... but now thatyou can say something like 'INSERT INTO foo ... RETURNING *' this isan essential step. With this commit, a RETURNING clause adds therequirement for SELECT permissions on the target table if and only ifthe clause actually reads the value of at least one target-table column.1 parent65b2f93 commit58538a0
2 files changed
+11
-4
lines changedLines changed: 4 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
8 | 8 |
| |
9 | 9 |
| |
10 | 10 |
| |
11 |
| - | |
| 11 | + | |
12 | 12 |
| |
13 | 13 |
| |
14 | 14 |
| |
| |||
173 | 173 |
| |
174 | 174 |
| |
175 | 175 |
| |
176 |
| - | |
177 |
| - | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
178 | 179 |
| |
179 | 180 |
| |
180 | 181 |
| |
|
Lines changed: 7 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
8 | 8 |
| |
9 | 9 |
| |
10 | 10 |
| |
11 |
| - | |
| 11 | + | |
12 | 12 |
| |
13 | 13 |
| |
14 | 14 |
| |
| |||
889 | 889 |
| |
890 | 890 |
| |
891 | 891 |
| |
| 892 | + | |
| 893 | + | |
| 894 | + | |
892 | 895 |
| |
893 | 896 |
| |
894 | 897 |
| |
| |||
930 | 933 |
| |
931 | 934 |
| |
932 | 935 |
| |
| 936 | + | |
| 937 | + | |
| 938 | + | |
933 | 939 |
| |
934 | 940 |
| |
935 | 941 |
| |
|
0 commit comments
Comments
(0)