diff options
| author | Stephen Frost <sfrost@snowman.net> | 2015-02-17 15:36:24 -0500 |
|---|---|---|
| committer | Stephen Frost <sfrost@snowman.net> | 2015-02-17 15:36:24 -0500 |
| commit | cfc14b2bf348528bc8f0f04fe1ff80e7abdf0529 (patch) | |
| tree | 15bfc41b29d9b1e436571c8d10cf35100ad67658 /src/backend/commands/trigger.c | |
| parent | 64e0458383c1fad7660499c312ab3e419513690f (diff) | |
Minor cleanup of column-level priv fix
Commit 9406884af19e2620a14059e64d4eb6ab430ab328 cleaned up
column-privilege related leaks in various error-message paths, but ended
up including a few more things than it should have in the back branches.
Specifically, there's no need for the GetModifiedColumns macro in
execMain.c as 9.1 and older didn't include the row in check constraint
violations. Further, the regression tests added to check those cases
aren't necessary.
This patch removes the GetModifiedColumns macro from execMain.c, removes
the comment which was added to trigger.c related to the duplicate macro
definition, and removes the check-constraint-related regression tests.
Pointed out by Robert.
Back-patched to 9.1 and 9.0.
Diffstat (limited to 'src/backend/commands/trigger.c')
| -rw-r--r-- | src/backend/commands/trigger.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/backend/commands/trigger.c b/src/backend/commands/trigger.c index bbb83911a9a..bd5dee4286a 100644 --- a/src/backend/commands/trigger.c +++ b/src/backend/commands/trigger.c @@ -60,12 +60,6 @@ int SessionReplicationRole = SESSION_REPLICATION_ROLE_ORIGIN; -/* - * Note that this macro also exists in executor/execMain.c. There does not - * appear to be any good header to put it into, given the structures that - * it uses, so we let them be duplicated. Be sure to update both if one needs - * to be changed, however. - */ #define GetModifiedColumns(relinfo, estate) \ (rt_fetch((relinfo)->ri_RangeTableIndex, (estate)->es_range_table)->modifiedCols) |