pgsql: Fix handling of COMMENT for domain constraints - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Fix handling of COMMENT for domain constraints
Date
Msg-id E1hat6p-0000EZ-4B@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix handling of COMMENT for domain constraints

For a non-superuser, changing a comment on a domain constraint was
leading to a cache lookup failure as the code tried to perform the
ownership lookup on the constraint OID itself, thinking that it was a
type, but this check needs to happen on the type the domain constraint
relies on.  As the type a domain constraint relies on can be guessed
directly based on the constraint OID, first fetch its type OID and
perform the ownership on it.

This is broken since 7eca575, which has split the handling of comments
for table constraints and domain constraints, so back-patch down to
9.5.

Reported-by: Clemens Ladisch
Author: Daniel Gustafsson, Michael Paquier
Reviewed-by: Álvaro Herrera
Discussion: https://postgr.es/m/15833-808e11904835d26f@postgresql.org
Backpatch-through: 9.5

Branch
------
REL9_6_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/ff1a25601e750e50a77fe09ceb6ed473f93d8e93

Modified Files
--------------
src/backend/catalog/objectaddress.c        | 24 +++++++++++++++++++++++-
src/test/regress/input/constraints.source  | 15 +++++++++++++++
src/test/regress/output/constraints.source | 14 ++++++++++++++
3 files changed, 52 insertions(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: David Rowley
Date:
Subject: pgsql: doc: Add best practises section to partitioning docs
Next
From: Tom Lane
Date:
Subject: pgsql: Fix ALTER COLUMN TYPE failure with a partial exclusionconstrain