pgsql: Fix handling of COMMENT for domain constraints

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Fix handling of COMMENT for domain constraints
Date: 2019-06-12 02:35:31
Message-ID: E1hat6p-0000EX-3z@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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
------
REL_10_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/56a932533aa2de49d654ca401e6593b3228627ce

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(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2019-06-12 16:30:15 pgsql: Fix ALTER COLUMN TYPE failure with a partial exclusion constrain
Previous Message Tom Lane 2019-06-11 20:41:39 Re: UCT (Re: pgsql: Update time zone data files to tzdata release 2019a.)