pg_dump: Fix quoting of domain constraint names

The original code was adding double quotes to an already-quoted
identifier, leading to nonsensical results.  Remove the quoting call.

I introduced the broken code in 7eca575d1c of 9.5 era, so backpatch to
9.5.

Report and patch by Elvis Pranskevichus
Reviewed by Michael Paquier

diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c
index 9c6f88540e335587027a20f92a7d43e9a106dd16..de1ece17c0c5766831bd487d228d215100a0600f 100644 (file)
--- a/src/bin/pg_dump/pg_dump.c
+++ b/src/bin/pg_dump/pg_dump.c
@@ -9611,7 +9611,7 @@ dumpDomain(Archive *fout, TypeInfo *tyinfo)
        appendPQExpBuffer(labelq, "CONSTRAINT %s ",
                          fmtId(domcheck->dobj.name));
        appendPQExpBuffer(labelq, "ON DOMAIN %s",
-                         fmtId(qtypname));
+                         qtypname);
        dumpComment(fout, labelq->data,
                    tyinfo->dobj.namespace->dobj.name,
                    tyinfo->rolname,