From 63a601457da46cefe781a9ffd41924327782a78a Mon Sep 17 00:00:00 2001 From: Itagaki Takahiro Date: Wed, 9 Jun 2010 01:00:32 +0000 Subject: [PATCH] Fix connection leak in dblink when dblink_connect() or dblink_connect_u() end with "duplicate connection name" errors. Backported to release 7.4. --- contrib/dblink/dblink.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/contrib/dblink/dblink.c b/contrib/dblink/dblink.c index b239ba33163..69997511abd 100644 --- a/contrib/dblink/dblink.c +++ b/contrib/dblink/dblink.c @@ -2124,9 +2124,14 @@ createNewConnection(const char *name, remoteConn * con) errmsg("out of memory"))); if (found) + { + PQfinish(rconn->conn); + pfree(rconn); + ereport(ERROR, (errcode(ERRCODE_DUPLICATE_OBJECT), errmsg("duplicate connection name"))); + } hentry->rcon = con; strncpy(hentry->name, name, NAMEDATALEN - 1); -- 2.39.5