pgsql: Minor cleanup of column-level priv fix - Mailing list pgsql-committers

From Stephen Frost
Subject pgsql: Minor cleanup of column-level priv fix
Date
Msg-id E1YNotT-0000gI-BQ@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
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.

Branch
------
REL9_1_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/cfc14b2bf348528bc8f0f04fe1ff80e7abdf0529

Modified Files
--------------
src/backend/commands/trigger.c           |    6 ------
src/backend/executor/execMain.c          |    9 ---------
src/test/regress/expected/privileges.out |    8 --------
src/test/regress/sql/privileges.sql      |    4 ----
4 files changed, 27 deletions(-)


pgsql-committers by date:

Previous
From: Stephen Frost
Date:
Subject: pgsql: Minor cleanup of column-level priv fix
Next
From: Heikki Linnakangas
Date:
Subject: pgsql: Fix knn-GiST queue comparison function to return heap tuples fir